Auch beim Taschenrechner gibt es Probleme!

Anmeldungsdatum: Jul 2012

Beiträge: 11

Morgen/Abend - egal.

Ich habe mich mal rangesetzt und einen Taschenrechner gecodet (nichts großes, aber jeder fängt klein an!)

Erst mal poste ich mein Code:

Spoiler

TEXT Code:

// Taschenrechner coded by flashst3p 
 
#include <iostream>
#include <windows.h>
#include <cmath>
#include <conio.h>
using namespace std;
 
// Variablen
 
int zahl1;
int zahl2;
string rechenzeichen;
string neustart;
 
// Programm einstieg
int main()
{
    cout <<"Taschenrechner coded by flashst3p"<< endl << "#################################\n\n" << endl;
 
	cout << "Bitte geben Sie die erste Zahl ein.\n" << endl;
	cin >> zahl1;
 
	cout << "\nBitte geben Sie die zweite Zahl ein.\n" << endl;
	cin >> zahl2;
 
    cout << "\nBitte geben Sie das Rechenoperation ein. (+ - / *)\n" << endl;
	cin >> rechenzeichen;
 
// Additionsfunktion
	if(rechenzeichen == "+")
	{
		cout << "\nDas Ergebnis betraegt: " << zahl1 + zahl2 << endl;
	}
 
 // Subtraktionsfunktion
    if(rechenzeichen == "-")
    {
        cout << "\nDas Ergebnis betraegt: " << zahl1 - zahl2 << endl;
    } 
 
// Divisionsfunktion
   if(rechenzeichen == "/")
   {
        cout << "\nDas Ergebnis betraegt: " << zahl1 / zahl2 << endl;
   }
 
// Multiplikation
   if(rechenzeichen == "*")
   {
        cout << "\nDas Ergebnis betraegt: " << zahl1 * zahl2 << endl;
   } 
 
   cout << "\nProgramm neusrarten? (Ja / Nein)\n" << endl;
   cin >> neustart;
 
   if(neustart == "Ja")
   {
            system("start taschenrechner2.exe");
            exit(1);
   } 
   else
   {
       return 0;
   }   
 
 
    getch();
	return 0;
}

So weit so gut. Doch wollte ich noch die Funktion einbauen, wenn jemand bei der Rechenoption etwas anderes eingibt, als "+ - / *" , dass kommt: Eingabe ungültig und es erneut nach der Rechenoption gefragt wird, sprich man wieder zum Ursprung zurückkehrt, eben zu

TEXT Code:

                          cout << "\nBitte geben Sie das Rechenoperation ein. (+ - / *)\n" << endl;
	cin >> rechenzeichen;

Ich dachte mir am Anfang ich mache etwas Allgemeines, sprich:

TEXT Code:

if(rechenoption == "")

Und in die "" eben quasi ungleich, der normalen Zeichen + - / *. Hab leider vergessen, wie man das ungleicht herstellt. Und dann eben das er zu der obrigen Frage zurück springt.

Hoffe ihr habt irgendwie verstanden was ich versuchen wollte, zu erklären.

Update: na klar != ist ungleich, boah wie schlecht. Ok. Aber wie mache ich es das er dann wieder zurückspringt und das Programm wieder ab der Stelle abliest?

Update2: Meine Güte, erst weiter denken, dann Frage stellen. So ich habe mal meine "Überprüfungszeile" nach oben geschoben, über die anderen if() teile. Jetzt muss ich eine Schleife erstellen, sprich solange rechenzeichen != */-+ , ausgeben "Ihre Eingabe ist ungültig, wiederholen sie" und dann wenn es eins von den ist, soll er weiter machen. Sieht so aus:

TEXT Code:

// Anderes Zeichen
   while(rechenzeichen != "*","/","-","+")
   {
        cout << "\nIhre Eingabe ist inkorrekt, bitte versuchen sie es erneut" << endl;
        cin >> rechenzeichen;
   }

So aber jetzt ist egal ob ich "+" eingebe oder nicht, wiederholt er es immer und immer wieder.

Okay, Leute im IRC haben schon geholfen, insofern irgendwer hier noch die "Lösung" haben will:

TEXT Code:

while(rechenzeichen != "*" && rechenzeichen != "/" && rechenzeichen != "-" && rechenzeichen != "+")
   {
        cout << "\nIhre Eingabe ist inkorrekt, bitte versuchen sie es erneut\n" << endl;
        cin >> rechenzeichen;
   }

Danke, bye.

__________________

# Code 2 have FUN.

Di 24. Jul 2012, 22:19		#2
brinkz Anmeldungsdatum: Apr 2012 Beiträge: 18	Hey, Edit: Edit nicht gesehen
brinkz ist offline

KN4CK3R

Administrator

Anmeldungsdatum: Aug 2007

Beiträge: 8643

Benutzer-Bewertung:

199 positiv

33 negativ

und richtig wäre das Programm so:

TEXT Code:

// Taschenrechner coded by flashst3p 
 
#include <iostream>
#include <string>
#include <limits>
 
// Programm einstieg
void clearCin()
{
	using std::cin;
	using std::numeric_limits;
	using std::streamsize;
 
	cin.clear();
	cin.ignore(numeric_limits<streamsize>::max(), '\n');
}
 
int main()
{
	using std::cout;
	using std::cin;
	using std::string;
 
	cout << "Taschenrechner coded by flashst3p\n#################################\n\n";
 
	string neustart;
	do
	{
		double zahl1;
		for (cout << "Bitte geben Sie die erste Zahl ein:\n"; !(cin >> zahl1); clearCin(),
			 cout << "Bitte geben Sie die erste Zahl ein:\n");
 
		double zahl2;
		for (cout << "Bitte geben Sie die zweite Zahl ein:\n"; !(cin >> zahl2); clearCin(),
			 cout << "Bitte geben Sie die zweite Zahl ein:\n");
 
		char rechenzeichen;
		for (cout << "Bitte geben Sie die Rechenoperation an: (+ - / *)\n"; !(cin >> rechenzeichen && string("+-/*").find(rechenzeichen) != string::npos); clearCin(),
			 cout << "Bitte geben Sie die Rechenoperation an: (+ - / *)\n");
 
		double result;
		switch (rechenzeichen)
		{
			case '+':
				result = zahl1 + zahl2;
				break;
			case '-':
				result = zahl1 - zahl2;
				break;
			case '/':
				result = zahl1 / zahl2;
				break;
			case '*':
				result = zahl1 * zahl2;
				break;
		}
		cout << "\nDas Ergebnis betraegt: " << result << "\n\n"
			 << "\nProgramm neustarten? (Ja / Nein)\n";
	} while (cin >> neustart && neustart == "Ja");
 
	return 0;
}

greetz KN4CK3R

__________________

Hallo