Serielle Schnittstelle calculator

[Serieousl L. (stephan2013)]

•  Einlesen d. 1. Operanten(scanf)   // 57
•  Einlesen d Operator(USART Getchar)  +
•  Einlesen d. 2. Operanten(scanf)    // 277
•  Einlesen d Operator(Getchar)    =
Berechnen und Ausgeben der Ergebnisse in d. Form 57+277= ….

Also im Prinzip ein Taschenrechner
Ich weiß nicht was hier falsch ist :/

#include <avr/io.h>
#include "Wait.h"
#include "USART.h"

int main(void)
{  int opnd1, opnd2;
  uint8_t oprt1, oprt2;
  uint8_t operator1, operator2;
  uint8_t erg;
  USARTInit(0,9600,1,0,1,0);

  scanf("%d", &opnd1);
    operator1 = USARTGetChar(scanf("%d", &oprt1));
    // oprt1= USARTReceive(0);
    scanf("%d", &opnd2);
    //oprt2= USARTReceive(0);
    operator2 = USARTGetChar(scanf("%d", &oprt2));
    while(1)
    {
    
    }
  switch (operator1)
    {
    case '*' : erg= oprt1*oprt2; break;
    case '+' : erg= oprt1+oprt2; break;
    case '-' : erg= oprt1-oprt2; break;
    case '/' : erg= oprt1/oprt2; break;
    }
      
  printf("%d %s %d %s %d", opnd1, operator1, opnd2, operator2, erg);
}

Getchar:
************************************************************************ 
*******/

[Roland .. (rowland)]

hisham moustafa schrieb:
> while(1)
>     {
>
>     }
>   switch (operator1)
>     {
>     case '*' : erg= oprt1*oprt2; break;
>     case '+' : erg= oprt1+oprt2; break;
>     case '-' : erg= oprt1-oprt2; break;
>     case '/' : erg= oprt1/oprt2; break;
>     }

Die Switchverzweigung wird nie erreicht, weil das Programm vorher schon 
in der Enlosschleife hängt.

Die while(1)-Schleife muss über den ganzen Code fassen, damit die 
Ein-Ausgabe zyxklisch widerholt wird.

[Serieousl L. (stephan2013)]

Roland ... schrieb:
> hisham moustafa schrieb:
>> while(1)
>>     {
>>
>>     }
>>   switch (operator1)
>>     {
>>     case '*' : erg= oprt1*oprt2; break;
>>     case '+' : erg= oprt1+oprt2; break;
>>     case '-' : erg= oprt1-oprt2; break;
>>     case '/' : erg= oprt1/oprt2; break;
>>     }
>
> Die Switchverzweigung wird nie erreicht, weil das Programm vorher schon
> in der Enlosschleife hängt.
>
> Die while(1)-Schleife muss über den ganzen Code fassen, damit die
> Ein-Ausgabe zyxklisch widerholt wird.

int main(void)
{  int opnd1, opnd2;
  uint8_t oprt1, oprt2;
  uint8_t operator1, operator2;
  uint8_t erg;
  USARTInit(0,9600,1,0,1,0);

  
    while(1)
    {
    scanf("%d", &opnd1);
    operator1 = USARTGetChar(scanf("%d", &oprt1));
    // oprt1= USARTReceive(0);
    scanf("%d", &opnd2);
    //oprt2= USARTReceive(0);
    operator2 = USARTGetChar(scanf("%d", &oprt2));
    
    switch (operator1)
    {
    case '*' : erg= oprt1*oprt2; break;
    case '+' : erg= oprt1+oprt2; break;
    case '-' : erg= oprt1-oprt2; break;
    case '/' : erg= oprt1/oprt2; break;
    }
      
  printf("%d %s %d %s %d", opnd1, operator1, opnd2, operator2, erg);
    }

}

[Rufus Τ. F. (rufus)]

hisham moustafa schrieb:
> operator1 = USARTGetChar(scanf("%d", &oprt1));

Ich glaube nicht, daß USARTGetChar etwas mit dem Rückgabewert von 
scanf anfangen kann.

[Serieousl L. (stephan2013)]

Oke..
Was macht USARTGetChar genau dann ?
oder welchen Übergabeparameter muss ich hineinschreiben ?

[DirkB (Gast)]

Du solltest dich erstmal mit scanf und printf auseinander setzen.
Das sind zwei unterschiedliche Funktionen,.
Was sie jedoch gemeinsam haben ist, dass die variablen Parameter zum 
Formatstring passen müssen.

Beispiele:
%d bei scanf dient zum Einlesen von einem dezimalen int. (das ist i.A 
kein uint8_t)
%s bei printf dient zur Ausgabe von einer nullterminierten Zeichenkette 
und erwartet einen Zeiger darauf.

Du darfst aber nicht von dem Verhalten von printf auf das Verhalten von 
scanf schließen (und umgekehrt)

[Floh (Gast)]

hisham moustafa schrieb:
> Form 57+277= ….
> uint8_t operator1, operator2;
>   uint8_t erg;

277 passt nicht in einen uint8_t, das Ergebnis der Addition noch 
weniger.
:-)

[Serieousl L. (stephan2013)]

Floh schrieb:
>> uint8_t operator1, operator2;

operator ist nicht die Zahl, sondern (+,-,*,/)

[Serieousl L. (stephan2013)]

jetzt habe ich das geändert wie Sie es gesagt haben aber:
beim Terminal wenn die Zahlen schreibe schreibt er nach einem 
Operator(zB: +) nichts :(

int main(void)
{  int opnd1, opnd2;
  uint8_t oprt1, oprt2;
  char operator1, operator2;
  uint8_t erg;
  USARTInit(0,9600,1,0,1,0);

  
    while(1)
    {
    scanf("%d", &opnd1);
    operator1 = USARTReceive(0);
    scanf("%d", &opnd2);
    operator2 = USARTReceive(0);
    switch (operator1)
    {
    case '*' : erg= oprt1*oprt2; break;
    case '+' : erg= oprt1+oprt2; break;
    case '-' : erg= oprt1-oprt2; break;
    case '/' : erg= oprt1/oprt2; break;
    }
  printf("%d %c %d %c %d", opnd1, operator1, opnd2, operator2, erg);
    }

}

[Dirk B. (dirkb2)]

Was wird denn wie eingegeben?
scanf lässt alles, was nicxht zur Zahl passt im Eingabestrom stehen.
Das ist auch das Leerzeichen oder das '\n' von der Returntaste.
Wozu überhaupt das USARTReceive ?

scanf("%d %c %d %c", &opnd1, &operator1, &opnd2, &operator2);

Wobei die Leerzeichen auch eine Bedeutung haben.
Wofür ist operator2 ?

Und erg sollte doch auch vom Typ der Operanden sein.