Kleiner Taschenrechner

Oneida

Aufgabe:
Entwickeln Sie eine Konsolenanwendung "Taschenrechner", welche nach Eingabe einer ersten Zahl a, des Rechenzeichens (+,-,* oder /) und einer zweiten Zahl b das Ergebnis oder ggf. eine Fehlermeldung ausgibt.

Hier mein Quelltext:

Code

int main()
{
    cout << " Geben Sie die erste Zahl ein: ";
    double a,b,erg;
    int op;
    cin >> a;

    cout << " Geben Sie den Operator (+,-,*,/) ein: ";
    cin >> (op);


    cout << " Geben Sie die zweite Zahl ein ";
    cin >> b;

    switch (op)
    {
              case '+':
              {
                   erg = a+b;
                   cout << "Das Ergebnis lautet: " << erg ;
                   break;
              }
              case '-':
              {
                   erg = a-b;
                   cout << " Das Ergebnis lautet: " << erg;
                   break;
              }
              case '*':
              {
                   erg = a*b;
                   cout << " Das Ergebnis lautet: " << erg;
                   break;
              }
              case '/':
              {
                   break;
              }
              default:
              {
                cout << " Falscher Operator ";   //sonst
              }
    } //end of Switch

  getch();    
}

Alles anzeigen

folgendes Problem: Nach der Eingabe der ersten Zahl und dem Operator (+,-,*,/) erscheint gleichzeitig:
Geben sie die zweite Zahl ein Falscher Operator

Könnte mir jemand sagen warum das so ist?

The User

Tach!

Wenn du einen Integer hast (op), so werden bei der Eingabe normale Ziffern und evtl. ein Vorzeichen erwartet. Du gibst allerdings ein Zeichen ein. Der Operator landet in einem undefinierten Zustand (er wurde nie zugewiesen) und cin gerät in einen Fehlerzustand, sodass weiter Eingaben nicht abgewartet werden. Deshalb solltest du als Typen für op char (Zeichen) wählen, dann kannst du ein beliebiges Zeichen eingeben. Käme es auf besondere Stabilität an, solltest du die Variablen entweder initialisieren oder prüfen, ob Eingabefehler aufgetreten sind (das machst du mit "if(!cin)", aber ich vermute, dass das an dieser Stelle weniger wichtig ist.

Ach ja: Verwende bitte die [ code ] - Tags.

Viele liebe Grüße
The User

Oneida

Hallo,

vielen dank, war wirklich nur die Umwandlung von int in char.

Hier der funktionierende code:

C

#include <cstdlib>
#include <iostream>
#include <conio.h>
#include <math.h>


using namespace std;


int main()
{
    cout << " Geben Sie die erste Zahl ein: ";
    double a,b,erg;
    char op;
    cin >> a;    

    cout << " Geben Sie den Operator (+,-,*,/) ein: ";
    cin >> (op);     

    cout << " Geben Sie die zweite Zahl ein ";
    cin >> b;

    switch (op)
    {
              case '+':
              {
                   erg = a+b;
                   cout << "Das Ergebnis lautet: " << erg ;
                   break;
              }
              case '-':
              {
                   erg = a-b;
                   cout << " Das Ergebnis lautet: " << erg;
                   break;
              }
              case '*':
              {
                   erg = a*b;
                   cout << " Das Ergebnis lautet: " << erg;
                   break;
              }
              case '/':
              {
                   erg = a/b;
                   cout << " Das Ergebnis lautet: " << erg;
                   break;
              }
              default:
              {
                cout << " Falscher Operator ";   //sonst
              }
    } //end of Switch

  getch(); 




}

Alles anzeigen

-close-