Czy dobrze jest skonstruowany ten kalkulator ? [lekcja 15]

/*Napisz prosty kalkulator, który będzie potrafił dodawać, odejmować, mnożyć i dzielić. Program ten ma działać następująco:
1. Wypisuje obecny wynik
2. Wprowadź liczbę
3. Wybierz działanie (jeżeli liczba różna od 0)
4. Wykonaj obliczenia (jeżeli liczba różna od 0)
5. Wróć do kroku 1.
6. Jeżeli wprowadzoną liczbą jest 0, zakończ program. */
#include<iostream>
using namespace std;
int main()
{
    int liczba;
    int wynik = 0;
    do
   
    {
       
        cout << "Wynik : " << wynik << endl;
        cout << "Wprowadz liczbe (wprowadz 0 aby zakonczyc)" << endl;
        cin.clear();
        cin.sync();
        cin >> liczba;
        if( cin.good() )
        {
            int dzialanie;
            cout << "Wprowadz dzialanie" << endl << "[1]Dodawanie" << endl << "[2]Odejmowanie" << endl << "[3]Mnozenie" << endl << "[4]Dzielenie" << endl;
            cin >> dzialanie;
            if( dzialanie < 5 and dzialanie > 0 and cin.good() )
            switch( dzialanie )
            {
            case 1:
                wynik = wynik + liczba;
                break;
            case 2:
                wynik = wynik - liczba;
                break;
            case 3:
                wynik = wynik * liczba;
                break;
            case 4:
                wynik = wynik / liczba;
                break;
               
            }
            else
                 cout << "Zly numer dzialania sproboj jeszcze raz lub podano niewlasciwy znak" << endl;
           
        }
        else
             cout << "Wprowadz normalna liczbe cwoku" << endl;
       
    }
    while( liczba != 0 or cin.fail() );
   
    cout << "Wprowadzono liczbe 0" << endl;
}