AVR-GCC: Umrechnug array nach integer macht Probleme

[Marcus (Gast)]

Ich benutze einen ATMEGA128 mit AVRStudio 4.14 und WinAVR20080610 und
möchte aus einem array einige Stellen auslesen und in dann in ein 32bit 
Integer speichern. Mit dem unten angegebenen Code gibt es aber Probleme. 
Das Problem liegt bei frequenz[4]. Wenn dort eine Null steht wird alles 
problemlos umgerechnet. Steht dort jedoch eine Zahl die größer ist als 3 
bekomme ich nur noch falsche Werte angezeigt.
Z.B.: im Array steht "12.345,678" wird freq = 12280142
Auch im Simulator kommen falsche Werte zustande.
Hat jemand eine Idee?

char frequenz[11] = "12.345,678";
uint32_t freq = 0;
  // umrechnen vom array nach integerzahl 
  freq =   (frequenz[0] - '0') * 10000000 + (frequenz[1] - '0') * 1000000 + (frequenz[3] - '0') * 100000 + 
      (frequenz[4] - '0') * 10000 + (frequenz[5] - '0') * 1000 + (frequenz[7] - '0') * 100 +
      (frequenz[8] - '0') * 10 + (frequenz[9] - '0');
  sprintf(Buffer, "%10lu ", freq);
  lcd_gotoxy(0,3);
  lcd_puts(Buffer);

Gruß

Marcus

[Helmut L. (helmi1)]

Wenn Frequenz von [4] grosser ist als 3 bekommst du an der Stelle einen 
ueberlauf weil der Compiler erstmal nur mit 16 Bit rechnet. Und das 
reicht nur von -32768 .. 32767

(frequenz[4] - '0') * 10000

Schreibe lieber mal

(frequenz[4] - '0') * 10000L

Dann macht er auch eine Longmultiplikation mit 32 Bit

Das gilt auch fuer die anderen Stellen.

Gruss Helmi

[Johann L. (gjlayde)]

char * frequenz = "12.345,678";

Deine Initializer macht sowas wie

frequenz[0] = (char) "12.345,678";

Nimmt also die Adresse von "12.345,678", schnippelt ab Bit 8 alles ab 
und macht das nach [0].

[Stefan E. (sternst)]

Johann L. wrote:

> Deine Initializer macht sowas wie
>
>

> frequenz[0] = (char) "12.345,678";
>

>
> Nimmt also die Adresse von "12.345,678", schnippelt ab Bit 8 alles ab
> und macht das nach [0].

Nein, es macht genau das, was der OP davon erwartet.

[Marcus (Gast)]

@ Helmut Lenzen
Vielen Dank für die Lösung.

Gruß

Marcus