Berechnung falsch

[Chypsylon]

Hallo
ich möchte auf meinem AVR folgende Rechnung durchführen.

 
uint16_t line_estimate = 0;
uint32_t wa_numerator = 0;
uint16_t wa_denominator = 0;

uint16_t line_values[8];  //Werte zwischen 0...1023

[...] //line values werden eingelesen und befüllt

 for(uint8_t i=0;i<8;i++)
{
  wa_numerator += line_values[i] * (i+1) *1000;
  wa_denominator += line_values[i];
}

line_estimate = wa_numerator/wa_denominator;

//Ausgabe
char buffer[20];
    lcd_puts(itoa(line_estimate, buffer, 10));

Allerdings funktioniert das nicht richtig. Wenn z.b alle Elemente von line_values 0 sind und nur das 1 einen anderen Wert hat müsste line_estimate 1000 sein, es schwankt aber nur "wild" hin und her und erreicht zeitweise negative bzw. Werte kleiner 1000 die es niemals erreichen dürfte. Meine Vermutung ist das ich noch irgendwo "typecasten" muss, ich weiss aber nicht wo
Hat irgendjemand eine Erklärung für dieses seltsame Verhalten bzw. kann mir sagen wo der Fehler liegt?

[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?