Berechnung falsch

[sternst]

Diese Berechnung

line_values[i] * (i+1) *1000

erfolgt in uint16_t. Für line_values[7] z.B. läuft das schon für alles >8 über.

[Chypsylon]

Diese Berechnung

line_values[i] * (i+1) *1000

erfolgt in uint16_t. Für line_values[7] z.B. läuft das schon für alles >8 über.

Deswegen ist wa_numerator ja auch als 32bit deklariert?

 
uint32_t wa_numerator = 0;
...
wa_numerator += line_values[i] * (i+1) *1000;

Oder seh ich grad den Wald vor lauter Bäumen nicht

[sternst]

Deswegen ist wa_numerator ja auch als 32bit deklariert?

Und?
Der Typ auf der linken Seite einer Zuweisung hat keinerlei Einfluss darauf, wie auf der rechten Seite der Ausdruck ausgewertet wird.

[Chypsylon]

Ich hab auch schon probiert die rechte Seite als long zu casten, hat aber nicht wirklich was geändert...

wa_numerator += (long)(line_values[i]) *((i+1) *1000);
bzw.
wa_numerator += (long)(line_values[i] *((i+1) *1000));

[sternst]

Was soll das "bzw." denn bedeuten? Die beiden Zeilen sind in ihrem Verhalten grundverschieden. Was genau hast du denn nun ausprobiert?

[Chypsylon]

Ich hab beide Varianten probiert. Meinem Verständnis nach sollte die erste eigentlich richtig sein

[sternst]

Dann liefere doch mal ein konkretes Beispiel. Was steht in line_values und was kommt als Ergebnis raus?