Pomoc w sprawdzeniu programu

[Daniok22]

Cześć, od niedawna zacząłem uczyć się nt. języka cpp. Jako początkujący mam do Was, małą prośbę. Bylibyście w stanie zerknąć na mój program i dać mi jakieś rady?. Mianowicie czego wystrzegać się w przyszłości etc. Z góry dzięki

#include <iostream>

using namespace std; 


    int main()
{
        int Liczba1;
cout << "Podaj Liczbe 1,5, lub 10\t";
cin >> Liczba1;
        bool Czy1JestLiczba = cin.good();
    if (Czy1JestLiczba == false )
{
cout << "Blad Aplikacji. Wprowadz liczbe z zakresu 1,5,10\t";
}
    else
    if (Liczba1 != 1 && Liczba1 != 5 && Liczba1 != 10)
{
cout << "Blad Aplikacji. Podaj Liczbe 1,5,10";
}

    else

switch ( Liczba1 )
{
case 1:
    cout << "Jeden + Jeden = \t" << 1 + 1 << endl;
    cout << "Jeden - Jeden = \t" << 1 - 1 << endl;
    cout << "Jeden * Jeden = \t" << 1 * 1 << endl;
    cout << "Jeden / Jeden = \t" << 1 / 1 << endl;
    break;
case 5:
    cout << "Piec + Piec = \t" << 5 + 5 << endl;
    cout << "Piec - Piec = \t" << 5 - 5 << endl;
    cout << "Piec * Piec = \t" << 5 * 5 << endl;
    cout << "Piec / Piec = \t" << 5 / 5 << endl;
    break;
case 10:
    cout << "Dziesiec + Dziesiec = \t " << 10 + 10 << endl;
    cout << "Dziesiec - Dziesiec = \t " << 10 - 10 << endl;
    cout << "Dziesiec * Dziesiec = \t " << 10 * 10 << endl;
    cout << "Dziesiec / Dziesiec = \t " << 10 / 10 << endl;
    break;

}


}

[GrAvItY1]

if (Czy1JestLiczba == false )

{

cout << "Blad Aplikacji. Wprowadz liczbe z zakresu 1,5,10\t";

}

else

if (Liczba1 != 1 && Liczba1 != 5 && Liczba1 != 10)

{

cout << "Blad Aplikacji. Podaj Liczbe 1,5,10";

}

To mógłbyś zrobić w jednym ifie.

[Gość]

Nic złego tu nie ma, ofc. można by to zrobić lepiej ale na obecnym poziomie nie ma najmniejszego sensu tego wymagać.

[Daniok22]

Dzięki za odpowiedź panowie. Czy ten if wyglądałby wtedy mniej więcej tak ?.

 

If (Czy1JestLiczba == false && liczba1 != 1 && liczba1 != 5 && liczba1 != 10)?

Sorki za brak linijki w kodzie ale pisze z telefonu i ciężko tutaj o bbcode

[Szymon2727]

Dzięki za odpowiedź panowie. Czy ten if wyglądałby wtedy mniej więcej tak ?.

 

If (Czy1JestLiczba == false && liczba1 != 1 && liczba1 != 5 && liczba1 != 10)?

Sorki za brak linijki w kodzie ale pisze z telefonu i ciężko tutaj o bbcode

 

Mniej więcej źle spojrzałem - ta pętla się wykona wtedy gdy Czy1jestLiczba zwróci wartość false i liczby będą różne od podanych liczb - co jest niemożliwe, żeby spełnić oba warunki, ale lepiej już zapętlać podawanie informacji:

do
{
std::cout << " Podaj 1 " << std::endl;
std::cin >> a;
}
while (a != 1);

[Sopelek997]

If (Czy1JestLiczba == false || (liczba1 != 1 && liczba1 != 5 && liczba1 != 10))

[Daniok22]

Dzięki za odpowiedzi panowie, warto zapamiętać na przyszłość żeby skrócić kod i żeby był bardziej przejrzysty