#include <iostream>

using namespace std;

int addition(int lhs, int rhs)
{
    return lhs+rhs;
}

int subtraktion(int lhs, int rhs)
{
    return lhs-rhs;
}

int multiplikation(int lhs, int rhs)
{
    return lhs*rhs;
}

int division(int lhs, int rhs)
{
    if(rhs==0) return 0;
    else return lhs/rhs;
}

int main()
{
    int a,b;
    char input;

    cout<<"Geben sie 2 Zahlen ein, mit denen sie rechnen möchten. \n";
    cin>>a;
    cin>>b;

    cout<<"Geben sie nun ein wie gerechnet werden soll. \n"
        <<"1) Addieren \n"
        <<"2) Subtrahieren \n"
        <<"3) Multiplizieren \n"
        <<"4) Dividieren \n"
        <<"q) Quit \n";

    cin>>input;
    switch(input)
    {
         case 1:
              cout<<"Das Ergebnis aus "<<a<<" + "<<b<<" ist "<<addition(a,b)<<endl;
         break;
         case 2:
              cout<<"Das Ergebnis aus "<<a<<" - "<<b<<" ist "<<subtraktion(a,b)<<endl;
         break;
         case 3:
              cout<<"Das Ergebnis aus "<<a<<" x "<<b<<" ist "<<multiplikation(a,b)<<endl;
         break;
         case 4:
              cout<<"Das Ergebnis aus "<<a<<" : "<<b<<" ist "<<division(a,b)<<endl;
         break;
         case 'q':
         return EXIT_SUCCESS;
    }
    system("PAUSE");
}

PS: Ich bin ein C++ Anfänger, deswegen bitte die Lösung ein wenig Anfängerfreundlich formulieren.

Zwar können char-Werte implizit in Ganzzahlen umgewandelt werden, aber dabei gilt halt '1' != 1 etc.

Du musst entweder den character in eine Ganzzahl umwandeln oder deine Switch anpassen. (1 durch '1' ersetzen, 2 durch '2' etc.)

#include <iostream>

using namespace std;

int addition(int lhs, int rhs)
{
    return lhs+rhs;
}

int subtraktion(int lhs, int rhs)
{
    return lhs-rhs;
}

int multiplikation(int lhs, int rhs)
{
    return lhs*rhs;
}

int division(int lhs, int rhs)
{
    if(rhs==0) return 0;
    else return lhs/rhs;
}

int main()
{
    int a,b;
    char input;

    cout<<"Geben sie 2 Zahlen ein, mit denen sie rechnen möchten. \n";
    cin>>a;
    cin>>b;

    cout<<"Geben sie nun ein wie gerechnet werden soll. \n"
        <<"1) Addieren \n"
        <<"2) Subtrahieren \n"
        <<"3) Multiplizieren \n"
        <<"4) Dividieren \n"
        <<"q) Quit \n";

    cin>>input;
    switch(input)
    {
         case '1':
              cout<<"Das Ergebnis aus "<<a<<" + "<<b<<" ist "<<addition(a,b)<<endl;
         break;
         case '2':
              cout<<"Das Ergebnis aus "<<a<<" - "<<b<<" ist "<<subtraktion(a,b)<<endl;
         break;
         case '3':
              cout<<"Das Ergebnis aus "<<a<<" x "<<b<<" ist "<<multiplikation(a,b)<<endl;
         break;
         case '4':
              cout<<"Das Ergebnis aus "<<a<<" : "<<b<<" ist "<<division(a,b)<<endl;
         break;
         case 'q':
         return EXIT_SUCCESS;
    }
    system("PAUSE");
}

Aber abgesehen davon brauchst du für so kleine Dinge wohl keine Funktionen zu schreiben. Und ich würde den gleich bleibenden Teil cout << "Das Erg... nur einmal nach der Abfrage schreiben, um Codeduplizierung zu vermeiden.

char input;

wäre es nicht einfacher und logischer es so zu machen(?):

int input;

aber ich glaube ich hab was übersehen oder dis is einfach nur dumm was ich laber....

fg Wurst

fg Wurst

Wenn das "Menue" allerdings mehr als 10 Eintraege hat, konnt man mit char nich sehr weit. Bei einem Integer haette man dann den Nachteil, dass man nur Zahlen pruefen koennte (anderenfalls wuerde eine Exception aufgerufen werden). Dann waere eventuell noch std::string eine Moeglichkeit.

Gruss
Cartman

Wenn das "Menue" allerdings mehr als 10 Eintraege hat,

zum glück ist das problem nicht akut.