Die Berechnung einer Summe mit C++

Schrieb ich den folgenden code, um die Summe der Reihe (-1)^i*(i/(i+1)). Aber wenn ich es bekomme ich -1 für jeden Wert von n.

Kann jemand bitte erklären, was ich falsch mache? Vielen Dank im Voraus!

#include <iostream>
using namespace std;

int main()
{
    int sum = 0;
    int i = 1.0;
    int n = 5.0;

    for(i=1;i<=n;i++)
        sum = (-1)^i*(i/(i+1));

    cout << "Sum" <<" = "<< sum << endl;
    return 0;
}
Was denken Sie, die ^ Betreiber übernimmt? Sind Sie versuchen, durchführen einer exklusiv-oder-operation, oder versuchen Sie zu führen Potenzierung?
Und warum weisen Sie eine Gleitkommazahl auf eine Ganzzahl? (int i = 1.0)
gut ersten Ihrer Einstellung Summe jedem einzelnen Durchlauf der Schleife und nicht hinzufügen, um den vorherigen Wert. zweite ^ nicht aufbringen einer Kraft, wie Sie denken würde. check-out der c Mathematik-Bibliothek. und watch out für die impliziten Typumwandlungen, die passieren, wenn Sie teilen sich in eine Fraktion.

InformationsquelleAutor Surfcast23 | 2012-02-03

  1. 14

    Problem #1: Die C++ ^ Betreiber ist nicht der Mathe-power-operator. Es ist eine bitweise XOR.

    Sollten Sie verwenden pow() statt.

    Problem #2:
    Sie sind speichern von Gleitkomma-Datentypen in einen integer-Typ. So wird Folgendes Ergebnis in integer-division (division abgeschnitten):

    i/(i+1)

    Problem #3:
    Sie sind tatsächlich nicht Fazit etwas:

    sum = ...

    werden sollte:

    sum += ...

    Eine korrigierte version des Codes ist wie folgt:

    double sum = 0;
int i = 1;
int n = 5;

for(i = 1; i <= n; i++)
    sum += pow(-1.,(double)i) * ((double)i / (i + 1));

    Obwohl Sie wirklich brauchen nicht zu verwenden pow in diesem Fall. Einen einfachen test auf gerade/ungerade zu tun.

    double sum = 0;
int i = 1;
int n = 5;

for(i = 1; i <= n; i++){
    double val = (double)i / (i + 1);
    if (i % 2 != 0){
        val *= -1.;
    }
    sum += val;
}
    1 Fehler unten, 2 zu gehen mehr.
    Zusätzlich, da alle seine Variablen ints wird es tun integer math, und i/(i+1) wird immer bewerten, zu 1, es sei denn i == 0 oder i==1
    zu 0 in der Tat.
    vergessen Sie nicht, über die Summe +=
    Das war überraschend schwer zu fangen... hat mich ein paar min.

    InformationsquelleAutor Mysticial

  2. 4

    Müssen Sie zu setzen Summe += pow(-1,i)*(i/(i+1));

    Sonst verlieren Sie bisherige Ergebnis jedes mal.

    Verwenden pow-Funktion für pow Betrieb.

    edit : wie gesagt im anderen post, verwenden Sie double oder float statt int zu vermeiden, abgeschnittene division.

    InformationsquelleAutor ClemPi

  3. 3

    Wie über diese

    ((i % 2) == 0 ? 1 : -1)

    statt

    std::pow(-1, i)

    ?

    Ausführliche Antwort:

    double sum = 0;
int i = 1.0;
int n = 5.0;
for (i = 1; i <= n; ++i) {
    signed char sign = ((i % 2) == 0 ? 1 : -1);
    sum += sign * (i / (i+1));
}

    InformationsquelleAutor Sergey Brunov

  4. 2

    Paar Probleme:

    1. ^ teh bitweise exklusiv-oder-in c++ nicht "erhabene macht". Verwenden pow () - Methode.

    2. Entfernen Sie die baumelnden öffnenden Klammer der letzten Zeile

    3. Verwendung von ints nicht schwebt, wenn die Zuweisung an int-Werten.

    InformationsquelleAutor Sid

  5. 2

    Scheinen Sie ein paar Dinge, die falsch mit Ihrem code:

    using namespace std;

    Dies ist nicht direkt mit Ihrem problem an die hand, aber nicht immer sagen using namespace std; Es führt subtile bugs.

    int i = 1.0;
int n = 5.0;

    Sind Sie initializaing integralen Variablen mit floating-point-Konstanten. Versuchen

    int i = 1;
int n = 5;

    sum = (-1)^i*(i/(i+1));

    Haben Sie zwei Probleme mit diesem Ausdruck. Erstens, die Menge (i/(i+1)) ist immer null. Denken Sie daran die Verteilung von zwei ints Runden das Ergebnis. Zweitens ^ tun nicht, was Sie denken, es tut. Es ist die exklusiv-oder-operator, nicht den potenzierungsoperator. Drittens ^ bindet weniger stark als *, so dass Ihr Ausdruck ist:

    -1 xor (i * (i/(i+1)))

-1 xor (i * 0)
-1 xor 0
-1
    was für Fehler nicht using namespace std verursachen, link?
    Diskussion hier und here.
    Danke für die links bezüglich der Verwendung von namespace std.

    InformationsquelleAutor Robᵩ

  6. 1

    ^ nicht tun, was Sie denken, es tut. Es gibt auch einige andere Fehler in Ihrem code.

    Was es sein sollte:

    #include <iostream>
#include <cmath>

int main( )
{
    long sum = 0;
    int i = 1;
    int n = 5;

    for( i = 1; i <= n; i++ )
        sum += std::pow( -1.f, i ) * ( i / ( i + 1 ) );

    std::cout << "Sum = " << sum << std::endl;

    return 0;
}

    Nehmen eine Leistung von Wert, nutzen std::pow (siehe hier). Auch Sie nicht zuordnen können int ein decimal-Wert. Für die, die Sie verwenden müssen float oder double.

    Den genannten ^ ist Bitweises XOR, nicht eine mark für ein exponent.

    Seien Sie auch vorsichtig Integer-Arithmetik wie erhalten Sie möglicherweise unerwartete Ergebnisse. Sie wahrscheinlich wollen, um zu ändern Ihre Variablen entweder float oder double.

    Vielen Dank an alle, die geantwortet haben!

    InformationsquelleAutor ssell

  7. 1

    Gibt es ein paar Probleme mit dem code:

    int sum = 0;

    Zwischenergebnisse sind nicht ganze zahlen, sollte dies eine doppelte

    int i = 1.0;

    Da Sie verwenden eine division, sollte es ein Doppel -, 1/2 ist 0, wenn, berechnet in Ganzzahlen.

    int n = 5.0;

    Dies ist ein int, nicht eine floating-point-Wert, keine .0 benötigt wird.

        for(i=1;i<=n;i++)

    Hast du bereits initialisiert ist ich 1, warum es wieder tun?

    sum = (-1)^i*(i/(i+1));

    Jeder iteration verlieren Sie den vorherigen Wert, den Sie verwenden sollten, Summe+= 'neue Werte'

    Auch, Sie brauchen nicht zu pow zu berechnen (-1)^i alle diese tut, ist der Schalter zwischen +1 und -1, je nach der gerade/ungerade status i. Sie tun kann dieser leichter mit einem if - Anweisung oder mit 2 for's, eine für ungerade i eine für sogar diejenigen... Viele Möglichkeiten wirklich.

    InformationsquelleAutor Andrei

Schreibe einen Kommentar