AVR Rechnet nicht richtig.

[Johannes (Gast)]

Hallo. Folgendes problem:

void auswertung (void) {
  unsigned char buffer [5] = {0,0,0,0,0};
  itoa (count, buffer, 10);  
  lcd_gotoxy (0,0);
  lcd_puts ("count: ");
  lcd_puts (buffer);

  long time;
  time = count * 547;
  unsigned char buffer2 [10] = {0,0,0,0,0,0,0,0,0,0};
  ltoa (time, buffer2, 10);
  lcd_puts ("\ntime: ");
  lcd_puts (buffer2);  
}

count enthält den wert 190.
wird auch richtig auf dem display angezeigt.
nun müsste logischerweise als 2. wert 105.024 (192 * 547) aufm display
stehen. es steht aber 28.105 auf dem display !?

was mach ich falsch ?

[Benedikt (Gast)]

Auf 28105 komme ich nicht, bei mir sind es 39488.
Was auf jedenfall falsch ist, es fehlt long, denn der Compiler rechnet
nur mit int, wenn count auch vom typ int ist:

time = (long)count * 547;

[Thomas S. (tstuetz)]

Außerdem solltest du den beiden Puffern noch ein zusätzliches Zeichen
spendieren - Strings in C haben meist noch ein "\0"=0x00 am Ende

du solltest auch anstatt long/int/char vielleicht
unsigned long/unsigned int/unsigned char verwenden.
Angenommen count ist ein char => 190=C0h => oberstes Bit gesetzt
Zahl is negativ, bei der Berechnung kommt was negatives raus.

Gruss

[Thomas S. (tstuetz)]

Sorry, kleiner Lapsus: 190 ist BEh und nicht C0h (192) das besagte 7.Bit
ist aber auch da gesetzt.

[Uwe Nagel (Gast)]

Der Compiler macht leider keine automatische Typwandlung, wenn man zwei
16-Bit Werte multipliziert und sie einer 32-Bit Variablen zuweist.
Bezihungsweise macht er die Typwandlung erst nach der Multiplikation.
Dadurch gehen die oberen 16 Bit der Produkts verloren.
Also schreib entweder:
time = (long)count * 547;
oder
time = count * 547L;

Uwe