Pętla i switch

Question

Chciałbym, aby w przypadku "default" program się powtórzył. Niestety pętla nie chce zadziałać.

#include <iostream>
#include <string>

using namespace std;

void graj ()
{
cout << "gra" << endl;
}
void wczytaj ()
{

}
void multiplayer ()
{

}
int main ()
{
    int wybor;
 while (wybor != "1" && wybor != "2" && wybor != "3" && wybor != "4")
        {

    cout << "1. Graj"<< endl;
    cout << "2. Wczytaj stan gry"<< endl;
    cout << "3. Tryb Multiplayer"<< endl;
    cout << "4. Koniec"<< endl;
    cout << "Wybierz: ";
    cin >> wybor;
    switch (wybor):

    {
    case 1:
        graj();
        break;
    case 2:
        wczytaj();
        break;
    case 3:
        multiplayer();
        break;
    case 4:
        cout << "Dziekuje za gre!"<< endl;
        break;
    default:
        cout << "Niepoprawny wybor."<< endl;
        break;


    }
}


}

 

Answer 1

#include <iostream>
#include <string>

using namespace std;

void graj()
{
    cout << "gra" << endl;
}

void wczytaj()
{
    cout << "wczytaj" << endl;
}

void multiplayer()
{
    cout << "multiplayer" << endl;
}

int main ()
{
    int wybor;
    while(true)
    {

        cout << "1. Graj"<< endl;
        cout << "2. Wczytaj stan gry"<< endl;
        cout << "3. Tryb Multiplayer"<< endl;
        cout << "4. Koniec"<< endl;
        cout << "Wybierz: ";
        if(cin >> wybor)
        {
            switch (wybor)
            {
            case 1:
                graj();
                break;
            case 2:
                wczytaj();
                break;
            case 3:
                multiplayer();
                break;
            case 4:
                cout << "Dziekuje za gre!"<< endl;
                break;
            default:
                cout << "Niepoprawny wybor."<< endl;
                break;
            }
        }
        else cout<<"Podales litere :C"<<endl;
    }
}

Tak może być?

Comments

"brak nawiasu klamrowego kończącego switch"

Możesz powiedzieć który? Jak tam widzę 3 otwarte i 3 zamknięte

---

Dzięki ^^

---

Źle skopiowałem kod ;)

---

Śwież4k - troszkę zmieniłem kod, by spełniał to co chcesz osiągnąć.

---

Zauważyłem, dzięki. Mógłbyś mi jeszcze wytłuamczyć czemu w pętli

while (wybor != "1" && wybor != "2" && wybor != "3" && wybor != "4")

Nie może być cudzysłowów, bo nie chcę popełniać tego ewentualnego błędu w przyszłości.

Napisałeś to uprzednio, ale przy edycji kodu najwyraźniej usunąłeś to wyjaśnienie

---

Zmienna wybór jest typem liczbowym. Liczby nie można przyrównać do ciągu znaków w C++ (czyli np "ala ma kota"). Czyli taki kod byłby dobrze tak:

while (wybor != 1 && wybor != 2 && wybor != 3 && wybor != 4)

Ewentualnie mógłbyś zmienić typ zmiennej wybór na string i by działało na poprzedniej zasadzie ;) 

---

To też można tłumaczyć kodem tak:

int liczba = 5; // po prostu liczba
string napis = "ala ma kota"; // napis musi miec swój początek i koniec
// definiują je cudzysłowia
char znak = 'a'; // znak nie musi mieć zaznaczone gdzie się kończy a 
// gdzie zaczyna, ale inaczej kompilator nie wiedziałby czy chodzi o // litere a czy o jakąś zmienną, która wcześniej została taka nazwana np

char a = 'd';
char znak = a; // byloby tak bez sensu :)

// no i analogicznie tutaj powinno być:
if(liczba == 5) // ...
if(napis == "5") // ...
if(znak == '5') // ...

 

Answer 2

#include <iostream>
#include <string>

using namespace std;

void graj()
{
    cout << "gra" << endl;
}
void wczytaj()
{

}
void multiplayer()
{

}


int main ()
{

    int wybor;

    cout << "1. Graj"<< endl;
    cout << "2. Wczytaj stan gry"<< endl;
    cout << "3. Tryb Multiplayer"<< endl;
    cout << "4. Koniec"<< endl;
    cout << "Wybierz: ";

    cin>> wybor;

    while (wybor!= 1 && wybor!=2 &&wybor!=3 &&wybor!=4)
    {
        cout << "Niepoprawny wybor" << endl;
        cin.clear();
        cin.sync();
        cin>>wybor;
    }

    switch (wybor)
    {
    case 1:
        graj();
        break;
    case 2:
        wczytaj();
        break;
    case 3:
        multiplayer();
        break;
    case 4:
        cout << "Dziekuje za gre!"<< endl;
        break;
    }
}

//Ja to zrobiłem tak, bez defaulta.

 

Comments

cin.clear() czyści flagi błędu, spowodowane przez wpisanie np tekstu zamiast liczby.

http://cpp0x.pl/kursy/Kurs-C++/Poziom-4/Wczytywanie-zawartosci-pliku-a-kontrola-bledow/485     (------ tutaj co nieco o flagach.

cin.sync() czyści bufor strumienia.

Tutaj trochę konkretniej wytłumaczone (z przykładami)

http://cpp0x.pl/kursy/Kurs-C++/Poziom-1/Obsluga-strumienia-wejsciowego/12

---

Swoją drogą co prawda w takim małym (jak na razie) programie może to wydać się niepotrzebne , ale bardzo zachęcam do nazywania zmiennych i funkcji po angielsku - to bardzo dobry nawyk i fajnie go pielęgnować.