Festkommaarithmetik und itoa Problem

[Stefan H. (hohmi)]

Hallo Community,

ich möchte ein Gerät zur Neigungsmessung zusammen bauen und via LCD die 
Werte anzeigen lassen.

der Bereich geht von -90.0° bis 90.0°.

Jetzt wollte ich zu testzwecken auf meinem myAVR Board (inkl. LCD) das 
ganze schon mal mit dem LCD fertigstellen bis das IC und das neue LCD 
kommt.

Also Poti an den ADC und ne kleine Rechnung um meinen Werte-Bereich 
anzeigen lassen zu könnrn:

  // Wert aus ADC in Buffer übertragen
  buffer = ADC_Wandler(0);
  
  // Umwandlung von -500 bis +500; x ist eine int32_t und textfeld eine char[12] Variable
  x = (((1000*buffer)/255)-500);

  itoa (x, textfeld, 10);

  //Variable textfeld ausgeben
  lcd_goto(2, 10);
  _delay_ms(10);
  lcd_writeText(textfeld, 4);
  _delay_ms(10);

ADC ist auf 0-255 skaliert und mit der kleinen Rechnung, müsste dann 
-500 bis +500 als Ergebnis erscheinen. Um nun das "," einfügen zu können 
hab schnell diesen Code zusammen gebastelt (ist nicht schön ich weiß, 
aber für meine Zwecke vollkommend ausreichend):

//Variable textfeld mit ","
lcd_goto(1, 4);
if ((x >= (-999)) && (x <= (-100))) {
      lcd_write('-');
      lcd_write(textfeld[1]);
      lcd_write(textfeld[2]);
      lcd_write(',');
      lcd_write(textfeld[3]);
    } else if ((x >= (-99)) && (x <= (-10))) {
      lcd_write('-');
      lcd_write(' ');
      lcd_write(textfeld[1]);
      lcd_write(',');
      lcd_write(textfeld[2]);
    } else if ((x >= (-9)) && (x < 0)) {
      lcd_write('-');
      lcd_write(' ');
      lcd_write('0');
      lcd_write(',');
      lcd_write(textfeld[0]);
    }else if (x == 0) {
      lcd_write(' ');
      lcd_write(' ');
      lcd_write('0');
      lcd_write(',');
      lcd_write('0');
    } else if ((x > 0) && (x <= 9)) {
      lcd_write(' ');
      lcd_write(' ');
      lcd_write('0');
      lcd_write(',');
      lcd_write(textfeld[0]);
    } else if ((x >= 10) && (x <= 99)) {
      lcd_write(' ');
      lcd_write(' ');
      lcd_write(textfeld[0]);
      lcd_write(',');
      lcd_write(textfeld[1]);
    } else if ((x >= 100) && (x <=999)) {
      lcd_write(' ');
      lcd_write(textfeld[0]);
      lcd_write(textfeld[1]);
      lcd_write(',');
      lcd_write(textfeld[2]);
    }

Aber aus mir unerklärlichen Gründen kommt bei x ein falscher Wert raus.

Daher hab ich den ganzen Code kopiert und in Dev-C++ eingefügt und alle 
lcd_write befehle durch print_f ersetzt --> Es funktioniert ohne wenn 
und aber.

Das einzige was ich mir erklären kann, ist das der DEV-C Kompiler eine 
andere Biblitothek für itoa benutzt als der des AVR Studio 6.

Habe auch schon den ABS vonx gebildet und dann erst gewandelt. Ebenso 
die Rechnung in einzelen Rechenschritte einzuteilen mit 
Zwischenergebnissen, es klappt im AVR Strudio einfach nicht.

Hat einer von euch nen Rat? Code sind im Anhang, danke euch im Voraus. 
Ich weiß nicht mehr weiter...

mfg
Hohmi

[Stefan H. (hohmi)]

Sorry, da war noch ein grober Schnitzer im ADC Kommentar, nicht dass das 
einer so abpinselt und sich wundert^^.

mfg
Hohmi

[Peter II (Gast)]

> uint8_t buffer = 0;
> x = (((1000*buffer)/255)-500);

wenn buffer > 66 ist dann wird 1000*buffer > 66000 und das Passt nicht 
mehr in einen int.

 x = (((1000l*buffer)/255)-500);

müsste funktionieren.

[Andreas H. (ahz)]

Huhu Stefan

Nur mal ein Schuss ins Blaue:

Stefan H. schrieb:
> x = (((1000*buffer)/255)-500);

Wenn buffer = 255 ist, wird der Wert (1000 * buffer) natürlich 255000.
Das ist aber vermutlich größer als der MAXINT des AVRs.

Mit anderen Worten: Du hast da einen Zahlenüberlauf. Darum klappt das 
dann auch auf dem PC, der ja mit 32, bzw. 64 bit rechnet.

Grüße
Andreas

[Andreas H. (ahz)]

Peter II schrieb:
> wenn buffer > 66 ist dann wird 1000*buffer > 66000 und das Passt nicht
> mehr in einen int.

Ups, da war Peter schneller ;-)

Grüße
Andreas

[Falk B. (falk)]

Siehe Festkommaarithmetik.

[Johann L. (gjlayde)]

Peter II schrieb:
>  1000l

Besser: 1000L dann gibt's auch keine Fehler beim Lesen (ja nach 
Firle-Font).

[Stefan H. (hohmi)]

Sooooooooo, danke an alle!!!!
an der buffer Variabel hat es nicht gelegen. Die hatte ich als int32_t 
deklariert.

Die Festkommaarithmetik hatte ich mir schon durchgelesen aber nicht 
genau genug...

Die Lösung:

"Man sollte mindestens eine Variable auf der rechten Seite genauso groß 
deklarieren wie das Ergebnis auf der linken Seite oder einen Cast 
benutzen."

ein cast dabei und das Problem hat sich in Luft aufgelöst:

x = (((1000 * (long) buffer)/255)-500);

Danke an alle die daran mitgewirkt haben!

mfg Hohmi

[Mike (Gast)]

Stefan H. schrieb:
> "Man sollte mindestens eine Variable auf der rechten Seite genauso groß
> deklarieren wie das Ergebnis auf der linken Seite oder einen Cast
> benutzen."

Da ist ein dicker Übersetzungsfehler drin. Richtig muss es heißen:
"Man muss mindestens ...", sonst nützt das alles nichts.

[Falk B. (falk)]

@ Stefan H. (hohmi)

>an der buffer Variabel hat es nicht gelegen. Die hatte ich als int32_t
>deklariert.

Dann hätte die Rechnung funktioniert. In deinem geposteten Code war sie 
uint8_t!

http://www.mikrocontroller.net/attachment/highlight/217499

>ein cast dabei und das Problem hat sich in Luft aufgelöst:

wenn buffer WIRKLICH uint32_t ist, ist der cast überflüssig.

[Stefan H. (hohmi)]

Hast recht falk!

Ich meinte die Variable x.

Sorry für den vertipper

mfg