Spalanie samochodu

Question

Witam, jestem  w trakcie pisania programu i utknąłem w takim miejscu, że potrzebuje pomocy jak wyjść z pętli, gdyż program nie chce mi przejść do kolejnego pytania, pewnie coś banalnego, ale dopiero zaczynam naukę i jeszcze nie dostrzegam błędów.

oto kod:

 

#include <iostream>

int main()
{
using namespace std;
int zuzycie, paliwo, wybor, wlasne_odleglosc, odleglosc, wlasne_zuzycie;

   cout << "Jak daleko chcesz jechac?" << endl;
   cout << "1. 100km" << endl;
   cout << "2. Pozwol mi wybrac: " << endl;
   cin >> wybor;
{
        while ( wybor == 1 )
            {
        odleglosc = 100;
        cout << "Wybrales odleglosc podrozy na " << odleglosc << "km. ";
            };
        while ( wybor == 2 )
            {
        cout << "Mozesz teraz podac wlasna odleglosc podrozy: " << endl;
        cin >> wlasne_odleglosc;
        odleglosc==wlasne_odleglosc;
        cin.clear();                            // czysci strumien wejsciowy, uzyty w celu zabezpieczenia przed podaniem np. liter
        cin.sync();
            };
        while ( wybor !=1 && wybor !=2 )
            {
            cout << "Wybierz poprawny klawisz: ";
            cin >> wybor;
            };
};
cout << "Teraz pora na srednie spalanie twojego auta: "<<endl;
cout << "1. 10 litrow na 100km" <<endl;
cout << "2. Podaj wlasne: ";

cin >> wybor;
{
    while ( wybor == 1 )
            {
        zuzycie = (100 / 10);
        cout << "Wybrales srednie spalanie auta na: " << zuzycie << "litrow na 100km. ";
            };
        while ( wybor == 2 )
            {
        cout << "Mozesz teraz podac wlasne spalanie auta: " << endl;
        cin >> wlasne_zuzycie;
        zuzycie == wlasne_zuzycie;
        cout << "Wybrales srednie spalanie auta na: " << wlasne_zuzycie << "litrow na 100km. ";
        cin.clear();                            // czysci strumien wejsciowy, uzyty w celu zabezpieczenia przed podaniem np. liter
        cin.sync();
            };
        while (wybor !=1 && wybor !=2)
            {
            cout << "Wybierz poprawny klawisz: ";
            cin >> wybor;
            };
}
   return 0;
}

 

Answer 1

{
        while ( wybor == 1 )
            {
        odleglosc = 100;
        cout << "Wybrales odleglosc podrozy na " << odleglosc << "km. ";
            };
        while ( wybor == 2 )
            {
        cout << "Mozesz teraz podac wlasna odleglosc podrozy: " << endl;
        cin >> wlasne_odleglosc;
        odleglosc==wlasne_odleglosc;
        cin.clear();                            // czysci strumien wejsciowy, uzyty w celu zabezpieczenia przed podaniem np. liter
        cin.sync();
            };
        while ( wybor !=1 && wybor !=2 )
            {
            cout << "Wybierz poprawny klawisz: ";
            cin >> wybor;
            };
};

tutaj zamiast while powinno być if albo switch bo while działa dopóki to co jest w nawiasach jest prawdziwe, porównywanie zmiennej której nie zmieniasz jest zawsze, poza tym te klamry na zewnątrz to jakieś do dekoracji?

Comments

Zmienilem na switch i dziala dobrze, tylko mam problem z zabezpieczniem wyboru, np jezeli nacisne 3 ( nie ma takiego przypisu ) wyświetla sie komunikat błędu poprawnie i program woła jeszcze raz żeby podać klawisz, tylko że gdy podam jeszcze raz błędny klawisz program przechodzi dalej, zamiast zapętlić się aż dostanie 1 lub 2. 

 

#include <iostream>

int main()
{
using namespace std;
int zuzycie, paliwo, wybor, wlasne_odleglosc, odleglosc, wlasne_zuzycie;

   cout << "Jak daleko chcesz jechac?" << endl;
   cout << "1. 100km" << endl;
   cout << "2. Pozwol mi wybrac: " << endl;
   cin >> wybor;

    switch(wybor)
    {
    case 1:
        {
        odleglosc = 100;
        cout << "Wybrales odleglosc podrozy na " << odleglosc << "km. ";
        };
    break;

    case 2:
        {
        cout << "Mozesz teraz podac wlasna odleglosc podrozy: " << endl;
        cin >> wlasne_odleglosc;
        odleglosc==wlasne_odleglosc;
        cin.clear();                            // czysci strumien wejsciowy, uzyty w celu zabezpieczenia przed podaniem np. liter
        cin.sync();
        };
    break;

    default:
        {
        cout << "Nie ma takiej opcji w menu: ";
        cin >> wybor;
        cin.clear();                            // czysci strumien wejsciowy, uzyty w celu zabezpieczenia przed podaniem np. liter
        cin.sync();
        };
    };
cout << "Teraz pora na srednie spalanie twojego auta: "<<endl;
cout << "1. 10 litrow na 100km" <<endl;
cout << "2. Podaj wlasne: ";

cin >> wybor;

    switch(wybor)
    {
    case 1:
        {
        zuzycie = (100 / 10);
        cout << "Wybrales srednie spalanie auta na: " << zuzycie << "litrow na 100km. ";
        };
    break;

    case 2:
        {
        cout << "Mozesz teraz podac wlasne spalanie auta: " << endl;
        cin >> wlasne_zuzycie;
        zuzycie == wlasne_zuzycie;
        cout << "Wybrales srednie spalanie auta na: " << wlasne_zuzycie << "litrow na 100km. ";
        cin.clear();                            // czysci strumien wejsciowy, uzyty w celu zabezpieczenia przed podaniem np. liter
        cin.sync();
        };
    break;

    default:
        {
            if (wybor == 1 && wybor ==2)
            cout << "ok";
            else
            cout << "Wybierz poprawny klawisz: ";
            cin >> wybor;

        };
    };
return 0;
};