Kalkulatorek w C++

Question

Witam pomału staram się ogarnąć temat Programowania w C++ aby móc spokojnie zacząć naukę w JavaScript Proszę o rzetelną ocenę mojego kalkulatora co jest dobrze co jest źle co zmienić ;-) Pozdrawiam was Mistrzowie ;-)

#include <iostream>

int main ()
{

int nr_dzialania;
float liczbaA;
float liczbaB;
using namespace std;
do {
  cout<<" Wybierz Dzialanie Matematyczne"<<endl;

  cout <<"[1] DODAWANIE"<<endl;
  cout <<"[2] ODEJMOWANIE"<<endl;
  cout <<"[3] MONOZENIE"<<endl;
  cout <<"[4] DZIELENIE"<<endl;
  cout <<"[5] KONIEC PROGRAMU"<<endl;

            cin.clear();
            cin.sync();
    cin>>nr_dzialania;
switch(nr_dzialania)
    {
    case 1:
    cout<<"Wprowadz liczbe A ";
                cin>>liczbaA;
            cout<<"Wprowadz liczbe B ";
                cin>>liczbaB;

            cout <<"Dodawanie twoj wynik to= "<<liczbaA+liczbaB<<endl;
            cout<< "[6] Kontynuj Program [5] Zakoncz Program"<<endl;
                cin>> nr_dzialania;
            cin.clear();
            cin.sync();
       break;
    case 2:
            cout<<"Wprowadz liczbe A ";
                cin>>liczbaA;
            cout<<"Wprowadzliczbe B ";
                cin>>liczbaB;
            cout <<"Odejmowanie twoj wynik to= "<<liczbaA-liczbaB<<endl;
            cout<< "[6] Kontynuj Program [5] Zakoncz Program"<<endl;
                cin>> nr_dzialania;
                cin.clear();
                cin.sync();
        break;
    case 3:
        {
            cout<<"Wprowadz liczbe A ";
            cin>>liczbaA;
            cout<<"Wprowadzliczbe B ";
            cin>>liczbaB;
                if (liczbaA==0 || liczbaB==0)
                cout<<"Nie wolno mnozyc przez zero"<<endl;
             else
                cout<<"Mnozenie twoj wynik to= "<<liczbaA*liczbaB<<endl<<endl;
                cout<< "[6] Kontynuj Program [5] Zakoncz Program"<<endl;
                cin>> nr_dzialania;
                cin.clear();
                cin.sync();
        }
        break;
    case 4:
           cout <<"Dzielenie twoj wynik to"<<endl;
           {
            cout<<"Wprowadz liczbe A ";
            cin>>liczbaA;
            cout<<"Wprowadzliczbe B ";
            cin>>liczbaB;
                if (liczbaA==0 || liczbaB==0)
                cout<<"Nie wolno dzielic przez zero"<<endl;

             else
                cout<<"dzielenie twoj wynik to= "<<liczbaA/liczbaB<<endl;
        }
        break;
    }
   }while( nr_dzialania !=5);

    cout<<"KONIEC PROGRAMU DZIEKUJE";

return 0;

}

 

Comments

staram się ogarnąć temat Programowania w C++ aby móc spokojnie zacząć naukę w JavaScript

To żeś wyleciał teraz z tematem. Po kiego grzyba ogarniasz cepa, skoro i tak chcesz się uczyć JSa? 

Answer 1

Według mnie powinieneś ten fragment:

    cout<<"Wprowadz liczbe A ";
                cin>>liczbaA;
            cout<<"Wprowadz liczbe B ";
                cin>>liczbaB;

Dać przed podjęciem decyzji o wyborze działania. Dzięki temu oszczędzisz pisaniu wielokrotnie tego samego. A i kod mocno skrócisz.

Answer 2

Ludzie, błagam, Najpierw czytajcie potem pytajcie.

Zasady korzystania z serwisu i chatu, punkt 2, podpunkt E.

Comments

Co do samego programu:

cin.sync();

// zamien na

cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

W przeciwnym razie program może nie wyczyścić bufora, sync() czyścił go przy okazji lecz nie jest to jego rola. VS np. ci tej funkcji nie zagwarantuje.

cout<<"Wprowadz liczbe A ";
                cin>>liczbaA;
            cout<<"Wprowadz liczbe B ";
                cin>>liczbaB;

// nie wiem czemu jest to tak powcinane, wygląda jakbyś tabulatory rzucał losowo;

case n:
{
   cout<<"Wprowadz liczbe A ";
   cin>>liczbaA;
   cout<<"Wprowadz liczbe B ";
   cin>>liczbaB;
}

// Swoją drogą wywaliłbym to w ogóle przed tego switcha
// albo zrobił funkcję, bo w twoim sposobie
// zamiast 4 potrzebnych linijek masz 16.

 

Answer 3

Bez rewelacji. Dodaj jakieś świecące przyciski, jakiegoś gadułę, co będzie czytał wynik, obsługę głosem i gestami i w ogóle, bo drętwo trochę.