[C++] Syntax Error?

[FrameVB]

Hey Leute!

Hab vor kurzem mit dem Lernen von C++ angefangen...
Da hab ich mir gedacht: Warum mach ich nicht so ein dummes Anfängerprogramm^^

Also hab ich micht entschieden einen kleinen Taschenrechner in Form einer Konsolenanwendung zu programmieren.(Hab aber erst das Addieren gemacht)

Jedoch bin ich auf ein Hinderniss gestoßen, das ich nicht verstehe.

#include <iostream>
#include <string>
using namespace std;


int addieren()
{
	int additor1, additor2, summe;

	cout << "Geben Sie bitte die erste Zahl ein: ";
	cin >> additor1;
	cout << "Geben Sie bitte die zweite Zahl ein: ";
	cin >> additor2;

	summe = additor1 + additor2;

	return summe;
}

void main ()
{
	char rechnerart, beenden;
	int ergebnis;

	cout << "Wählen Sie bitte eine der 4 Rechenarten aus:" << endl;
	cout << "a... Addieren" << endl;
	cout << "s... Subtrahieren" << endl;
	cout << "m... Multiplizieren" << endl;
	cout << "d... Dividieren" << endl;
	cout << endl;

	beenden = '1';
	do while(beenden != 'Y')
	{
		cin >> rechnerart;

		switch(rechnerart)
		{
		case 'a':
			ergebnis = addieren();
			break;
		}

		cout << "Ergebnis = " << ergebnis << endl;
		cout << "Programm schließen? (Y/N)";
		cin >> beenden;
	}
}

Das Problem das ich habe ist, dass mir der Compiler bei der letzten geschwungenen Klammer einen Syntaxerror ausspuckt, obwohl ich nicht der Meinung bin einen gemacht zu haben^^(Aber der Compiler ist in dem Punkt bestimmt klüger als ich)

Der Fehler lautet:
error C2059: Syntaxfehler: '}'

Ich wollte fragen, ob sich jemand das kleine Programm anschaun kann und mir evtl. sagen, was ich falsch gemacht habe...(Bitte nur den gefragten Fehler ausbessern^^)


Mit besten Grüßen
Frame

[lukeelten]

Es liegt an deiner Schleife.

do {


} while (Bedingung);

Das ist die richtige Syntax für do while Schleifen.

Zudem bei einem Switch immer eine default Option geben, sonst wird das auch zur Schleife.

switch () {

...

default:
break;
}

[Larius]

Bei einer Do-While Schleife gehört das While nach der geschlossenen geschwungenen Klammer des Anweisungsblockes hin. Also in deinem Fall:

do 
	{
		cin >> rechnerart;

		switch(rechnerart)
		{
		case 'a':
			ergebnis = addieren();
			break;
		}

		cout << "Ergebnis = " << ergebnis << endl;
		cout << "Programm schließen? (Y/N)";
		cin >> beenden;
	}while(beenden != 'Y'); /*Achtung, hier brauchst du einen Strichpunkt!*/

[FrameVB]

^^

Das ich da nicht selber draufgekommen bin

Danke für eure schnellen Antworten! Hab schon gedacht ich bin ganz depatt geworden^^

[sumisumi]

int additor1, additor2, summe; 

meinst du summand1 und summand2?

[issen9]

Zudem bei einem Switch immer eine default Option geben, sonst wird das auch zur Schleife.

Wie soll man das verstehen?

int main()
{
    int a = 1;
    switch(a)
    {
    case 2: std::cout << "2" << std::endl;
            break;
    }

    std::cout << "end" << std::endl;
    std::cin.get();
    return 0;
} 

gibt einfach "end" aus und exited.

[Larius]

Also das eine Switch ohne Default zu einer Schleife wird hör ich heut zum ersten Mal - und wie Issen9 es am Code gezeigt hat ist es ein Mythos. Das Default ist ja nur dazu da, um alle nicht ausprogrammierten Fälle der Switch-Anweisung "einzufangen" und zu behandeln.

[lukeelten]

Ich hatte mal gelesen, dass wenn switch kein passendes Element findet, es nicht aufhört zu suchen. Getestet hab ich es nie, trotzdem schadet meiner Meinung nach ein default nicht.

[Scrion]

Ich hatte mal gelesen, dass wenn switch kein passendes Element findet, es nicht aufhört zu suchen

Das ist falsch.