Jetzt wo meine Prüfungen vorbei sind hab ich mal richtig angefangen mit C++, bzw. seit 3 Tagen mit Hilfe von nem Buch *g* Jetzt hab ich mal nen kleinen Taschenrechner geschrieben, allerdings möchte ich gerne wissen was ich hätte besser machen können. Fehler kenn ich bis jetzt nur einen, und zwar das der Taschenrechner nicht funktioniert wenn man im Auswahlmenü ne zahl <=0 eingibt.

#include <iostream>
using namespace std;

int main()
{
	int eingabe;
	double eingabe1;
	double eingabe2;

	cout << endl;
	cout << "Maddy's Rechtentestprogramm Version 1.0" << endl;	
	cout << "" << endl;
	cout << "[1] Addieren" << endl;
	cout << "[2] Subtrahieren" << endl;
	cout << "[3] Multiplizieren" << endl;
	cout << "[4] Dividieren" << endl;
	cout << endl;
	cout << "Wähle ein Programm: ";
	cin >> eingabe;

	switch(eingabe)
	{
	case 1: cout << endl;
		cout << "Addieren" << endl;
		break;
	case 2: cout << endl;
		cout << "Subtrahieren" << endl;
		break;
	case 3: cout << endl;
		cout << "Multiplizieren" << endl;
		break;
	case 4: cout << endl;
		cout << "Dividieren" << endl;
		break;
	default:cout << endl;
		cout << "Nicht möglich, bitte ein anderes Programm wählen." << endl;
		break;
	}

	while (eingabe >= 5)
	{
	cout << "" << endl;
	cout << "[1] Addieren" << endl;
	cout << "[2] Subtrahieren" << endl;
	cout << "[3] Multiplizieren" << endl;
	cout << "[4] Dividieren" << endl;
	cout << endl;
	cout << "Wähle ein Programm: ";
	cin >> eingabe;
	}

	cout << "" << endl;
	cout << "Gib die erste Zahl ein: ";
	cin >> eingabe1;

	cout << "Gib die zweite Zahl ein: ";
	cin >> eingabe2;

	if (eingabe == 1)
	{
	cout << "" << endl;
	cout << "Das Ergebnis lautet: " << eingabe1 + eingabe2 << endl;
	cout << "" << endl;
	}

	if (eingabe == 2)
	{
	cout << "" << endl;
	cout << "Das Ergebnis lautet: " << eingabe1 - eingabe2 << endl;
	cout << "" << endl;
	}

	if (eingabe == 3)
	{
	cout << "" << endl;
	cout << "Das Ergebnis lautet: " << eingabe1 * eingabe2 << endl;
	cout << "" << endl;
	}

	if (eingabe == 4)
	{
	cout << "" << endl;
	cout << "Das Ergebnis lautet: " << eingabe1 / eingabe2 << endl;
	cout << "" << endl;
	}

	return 0;
}

while (eingabe >= 5) <- das hab ich versucht zu ändern, eben das es auch für <=0 funktioniert, hab ich aber nicht hinbekommen

Gruß

-> Eventuell das Programm erweitern

Zu der Begrenzung der Eingabe: mehrere Bedingungen kannst du mit && (UND) bzw. || (ODER) zusammenfassen - in deinem Fall brauchst du die Schleifenbedingung while(eingabe<=0 || eingabe>=5);

while (eingabe >=5 && eingabe <=0)

Aber was ich empfehlen würde ist, gleich objektorientiert zu lernen! Sonst festigt sich im Kopf gleich die strukturierte Programmierung, die man im Grunde nicht mehr braucht!

if (eingabe == 1) 
    { 
    cout << "" << endl; 
    cout << "Das Ergebnis lautet: " << eingabe1 + eingabe2 << endl; 
    cout << "" << endl; 
    } 

    if (eingabe == 2) 
    { 
    cout << "" << endl; 
    cout << "Das Ergebnis lautet: " << eingabe1 - eingabe2 << endl; 
    cout << "" << endl; 
    } 

    if (eingabe == 3) 
    { 
    cout << "" << endl; 
    cout << "Das Ergebnis lautet: " << eingabe1 * eingabe2 << endl; 
    cout << "" << endl; 
    } 

    if (eingabe == 4) 
    { 
    cout << "" << endl; 
    cout << "Das Ergebnis lautet: " << eingabe1 / eingabe2 << endl; 
    cout << "" << endl; 
    } 
}

DAraus würd ich auf jeden Fall sowas machen:

double result;
switch (eingabe)
{
case 1:
    result = a + b;
    break;
case 2:
    ...
default:
   cout << "fehler";
   result = 0.0;
   break;
}
cout << "Das Ergebnis lautet: " << result;

Die Bedingung im while sollte doch heißen:

while (eingabe >= 0 && eingabe <= 4)

die while Bedingung müsste dann wie folgt lauten:
while (eingabe >= 5 || eingabe <= 0)
Du könntest zum Beispiel doppelte Codestellen wie das "Menü":

cout << "" << endl;
    cout << "[1] Addieren" << endl;
    cout << "[2] Subtrahieren" << endl;
    cout << "[3] Multiplizieren" << endl;
    cout << "[4] Dividieren" << endl;
    cout << endl;
    cout << "Wähle ein Programm: ";

in eine eigene Funktion packen, damit du es nicht im Code duplizieren musst, das ist sehr fehleranfällig!
Weiterhin könntest du das noch ein wenig vereinfachen mit arrays:

string wahl[4] = {"Addieren", "Subtrahieren", "Multiplizieren", "Dividieren"};

for(int i = 0; i < 4; ++i)
   cout << "[" << (i+1) << "] " << wahl[i] << endl;

Dafür musst du allerdings noch #include <string> hinzufügen.

KaPtainCugel

Gruß

geht es nicht mit einer UND-Verknüpfung?

while (eingabe >=5 && eingabe <=0)

Nein, die while-Schleife wird so lange durchlaufen, wie diese Bedingung erfüllt ist, also gar nicht (nenn' mir eine natürliche Zahl, die gleichzeitig größer als 5 UND kleiner als 0 ist ;)).

Aber was ich empfehlen würde ist, gleich objektorientiert zu lernen! Sonst festigt sich im Kopf gleich die strukturierte Programmierung, die man im Grunde nicht mehr braucht!

Objektorientiert? Worin unterscheidet die sich denn von der die ich lerne?

Mein Taschenrechner is eben nur mit dem gewesen was ich jetzt in 3 Tagen gelernt und verstanden hab *g*

plizer schrieb:

geht es nicht mit einer UND-Verknüpfung?

while (eingabe >=5 && eingabe <=0)

Nein, die while-Schleife wird so lange durchlaufen, wie diese Bedingung erfüllt ist, also gar nicht (nenn' mir eine natürliche Zahl, die gleichzeitig größer als 5 UND kleiner als 0 ist ;)).

Ups, da hab ich was verdreht ^^.

Das mit dem Bankautomaten kommt in meinem Buch auch noch, allerdings erst ganz am Schluss Dann muss ich mir demnächst nach nem neuen gucken *g*

Gruß