Czy pomysł jest dobry?

Question

Czy pomysł połączenia while ... do z switch ... case jest dobry? 

#include <iostream>
using namespace std;
int main()
{
	cout<<"Menu:"<<endl;
	cout<<"[1] Dodawanie"<<endl;
	cout<<"[2] Odejmowanie"<<endl;
	cout<<"[3] Mnożenie"<<endl;
	cout<<"[4] Dzielenie"<<endl;
	cout<<"[5] Rezygnuj"<<endl;
	
	int liczba;
	cout<<"Podaj liczbę"<<endl;
	if((liczba>0)&&(liczba<=5))
	{
	do
	{
		switch(liczba)
		{
		case 1:
			{
				int x,y;
				cout<<"Podaj liczbę x:"<<endl;
				cin>>x;
				cout<<"Podaj liczbę y:"<<endl;
				cin>>y;
				cout<<"Suma x i y to:"<< x+y <<endl;
				break;
			}
		case 2:
			{
				int a,b;
				cout<<"Podaj liczbę a:"<<endl;
				cin>>a;
				cout<<"Podaj liczbę b:"<<endl;
				cin>>b;
				cout<<"Różnica x i y to:"<< a-b <<endl;
				break;
	        }
		case 3:
			{
				int c,d;
				cout<<"Podaj liczbę c:"<< endl;
				cin>>c;
				cout<<"Podaj liczbę d:"<<endl;
				cin>>d;
				cout<<"Iloczyn x i y to:"<< c*d <<endl;
				break;	
			}
		case 4:
			{
				int e,f;
				cout<<"Podaj liczbę e:"<<endl;
				cin>>e;
				cout<<"Podaj liczbę f:"<<endl;
				cin>>f;
				cout<<"Iloraz x i y to:"<< e/f <<endl;
				break;	
		case 5:
			{
				cout<<"Rezygnujesz."<<endl;
				break;
			}
	    }
	} while((liczba>0)&&(liczba<=5));
	}	
		
	return 0;
	system("pause");
}

W linicje 68 wyskakuje mi błąd. Nie mam pojęcia, co jest źle. 

Answer 1

w linijce 68 masz blad poniewaz wykonujesz instrukcje system("pause") po zakonczeniu funkci return 0; zapisz na odwrot i bedzie okej :)

Comments

Tym razem Ty byłeś pierwszy ;-)

---

Haha musi byc jakas sprawiedliwosc ;D

---

Zrobiłam tak jak mówisz i nadal błąd w tym samym miejscu.

---

nie zgadza się ilość klamr {} :)

---

Właśnie wydaje mi się, że się zgadza, choć mogę się mylić.

:)

---

Dodam tylko, że pomysł wykorzystania pętli do while i switch jest dobry.

Answer 2

return 0;

kończy funkcję main, kompilator krezyczy że system("pause"); się nie wykona.

Answer 3

Nie potrzebnie deklarujesz inne zmienne dla każdego działania a ta pętla też jest niezbyt dobra :P Przeanalizuj sobie ten kod

#include <iostream>
using namespace std;
int main()
{
	int liczba, x, y;

	while(true)
	{
		system("cls");
	        cout << "Menu:" << endl;
	        cout << "[1] Dodawanie" << endl;
	        cout << "[2] Odejmowanie" << endl;
	        cout << "[3] Mnozenie" << endl;
	        cout << "[4] Dzielenie" << endl;
	        cout << "[5] Rezygnuj" << endl << endl;
		cout << "Podaj numer dzialania : "; 
                cin >> liczba;

		if (liczba == 5) { cout << "Rezygnujesz" << endl; break; }

		switch (liczba)
		{
			case 1: 
				cout << "Podaj liczbe x : "; cin >> x;
				cout << "Podaj liczbe y : "; cin >> y;
				cout << "Suma x i y to : " << x + y << endl; 
				break;
			case 2: 
				cout << "Podaj liczbe x : "; cin >> x;
				cout << "Podaj liczbe y : "; cin >> y;
				cout << "Roznica x i y to : " << x - y << endl; 
				break;
			case 3: 
				cout << "Podaj liczbe x : "; cin >> x;
				cout << "Podaj liczbe y : "; cin >> y;
				cout << "Iloczyn x i y to : " << x * y << endl; 
				break;
			case 4: 
				cout << "Podaj liczbe x : "; cin >> x;
				cout << "Podaj liczbe y : "; cin >> y;
				cout << "Iloraz x i y to : " << x / y << endl;
				break;
			default: cout << "Niepoprawne dane !" << endl; break;
		}
		cin.sync();
		cin.get();
	}

	return 0;
}

i zobacz jak działają zmienne oraz ta pętla ze switchem :)

Comments

Okej, tak zrobię.

Dopiero zaczynam z C++.

Bardzo wielu rzeczy jeszcze nie wiem i myślę, że jeszcze tego nie 'czuję'.