[c++] wlasne projekty KALKULATOR

[ObeyTheLaw]

Witam, zaczynam swoją przygodę z programowaniem. Dzisiaj pod ostrzał rzuciłem kalkulator.

 

 

 

#include <iostream>
#include <cstdlib>
#include <windows.h>

using namespace std;

int a, liczba1, liczba2;

int main()
{
M:
    {
        cout << "PROSTY KALKULATOR \n\n";
        cout << "Dodawanie     [1]"<<endl;
        cout << "Odejmowanie   [2]"<<endl;
        cout << "Mnozenie      [3]"<<endl;
        cout << "Dzielenie     [4]"<<endl;
        cout << "Modulo        [5]   (reszta z dzielenia)";
        cout << "\n\nWybierz dzialanie :";
        cin >> a;
    }
    system("cls");
    switch (a)
    {
        case 1:
            cout <<"Podaj pierwsza liczbe :\a";
            cin >> liczba1;
            cout <<"Podaj druga liczbe :\a";
            cin >> liczba2;
            cout <<"\nWynik dodawania to :\a"<<liczba1+liczba2<<endl;
            Sleep (3000);
            system("cls");
            goto M;
            break;
        case 2:
            cout <<"Podaj pierwsza liczbe :\a";
            cin >> liczba1;
            cout <<"Podaj druga liczbe :\a";
            cin >> liczba2;
            cout <<"\nWynik odejmowania to :\a"<< liczba1-liczba2<<endl;
            Sleep (3000);
            system("cls");
            goto M;
            break;
        case 3:
            cout <<"Podaj pierwsza liczbe :\a";
            cin >> liczba1;
            cout <<"Podaj druga liczbe :\a";
            cin >> liczba2;
            cout <<"\nWynik mnozenia to :\a"<< liczba1*liczba2<<endl;
            Sleep (3000);
            system("cls");
            goto M;
            break;
        case 4:
            cout <<"Podaj pierwsza liczbe :\a";
            cin >> liczba1;
            cout <<"Podaj druga liczbe :\a";
            cin >> liczba2;
            cout <<"\nWynik dzielenia to :\a"<< liczba1/liczba2<<endl;
            Sleep (3000);
            system("cls");
            goto M;
            break;
        case 5:
            cout <<"Podaj pierwsza liczbe :\a";
            cin >> liczba1;
            cout <<"Podaj druga liczbe :\a";
            cin >> liczba2;
            cout <<"\nWynik reszty dzielenia to :\a"<< liczba1%liczba2<<endl;
            Sleep (3000);
            system("cls");
            goto M;
            break;
    }
    return 0;
}

 

 

Co o nim myslicie, czy mozna cos lepiej bylo zrobic ? Czy moze cos dodac ?

[Phoenix_PL]

potęgi, silnie, nie wiem... i zrób to na paru funkcjach.

[ObeyTheLaw]

potęgi, silnie, nie wiem... i zrób to na paru funkcjach.

Po co na kilku funkcjach ?

[Phoenix_PL]

Czytelniej wygląda. I jest bardziej "profesjonalnie". No i to dobre ćwiczenie dla początkującego, robienie wszystkiego w jednej funkcji to średni pomysł.

[RówneTrzy]

Bue goto. Już dużo lepiej wyglądałoby, jeśli użyłbyś pętli while z warunkiem opuszczenia. To nie asembler!

bool exit = false;
while(!exit){
    //pętla, która się powtarza.
    //...

   //...
   //if lub case, gdzie funkcja ma się zamykać:
   exit = true;
}

Być może tak będzie estetyczniej .