Auch beim Taschenrechner gibt es Probleme!

Join Date: Jul 2012

Posts: 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.

Tue 24. Jul 2012, 22:19		#2
brinkz Join Date: Apr 2012 Posts: 18	Hey, Edit: Edit nicht gesehen
brinkz is offline

KN4CK3R

Administrator

Join Date: Aug 2007

Posts: 8646

User-Rating:

199 positive

33 negative

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