Funktion, die Zahl aus Zeichen zusammenbastelt

[Martin S. (tungl)]

Hallo,

ich habe eine Funktion geschrieben, der man die Anzahl der zu 
erwartenden Zeichen/Ziffern, die ueber UART reinkommen, mit gibt und die 
so lang darauf wartet, bis genug Zeichen eingetroffen sind und danach 
dann eine Zahl daraus machen soll.

Das Ganze sieht so aus:

int expectNumber(int digits) {
    char input;
    int r = 0;
    int i;
    
    for (i=1; i<=digits; i++)
    {
        input = UARTReceive();
        while (!characterIsNumber(input)) {
            input = UARTReceive();
        }
        
        int n = atoi(&input);
  
        r += (10^(digits-i)) * n;
        
        char buf[50];
        sprintf(buf, "\n\n Char is: %c, n is: %d, r is now: %d\n", input,n,r);
        UARTPuts(buf);
    }
    return r;
}

Das rufe ich nun zum Beispiel mit expectNumber(2) auf, um 2 Ziffern zu 
erwarten.
Nun wuerde ich erwarten, dass, wenn ich z.B. als erstes Zeichen 1 
uebermittle, r im ersten Schleifendurchlauf dann den Wert (10^1)*1=10 
annimmt. Tatsaechlich kommt aber 11 raus. Was ich mir nicht wirklich 
erklaeren kann.
Uebersehe ich was?

Viele Gruesse,
Martin

[gagast (Gast)]

>10^(digits-i)
^ ist XOR, ist das gewollt?

[Martin S. (tungl)]

gagast schrieb:
>>10^(digits-i)
> ^ ist XOR, ist das gewollt?
Oh. Ohhhh. Danke. Das soll natuerlich kein XOR sein, sondern ein "hoch". 
Und ich zerbrech mir Stunden den Kopf darueber, warum das jetzt nicht 
geht.

Vielen Dank fuer die Erleuchtung!

[Rolf M. (rmagnus)]

Martin S. schrieb:
> int n = atoi(&input);

input ist ein char. atoi erwartet einen String, also ein 
nullterminiertes Array aus char.   Wenn du die Adresse von input 
übergibst, nimmt atoi das und dann noch alles, was danach zufällig so im 
Speicher steht, bis es dann eine 0 findet und wertet alles zusammen als 
den übergebenen String.

[Martin S. (tungl)]

Rolf Magnus schrieb:
> input ist ein char. atoi erwartet einen String, also ein
> nullterminiertes Array aus char.   Wenn du die Adresse von input
> übergibst, nimmt atoi das und dann noch alles, was danach zufällig so im
> Speicher steht, bis es dann eine 0 findet und wertet alles zusammen als
> den übergebenen String.

Danke fuer die Erlaeuterung. Ich habe mir da auch schon Gedanken 
gemacht, aber so funktioniert es. Was waere denn die korrekte Loesung?

[Karl H. (kbuchegg)]

Martin S. schrieb:

> Danke fuer die Erlaeuterung. Ich habe mir da auch schon Gedanken
> gemacht, aber so funktioniert es.

Das ist kein Argument. Es 'funktioniert' höchstens zufällig

> Was waere denn die korrekte Loesung?

  n = input - '0';

[Karl H. (kbuchegg)]

und als ganzes wäre eine einfache Lösung

(und gewöhn dir an, dass wir bei 0 anfangen zu zählen)

    r = 0;

    for (i=0; i<digits; i++)
    {
        do {
          input = UARTReceive();
        } while (!characterIsNumber(input));

        r = 10*r + (input - '0'); 
    }

oder

     r = 0;
     i = 0;

     while( i < digits )
     {
       input = UARTReceive();
       if( characterIsNumber(input) )
       {
         r = 10*r + (input - '0');
         i++;
       }
     }

PS: dir ist bekannt, dass es eine Standardfunktion isdigit() gibt?

[Martin S. (tungl)]

Vielen Dank noch fuer die Anregungen. Nein, war mir nicht bewusst, dass 
es die Funktion schon gibt. Habe eigentlich erst heute damit angefangen 
und bin noch nicht wirklich heimisch. ;-)

[Karl H. (kbuchegg)]

Martin S. schrieb:
> Vielen Dank noch fuer die Anregungen. Nein, war mir nicht bewusst, dass
> es die Funktion schon gibt. Habe eigentlich erst heute damit angefangen
> und bin noch nicht wirklich heimisch. ;-)

Ein C-Buch durcharbeiten kann wahre Wunder bei fehlendem Wissen wirken.
zb
http://openbook.galileocomputing.de/c_von_a_bis_z/

[Peter D. (peda)]

Martin S. schrieb:
> die ueber UART reinkommen, mit gibt und die
> so lang darauf wartet, bis genug Zeichen eingetroffen sind

Damit läuft man Gefahr, daß Dein Programm hängen bleibt und nichts 
anderes mehr machen kann.
Die übliche Methode ist daher, daß man von der UART in einen Puffer 
einliest. Und erst, wenn z.B. ein Endezeichen (z.B. CR,LF) emfangen 
wurde, übergibt man den Puffer zur Auswertung.
So wird nicht unnütz CPU-Zeit mit Warten vergeudet.


Peter