You are not logged in.

  • Login

Mein erstes Programm (Taschenrechner)...

necron9

Beginner

  • "necron9" started this thread

Posts: 1

  • Send private message

1

Friday, September 12th 2008, 12:49am

Mein erstes Programm (Taschenrechner)...

Huhu
Bin neu hier und hoffe das ich hier richtig bin...
Ich bin seit 3 Tage dabei, C++ zu lernen.... _naja_ ... Und ich habe auch schon was "programmiert", nämlich einen Taschenrechner... das übliche Anfängerprogramm...
Ich würde gerne eure Meinung wissen, da ich glaube das ich warscheinlich total "unsauber" gearbeitet habe und mir das viel zu kompliziert gemacht habe... bzw. Anfängerfehler gemacht habe, die einfach nicht sein müssen :)
Das Teil hat mich 3-4 Stunden gekostet, hat aber sehr viel spaß gemacht und ich bin motiviert weiter zu lernen :)
Ich benutze größtenteils das Tutorial hier : http://www.cplusplus.com/doc/tutorial/
Das ist leider auf Englisch, soll aber wohl sehr gut sein. Ich verstehe auch alles mehr oder weniger... (wollte mir später aber noch ein Buch kaufen... um genauer darauf einzugehen)...

Das Programm funktioniert bestens... nur zwei Fehler hat es noch und ich weiß nicht wie ich diese behebe...
1.) z.B Zeile 59... Bei der Abfrage ob eine neue Rechnung erfolgen soll, muss der Benutzer zwingend klein schreiben... leider weiß ich nicht wie man das ändert...
2.) ... wenn der Benutzer z.B. x oder y eingeben soll (sprich eine Zahl) und man anstatt einer Zahl einen Buchstaben eingibt... dann kommt ein Ergebnis, das aber die ganze Zeit erscheint... also das man nicht mehr lesen kann was angezeigt wird da sich das unendlich lang wiederholt ... mhmpf...

Also nochmal im Klartext : Was haltet ihr von dem Programm...? Irgendwelche Ratschläge, Tipps oder so? Wäre euch sehr dankbar...

Gruß Necron9


C/C++ Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126

#include <string>
#include <iostream>
 
using namespace std;
 
int main ()
{
 
 double ergebnis;
 double x;
 double y;
 double zeichen;
 string antwortja = "ja";
 
 
 
 Start:
 cout << "\n\n";
 cout << "****************************\n";
 cout << "****** Taschenrechner ******\n";
 cout << "****************************\n\n\n";
 
 
 //Eingabe von Rechenart
 cout << "Druecken Sie [ ] und Enter...\n";
 cout << "(1) fuer Addition\n";
 cout << "(2) fuer Subtraktion\n";
 cout << "(3) fuer Multiplikation\n";
 cout << "(4) fuer Division\n\n";
 rechenart:
 cout << "\n\n";
 cout << "Geben Sie ihre Rechenart an: ";
 cin >> zeichen;
 cout << "\n";
 
 
 //Eingabe der Rechenart
 if (zeichen > 4)
{
cout << "Sie muessen 1,2,3 oder 4 eingeben!";
goto rechenart;
}
 
 //Rechenart 1
 if (zeichen == 1)
{
cout << "Geben sie x an: ";
cin >> x;
cout << "Geben sie y an: ";
cin >> y;
ergebnis=x+y;
cout << "------------------\n";
cout << "Das Ergebnis lautet: " << ergebnis;
cout << "\n\n\n\n\n\n";
 
 
  cout << "Neue Rechnung? ";
  cin >> antwortja;
  if (antwortja == "ja")
{
goto Start;
}
 
 
}
 //Rechenart 2  
 if (zeichen == 2)
{ 
cout << "Geben sie x an: ";
cin >> x;
cout << "Geben sie y an: ";
cin >> y;
ergebnis=x-y;
cout << "------------------\n";
cout << "Das Ergebnis lautet: " << ergebnis;
cout << "\n\n\n\n\n\n";
 
  cout << "Neue Rechnung? ";
  cin >> antwortja;
  if (antwortja == "ja")
{
goto Start;
}
}
 
 //Rechenart 3   
 if (zeichen == 3)
{
cout << "Geben sie x an: ";
cin >> x;
cout << "Geben sie y an: ";
cin >> y;
ergebnis=x*y;
cout << "------------------\n";
cout << "Das Ergebnis lautet: " << ergebnis;
cout << "\n\n\n\n\n\n";
 
  cout << "Neue Rechnung? ";
  cin >> antwortja;
  if (antwortja == "ja")
{
goto Start;
}
}
 
//Rechenart 4   
 if (zeichen == 4)
{
cout << "Geben sie x an: ";
cin >> x;
cout << "Geben sie y an: ";
cin >> y;
ergebnis=x/y;
cout << "------------------\n";
cout << "Das Ergebnis lautet: " << ergebnis;
cout << "\n\n\n\n\n\n";
 
  cout << "Neue Rechnung? ";
  cin >> antwortja;
  if (antwortja == "ja")
{
goto Start;
}
 
}
}

truespin

Professional

Posts: 378

  • Send private message

2

Friday, September 12th 2008, 6:38am

guten morgen,

ich bin kein c++ experte, wollte nur mal hallo und viel spass/erfolg im forum wünschen!!!


thx, truespin

JFoX

Professional

Posts: 535

  • Send private message

3

Friday, September 12th 2008, 7:55am

Hallo und Willkommen im Forum,

Dein Quellcode ist an sich ganz in Ordnung. Nur eine Fatale Sache ist da drinn, die solltest du dir schleunigst abgewöhnen.

C/C++ Quellcode

 
1

goto Start;


Sowas macht man nicht, die goto zeiten sind vorbei :)

Du sagtest ja, das du noch Anfänger bist, also mache dir mal keine Sorgen um die Optik, das kann mit etwas übung von ganz alleine. Du wirst deinen eigenen Style finden müssen. Eigne dir erst mal mehrere Techniken an, dann wirst du automatisch dahinter kommen wie man sachen anders lösen kann.

Hafner

Professional

Posts: 630

  • Send private message

4

Friday, September 12th 2008, 11:54pm

Um die gotos zu vermeiden (was du wirklich tun solltest) solltest du dich mit den Möglichkeiten von Funktionen vertraut machen. Das ermöglicht dir dein Programm übersichtlicher zu strukturieren.

bcc-fan

Professional

Posts: 384

  • Send private message

5

Saturday, September 13th 2008, 2:29am

Ein weiter Vorschlag ist Redundanzen vermeiden - sieht dann etwa so aus:

C/C++ Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80

#include <string>
#include <iostream>
 
using namespace std;
 
int main ()
{
 
	double ergebnis;
	double x;
	double y;
	double zeichen;
	string antwortja = "ja";
 
 
 
	Start:
	cout << "\n\n";
	cout << "****************************\n";
	cout << "****** Taschenrechner ******\n";
	cout << "****************************\n\n\n";
 
 
	//Eingabe von Rechenart
	cout << "Druecken Sie [ ] und Enter...\n";
	cout << "(1) fuer Addition\n";
	cout << "(2) fuer Subtraktion\n";
	cout << "(3) fuer Multiplikation\n";
	cout << "(4) fuer Division\n\n";
	rechenart:
	cout << "\n\n";
	cout << "Geben Sie ihre Rechenart an: ";
	cin >> zeichen;
	cout << "\n";
 
 
//Eingabe der Rechenart
	if (zeichen > 4)
	{
		cout << "Sie muessen 1,2,3 oder 4 eingeben!";
		goto rechenart;
	}
 
	cout << "Geben sie x an: ";
	cin >> x;
	cout << "Geben sie y an: ";
	cin >> y;
//Rechenart 1
	if (zeichen == 1)
	{
		ergebnis=x+y;
	}
 //Rechenart 2  
	if (zeichen == 2)
	{ 
		ergebnis=x-y;
	}
 //Rechenart 3   
	if (zeichen == 3)
	{
		ergebnis=x*y;
 	}
//Rechenart 4   
	if (zeichen == 4)
	{
		ergebnis=x/y;
	}
	cout << "------------------\n";
	cout << "Das Ergebnis lautet: " << ergebnis;
	cout << "\n\n\n\n\n\n";
 
 
	cout << "Neue Rechnung? ";
	cin >> antwortja;
	if (antwortja == "ja")
	{
		goto Start;
	}
 
}


Gross- und Kleinschreibung:
http://support.microsoft.com/kb/815783/de

Division durch 0 ?

goto steht schon in deinem Tutorial drin, das es kritisch werden kann. ;)

Mal nach "switch" suchen.

MfG bcc-fan

xX-rising-Xx

Beginner

Posts: 1

  • Send private message

6

Tuesday, September 16th 2008, 2:36pm

Source code

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84

#include <string>
#include <iostream>
 
using namespace std;
 
int main ()
{
 
	double ergebnis;
	double x;
	double y;
	double zeichen = 1;
	string antwortja = "j";
 

	cout << "\n\n";
	cout << "****************************\n";
	cout << "****** Taschenrechner ******\n";
	cout << "****************************\n\n\n";
 
 
	//Eingabe von Rechenart
	cout << "Druecken Sie [ ] und Enter...\n";
	cout << "(1) fuer Addition\n";
	cout << "(2) fuer Subtraktion\n";
	cout << "(3) fuer Multiplikation\n";
	cout << "(4) fuer Division\n\n";
	cout << "\n\n";
	cout << "Geben Sie ihre Rechenart an: ";
	cin >> zeichen;
	cout << "\n";
 
 
//Eingabe der Rechenart
	if (zeichen > 4)
	{
		cout << "Sie muessen 1,2,3 oder 4 eingeben!";
		return main();
	}
 
	cout << "Geben sie x an: ";
	cin >> x;
	cout << "Geben sie y an: ";
	cin >> y;
//Rechenart 1
	if (zeichen == 1)
	{
		ergebnis=x+y;
	}
 //Rechenart 2  
	if (zeichen == 2)
	{ 
		ergebnis=x-y;
	}
 //Rechenart 3   
	if (zeichen == 3)
	{
		ergebnis=x*y;
 	}
//Rechenart 4   
	if (zeichen == 4)
	{
		ergebnis=x/y;
	}
	cout << "------------------\n";
	cout << "Das Ergebnis lautet: " << ergebnis;
	cout << "\n\n\n\n\n\n";
 
/* 
	cout << "Neue Rechnung? ";
	cin >> antwortja;
	if (antwortja == "ja") // hier haben wir das problem, dass "JA" und "jop" nicht gehen!
	{
		goto Start; // goto ist keine gute Loesung.
	}
*/

    cout << "Neue Rechnung? j/n:\n";
	cin >> antwortja;
	if (antwortja == "j") 
	{
		return main();
	}
}


Man kann das auch noch verbessern, aber das würde den Code zu stark verändern.
Aber fürs erst Programm nicht schlecht.
Auch wenn man jetzt durch return main() zum Anfang kommt ist es immer noch besser als goto. Wenn du lernst neue Funktionen zu machen , dann kann man mit return alles "schöner" machen.
mfg

bcc-fan

Professional

Posts: 384

  • Send private message

7

Tuesday, September 16th 2008, 6:36pm

Also hier eine einfache Version ohne goto oder ähnliche Verrenkungen:

Todo hießt da sollte noch mal nachgedacht werden ;)

C/C++ Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60

// Taschenrechner simpel
// ToDo:
// Operanten -> int oder besser ?
// Bei der Division Sicherheitsabfrage einfügen!
// Wenn der Rechner mehr leisten soll, dann ergaenzt ihn
 
#include <iostream> 
using namespace std; 
 
int main () 
{ 
	int zeichen;
	int O1, O2;
 
	bool bigloop;
	do
	{
		cout << "Bitte geben Sie den 1. Operanten ein: ";
		cin >> O1;
 
		bool loop = false;
		do
		{
			cout << "Geben Sie die Rechenart ein ( (1) -> +  (2) -> -  (3) -> * (4) -> / )" << endl; 
			cout << ", nur Zahlen  von 1 bis 4 werden aktzeptiert. "; 
			cin >> zeichen; 
			if ((zeichen >= 1) && (zeichen <=4)) 
			{ 
				loop = true;
			}
			else
			{
				cout << endl << " Rechenart nicht implementiert" << endl;
			}                
		}while(!loop); 
 
		cout << "Bitte geben Sie den 2. Operanten ein: ";
		cin >> O2;
 
		cout << endl;
		switch(zeichen) 
		{ 
			case 1: cout << O1 << " + " << O2 << " = " << O1 + O2 << endl; break; 
			case 2: cout << O1 << " - " << O2 << " = " << O1 - O2 << endl; break; 
			case 3: cout << O1 << " * " << O2 << " = " << O1 * O2 << endl; break; 
			case 4: cout << O1 << " / " << O2 << " = " << O1 / O2 << endl; break; 
 
			default: cout << "\n\tSomething go's wrong"; 
			break; 
		} 
		bigloop = true;
		cout << endl << " Zeichen eingeben - 9 für neue Aufgabe" << endl;	// statt der 9 etwa in J/N oder Y/N aendern
		cin >> zeichen;
		if (zeichen == 9)
		{
			bigloop = false;
		}
 
	}while(!bigloop);
}


MfG bcc-fan

Kruemmel

Beginner

Posts: 5

  • Send private message

8

Monday, February 23rd 2009, 7:32pm

Ich finde dieses Ergebnis am leichtesten :D





C/C++ Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61

#include <cstdlib>
#include <iostream>
#include <string>
 
 
using namespace std;
 
int eingabe;
float zahl1;
float zahl2;
float ergebnis;
string eingabe1;
 
 
int main()
{
      cout << "  " << endl;
      cout << "  " << endl;
    cout << " TASCHENRECHNER " << endl;
    cout << "  " << endl;
      cout << "  " << endl;
 
    cout << "Möchten sie eine Addition(1) Multiplikation(2) Division(3)\n oder Subraktion(4) durchführen?"<< endl;
    cin >> eingabe;
    cout << "Erste Zahl :" << endl;
    cin >> zahl1;
    cout << "Zweite Zahl : " << endl;
    cin >> zahl2;
 
    switch (eingabe)
    {
    case 1:
         cout << "Ergebnis : " << zahl1+zahl2 << endl;
    break;
    case 2:
         cout << "Ergebnis : " << zahl1*zahl2 << endl;
     break;   
    case 3 :
         cout << "Ergebnis : " << zahl1/zahl2 << endl;
     break;    
    case 4 :
         cout << "Ergebnis : " << zahl1 - zahl2 << endl;          
     break;
 
 
}
 
  cout << "Möchten sie nochmal" << endl;
  cin >> eingabe1;  
 
 
  if (eingabe1=="ja" || eingabe1=="Ja")
  {
       return main();              
                     }
 
 
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

bcc-fan

Professional

Posts: 384

  • Send private message

9

Tuesday, February 24th 2009, 9:44pm

Folgendes sollte man in C oder C++ möglichst nicht nutzen:

C/C++ Quellcode

 
1

return main();


und

C/C++ Quellcode

 
1

system("PAUSE");

funktioniert nicht auf jedem OS ( = Betriebssystem )

Bei dem Taschenrechnernbeispiel ist dies nicht und auch
kein goto notwendig.

MfG bcc-fan

zyklo

Intermediate

Posts: 63

  • Send private message

10

Thursday, February 26th 2009, 11:13pm

Geil, ich wüsst garnicht, wie man das Goto benutzt - das ist doch echt das no-go des Jahrhunderts in Sachen Programmiersprachen :-D

Similar threads

Tagging

Social bookmarks