Kalkulator Dobry kod ?

[R@F@l]

Trzeba coś zmienić czy jest dobrze ?

I jeszcze pytanie : Jak robić pętle poprzez bool ?

 

 

#include <iostream>
using namespace std;
int main()
{
cout << "------------" << endl;
cout << "|Kalkulator|" << endl;
cout << "| By R@F@L |" << endl;
cout << "------------" << endl << endl;
long long a;
long long b;
int wybor;
int k;

do
{

cout << "Podaj 1 liczbe = ";
cin >> a;
cout << "" << endl;

cout << "Podaj 2 liczbe = ";
cin >> b;
cout << "" << endl;

cout << "	  MENU:  " << endl;
cout << "[1] Dodawanie" << endl;
cout << "[2] Odejmowanie" << endl;
cout << "[3] Mnozenie" << endl;
cout << "[4] Dzielenie" << endl << endl;

cout << "		";
cin >> wybor;
cout << "" << endl;

switch( wybor )
{
	case 1:
		cout << a << " + " << b <<" = " << a + b << endl;
		break;
	case 2:
		cout << a << " - " << b <<" = " << a - b << endl;
		break;
	case 3:
		cout << a << " * " << b <<" = " << a * b << endl;
		break;
	case 4:
		if (b != 0)
		cout <<  a << " / " << b <<" = " << a / b << " reszta = " << a % b << endl;
			else
		cout << "Cholero nie dziel przez zero!" << endl;
		break;
}
cout << "" << endl;
cout << "Napisz 1 jezeli chce liczyc dalej." << endl;
cout << "Napisz 2 jezeli chcesz wylaczyc." <<  endl << endl;
cout << "			  ";
cin >> k;
cin.clear();
cin.sync();
cout << "" << endl;
} while(k != 2 );

cout << "" << endl;
cout << "Koniec Wcisnij klawisz by zamknac !!!" << endl;
cin.sync();
cin.clear();
cin.get();
return 0;
}

 

[iLikeGood]

Skoro Ci działa to jest dobry, pytanie tylko czy optymalny.

 

cout << "" << endl; // Co to robi ? ; ] cout << "\n" albo cout << endl; po co Ci << "" << ?

[`ReborN.]

Sprawdziałem. Program dobry, lecz oczywiście bez if/else. Nie jest on zabezpieczony, szczerze nie wyglada to dobrze jak po kontynuacji kalkulatora, nie jest wyczyszczony ekran i widzimy poprzednie liczby. Tyle ode mnie

[R@F@l]

@up

 

Zabezpieczony przed czym dokładnie ? Bo przed dzieleniem przez 0 jest.

 

A i podał byś co mam wpisać zęby ekran był czysty ?

 

Skoro Ci działa to jest dobry, pytanie tylko czy optymalny.

 

cout << "" << endl; // Co to robi ? ; ] cout << "\n" albo cout << endl; po co Ci << "" << ?

 

Jakoś cout << "\n" mi nie działało dlatego tak sobie zrobiłem Trzeba sobie radzić

[fanmajonezu]

żeby wyczyścić ekran wpisujesz system("cls");

[Shibitos]

W sumie, to mógłbyś napisać klasę zarządzającą konsolą i po prostu pisać linijka po linijce z bufora, a czyszczenie to po prostu pisanie spacji ;p

[AbillMara]

Zabezpiecz kalkulator przed wprowadzeniem błędnych danych..