ich wollte mir zum üben einen Taschenrechner schreiben, den ich im Laufe der Zeit weiter schreibe.

Nun hab ich das Problem das egal welche Grundrechenart ich auswähle, (Eingabe), immer die Addition startet, obwohl die Bedingung (Eingabe ==1) sein sollte.

Vielleicht weiß einer von euch woran das liegt?!

/*
  Name: Tim
  Copyright: nö
  Version: 1.1
  Date: 25.03.09 20:43
  Description: Taschenrechner
*/
#include <cstdlib>     //Einbinden der C-Standartbibliothek
#include <iostream>

using namespace std;

void Add(void);     //Unterprogramme global initialisieren
void Sub(void);
void Mul(void);
void Div(void);
int Eingabe;

int main()
{
          cout <<"*****TASCHENRECHNER*****" <<endl;
          cout <<"Wählen Sie eine Rechenfunktion" <<endl;

          cout <<" (1) Addition" <<endl;
          cout <<" (2) Subtraktion" <<endl;
          cout <<" (3) Multiplikation" <<endl;
          cout <<" (4) Division" <<endl;

    Eingabe = 0;
        cin >> Eingabe;

    if(Eingabe = 1);
    {
    Add();                                          //Aufruf Unterprogramm Addition
    }
    else(Eingabe = 2);
    {
    Sub();                                          //Aufruf Unterprogramm Subtraktion
    } 
    if(Eingabe = 3);
    {
    Mul();                                          //Aufruf Unterprogramm Multiplikation
    }
    else(Eingabe = 4);
    {
    Div();                                          //Aufruf Unterprogramm Division
    }

    system("PAUSE");
    return EXIT_SUCCESS;
}
void Add(void)
{
     cout <<"*****Addition*****" <<endl;
     cout <<"Bitte erste Zahl eingeben"<<endl;
     long zahl1 = 0;         //Definiere lokale Variable "zahl1" und setze auf 0
     long zahl2 = 0;         //Definiere lokale Variable "zahl2" und setze auf 0
     long Ergebnis = 0;      //Definiere lokale Variable "Ergebnis" und setze auf 0

     cin >> zahl1;
     cout <<"Bitte zweite Zahl eingeben"<<endl;
     cin >> zahl2;
     Ergebnis=zahl1+zahl2;
     cout <<"Das Ergebnis lautet" << Ergebnis <<endl;

}

void Sub(void)
{
     cout <<"*****Subtraktion*****" <<endl;
     cout <<"Bitte erste Zahl eingeben"<<endl;
     long zahl1 = 0;         //Definiere lokale Variable "zahl1" und setze auf 0
     long zahl2 = 0;         //Definiere lokale Variable "zahl2" und setze auf 0
     long Ergebnis = 0;      //Definiere lokale Variable "Ergebnis" und setze auf 0

     cin >> zahl1;
     cout <<"Bitte zweite Zahl eingeben"<<endl;
     cin >> zahl2;
     Ergebnis=zahl1-zahl2;
     cout <<"Das Ergebnis lautet" << Ergebnis <<endl;
}

void Mul(void)
{ 
     cout <<"*****Multiplikation*****" <<endl;
     cout <<"Bitte erste Zahl eingeben"<<endl;
     long zahl1 = 0;         //Definiere lokale Variable "zahl1" und setze auf 0
     long zahl2 = 0;         //Definiere lokale Variable "zahl2" und setze auf 0
     long Ergebnis = 0;      //Definiere lokale Variable "Ergebnis" und setze auf 0

     cin >> zahl1;
     cout <<"Bitte zweite Zahl eingeben"<<endl;
     cin >> zahl2;
     Ergebnis=zahl1*zahl2;
     cout <<"Das Ergebnis lautet" << Ergebnis <<endl;
}

void Div(void)
{  
     cout <<"*****Division*****" <<endl;
     cout <<"Bitte erste Zahl eingeben"<<endl;
     long zahl1 = 0;         //Definiere lokale Variable "zahl1" und setze auf 0
     long zahl2 = 0;         //Definiere lokale Variable "zahl2" und setze auf 0
     long Ergebnis = 0;      //Definiere lokale Variable "Ergebnis" und setze auf 0

     cin >> zahl1;
     cout <<"Bitte zweite Zahl eingeben"<<endl;
     cin >> zahl2;
     Ergebnis=zahl1/zahl2;
     cout <<"Das Ergebnis lautet" << Ergebnis <<endl;
}

...ich hoffe jemand weiß Rat!?

Grüße Tim

Im Zweifelsfall bitte auch folgende Hinweise beachten:
C/C++ Forum :: FAQ - Sonstiges :: Wohin mit meiner Frage?

Dieses Posting wurde automatisch erzeugt.

auf Gleichheit prüfst du mit "==". In deinen if-Abfragen (Zeilen 34,38,42 und 46) steht allerdings der Zuweisungsoperator "=". C/C++ wertet eine durchgeführte Zuweisung als wahr, womit du mit deiner ersten if-Abfrage immer in der Addition landest.

Viele Grüße, Chris

Dies war auf jeden Fall schonmal ein Fehler. Aber beim kompilieren mekert er bei
dem "else".

______________________
38 D:\C++ Buch\Übung Taschenrechner\main.cpp expected primary-expression before "else"
______________________

Änder ich alles in "if" führt er es aus aber wieder mit o.g Fehler...

Grüße Tim

Korrekt sollte das so aussehen:

if(Eingabe == 1)
{
  Add();
} 
else if (Eingabe == 2)
{
  Sub();
}
else if (Eingabe == 3)
{
  Mul();
}
else if (Eingabe == 4)
{
  Div();
}
else {
  // Fehlermeldung ausgeben
}

-Wenn du nach einem else eine weitere Bedingung hinzufügen willst musst du erst ein neues if hinzufügen
-Nach der schließenden Klammer einer if Bedingung kommt kein ;
-Guck dir mal das switch Statement an
-Wenn du die Bedingung andersrum formulierst kann der Fehler den du als erstes hattest nicht auftreten (also z.B. if (1 == Eingabe))

Eingabe = 0;
        cin >> Eingabe;

    if(1 == Eingabe)
               {
               Add();                                          //Aufruf Unterprogramm Addition
               }
    else if(2 == Eingabe)
         {
         Sub();                                          //Aufruf Unterprogramm Subtraktion
         } 
    else if(3 == Eingabe)
         {
         Mul();                                          //Aufruf Unterprogramm Multiplikation
         }
    else if(4 == Eingabe)
         {
         Div();                                          //Aufruf Unterprogramm Division
         }
    else if(4 < Eingabe)
         {
         cout <<"FEHLER: Geben Sie 1,2,3 oder 4 ein" << endl; //Fehlerausgabe /Eingabe >4
         }

funktioniert einwandfrei und danke für die schnelle Hilfe!

Grüße Tim

hab mich mit

switch ()
case

etwas vertraut gemacht. Hier mal mit dieser Variante:

Eingabe = 0;
        cin >> Eingabe;

    switch (Eingabe)
           {
           case 1: Add();
           break; 

           case 2: Sub();
           break;

           case 3: Mul();
           break;

           case 4: Div();
           break;

           default: cout << "Der von Ihnen eingegebene Wert ist keiner Rechenfunktion zugeordnet. " <<endl;
                    cout << "Bitte versuchen Sie es erneut. " <<endl;
                    break;

           }

Ist meiner Meinung nach viel eleganter, kompakter als mit if/else!
Danke für den Tip !

Grüße Tim

-Wenn du die Bedingung andersrum formulierst kann der Fehler den du als erstes hattest nicht auftreten (also z.B. if (1 == Eingabe))

würde es nicht reichen, die warnungen anzuschalten? oder zu beachten!
von dem umdrehen haklte ich gar gar nic. macht den code nur schwerlesbar.

Ist meiner Meinung nach viel eleganter, kompakter als mit if/else!
Danke für den Tip !

Ich würde jetzt noch ein wenig an der Einrückung ändern (z.B. geschweifte Klammern), aber das ist natürlich Geschmackssache.

volkard schrieb:

von dem umdrehen haklte ich gar gar nic. macht den code nur schwerlesbar.

Genau. Ich finde es auch sehr hässlich, den Vergleich umzudrehen. Denn es ist von der Überlegung her nicht ganz das Gleiche, ob ich var auf Gleichheit mit 3 oder 3 auf Gleichheit mit var prüfe (so sehe ich das zumindest). Zudem geht das auch nur, wenn eine Seite nicht zuweisbar ist, z.B. weil const oder skalarer RValue.