[C++] Einfacher Taschenrechner

[DMW007]

Guten Tag,

IT-Hausaufgabe für meine Klasse & Übungsstuff für den Rest:

Sie sollen einen einfachen Taschenrechner programmieren. Dieser soll zwei Operanden sowie die folgenden Operatoren verarbeiten können:

Addition, Operator: +
Subtraktion, Operator: -
Multiplikation, Operator: *
Division mit Dezimalstellen (falls vorhanden): / (Bedingung: 2. Operand darf nicht 0 sein)
Ganzzahlige Division mit Rest: : (Bedingung: 2. Operand darf nicht 0 sein, jeder der Operanden muss ganzzahlig sein)

Ungültige Operanden oder Operatoren sollen über eine Fehlermeldung abgewiesen werden.

Anmerkung:
Coding-Styl entspricht den Anforderungen des Lehrers (Öffnende Geschweifte Klammern in neue Zeile setzen etc, alles was schwul ist halt).
Compiliert mit Borland C++ Builder 5.

Source:

#include <iostream>
#include <conio>

using namespace std;

void main()
{
     float Operand1;
     cin >> Operand1;

     char Operator;
     cin >> Operator;

     float Operand2;
     cin >> Operand2;

     switch(Operator)
     {
                      case '+':
                           cout << Operand1 + Operand2;
                           break;

                      case '-':
                           cout << Operand1 - Operand2;
                           break;

                      case '*':
                           cout << Operand1 * Operand2;
                           break;

                      case '/':
                           if(Operand2 == 0)
                           {
                                       cout << "Division durch 0 nicht m" << char(148) << "glich!";
                           }
                           else
                           {
                                 cout << Operand1 / Operand2;
                           }
                           break;

                      case ':':
                           if(Operand2 == 0)
                           {
                                       cout << "Division durch 0 nicht m" << char(148) << "glich!";
                           }
                           else if((int)Operand1 != Operand1 || (int)Operand2 != Operand2)
                           {
                                 cout << "Nur ganzzahlige Operanden m" << char(148) << "glich!";
                           }
                           else
                           {
                                 cout << (int)(Operand1 / Operand2) << " Rest " << (int)Operand1 % (int)Operand2;
                           }
                           break;

                      default:
                              cout << "Ung" << char(129) << "ltiger Operator!";
                              break;
     }

     getch();
} 

[Ta1lor]

Du kannst die Eingabe auch so machen, finde ich persönlich schöner.

cin >> Operand1 >> Operator >> Operand2

Und warum machst du nur bei ":" das ganze mit dem Rest?

Ich hätte das so gemacht:

float Operand1, Operand2;
     char Operator;
     cin >> Operand1 >> Operator >> Operand2;
     switch(Operator)
     {
        case '+': cout << Operand1 + Operand2; break;
        case '-': cout << Operand1 - Operand2; break;
        case '*': cout << Operand1 * Operand2; break;
        case '/': 
            if (Operand2 == 0)
            {
                cout << "Geht nicht" << endl;
            } 
            else if (int (Operand1) != Operand1 || int (Operand2) != Operand2)
                { 
                    cout << "Nur ganzzahlige Operanten m" << char(148) << "glich!" << endl; 
                } 
            else if (int (Operand1) % int (Operand2) != 0) 
                    { 
                    cout << int ((Operand1) / Operand2) << " Rest " << int (Operand1) % int (Operand2) << endl; 
                    } 
             
                    
            else cout << Operand1 / Operand2 << endl;
             
                    break;
        default:
                cout << "Ung" << char(129) << "ltiger Operator!";
                _getch();
                return;
                break;
     }

     _getch(); 

/e Ahja, jetzt hab ich das mit dem ":" auch gelesen..

[DMW007]

Du kannst die Eingabe auch so machen, finde ich persönlich schöner.

cin >> Operand1 >> Operator >> Operand2

Anmerkung:
Coding-Styl entspricht den Anforderungen des Lehrers (Öffnende Geschweifte Klammern in neue Zeile setzen etc, alles was schwul ist halt).

Wirtschaftsinformatiker halt, die sind sehr penibel.
Der will das nach dem Schema

Datentyp Variablename;
cin >> Variablename; 

Dasselbe in switch-Blöcken.

[...]
case '+': cout << Operand1 + Operand2; break; 

Wenns nur um eine so kurze Codezeile geht würde ich das auch in eine Zeile knallen, gibt bei ihm aber Punktabzug da seine Vorgabe für switch eben so lautet:

switch(Variable)
{
    case "bla":
        //TuWas
        break;
} 

Wenn ich das für mich selbst schreibe, würde ich das auch etwas anders lösen, daher auch der Hinweis.

Und warum machst du nur bei ":" das ganze mit dem Rest?

Aufgabenstellung.
/ gibt normale Kommazahlen aus, : nur ganze Zahlen mit Rest.