Praca domowa zadania 15.2, sprawdzenie,

Siemanko,
czy tak ogólnie to zadanie zostało dobrze wykonane?
Trzeba coś dodać?


#include <iostream>
int main()
{
    int b = 0;
    int a;
    int d;
    int c;
    do
    {
     std::cout << "Obecny wynik: " << b << std::endl;
     std::cout << "Liczba: ";
     std::cin >> a;
     if ( std::cin.good() )
         {
     std::cout << "Liczba 2: ";
     std::cin >> d;
     if ( std::cin.good() )
         {
              if ( a != 0 && d != 0 )
    {
     std::cout << "Co chcesz zrobic? " << std::endl;
     std::cout << "[1] Dodawanie" << std::endl;
     std::cout << "[2] Odejmowanie" << std::endl;
     std::cout << "[3] Mnozenie" << std::endl;
     std::cout << "[4] Dzielenie" << std::endl;
     std::cout << "[5] Rezygnuje" << std::endl;
     std::cin >> c;
     switch ( c )
     {
     case 1:
        b = ( a + d );
        break;
     case 2:
        b = ( a - d );
        break;
     case 3:
        b = ( a * d );
        break;
     case 4:
        b = ( a / d );
        std::cout << "reszta z dzielenia: " << ( a % d ) << std::endl;
        break;
     case 5:
        std::cout << "Do zobaczenia!";
        return 0;
        break;
     case 0:
        return 0;
     default:
        std::cout << "Wybrales zla opcje" << std::endl;
        break;
     }
    }
     else
         std::cout << "Podales zla liczbe" << std::endl;

         }
    else
    {
        std::cout << "Podales zla liczbe, sproboj ponownie" << std::endl;
        std::cin.clear();
        std::cin.sync();
    }
         }
         else
         {
             std::cout << "Podales zla liczbe, sproboj ponownie" << std::endl;
             std::cin.clear();
             std::cin.sync();
         }
    } while ( c != 5 );

    return 0;
}

Nazywaj zmienne intuicyjnie. Składniki działań + - * : jako a i b ujdą, ale dlaczego wybór nazwałeś "c" zamiast "wybor"/"choice". To samo z b. Dlaczego pierwszy składnik to "a" ale drugi już "d"? Czemu nie b? Bo b jest wynikiem? A nie powinno, nazwa dla zmiennej "b" to wynik/result/Ergebnis, cokolwiek, co mówi, że to wynik.
PS. a co jeśli podziele przez 0 :P?

No okej :P
Po prostu robiłem to na szybko i nie chciało mi się zmieniać, ważne dla mnie że dobrze dalej było, gdybym musiał to bym zmienił.
Z tym dzieleniem to zaraz się zastanowię :P

@edit ale jak chcesz podzielić przez zero jeżeli po podaniu 0 w liczbach jest komunikat że źle podano i trzeba jeszcze raz :P

Wybacz, nie zauważyłem, że już wcześniej sprawdzasz, czy a i d są różne od 0. :P Tylko w takim razie nie mogę wykonać działania 0 * 4, które przecież jest poprawne.