Problem z Switch

Question

Witam, jak dodac tutaj aby przy "default" wracalo mnie do "Twoj wybor:" i tak sie robilo az nie wpisze poprawnej wartosci czyli 1 lub 2? :) Probowalem petlami i tak dalej, ale nie wiem jak po prostu do tamtego momentu wrocic.

Probowalem return main(); ale to nie przynioslo niczego dobrego :D

 

#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int wybor;
string nickGracza;

void menu()
{
    cout << "Wybierz jedna z opcji:" << endl;
    cout << "-------------------------------------------" << endl;
    cout << "1. Graj" << endl;
    cout << "2. Pomoc" << endl;
    cout << "-------------------------------------------" << endl;
}

int main()
{
    cout << "Witaj - poczatek";
    menu();
    cout << "Twoj wybor: ";
    cin >> wybor;
    switch (wybor)
    {
        case 1:
            {
                cout << "funkcja gry";
            }
        break;
        case 2:
            {
                cout << "pomoc";
            }
        break;
        default:
            {
                cout << "zle wpisales gamoniu";
            }
    }

    return 0;
}

 

Comments

Gdybyś zamiast tworzyć switcha w main, wstawił całość do osobnej funkcji, to w razie innego wyboru mógłbyś ponownie wywołać tę funkcję, tak jak chciałeś zrobić return main. Polecam (przynajmniej ja tak się nauczyłem) zamiast cin, używać funkcji getch():

wybor = getch();
switch (wybor)
{
     case '1': 
     case '2': 
...

 Dzięki temu wpisanie litery nie wysypie programu, a samego wyboru nie trzeba potwierdzać enterem.

Answer 1

Witaj.

Proponuje dodać jeszcze 3 opcje Wyjdź

Wtedy Twój problem będzie można rozwiązać stosując  np to rozwiązanie

Weź kod programu do nieskończonej pętli i dodaj jeszcze jedną opcje w switchu w ktrrej ciele będzie funkcja exit(0); która zakończy jej działanie jeżeli użytkownik będzie chciał przestać korzystać z aplikacji.

Żeby rozwiązać problem z daniem szasy na wpisanie dobrych danych do switcha w instrukcji deafault wpisz continue co wywoła ponowne powtórzenie pętli.

 

Przykładowy kod podam w komentarzu.

 

Edit.

Stosowanie goto to zła praktyka. Postaraj się to zrobić na normalnych pętlach.

Comments

Kod

#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int wybor;
string nickGracza;

void menu()
{
    cout << "Wybierz jedna z opcji:" << endl;
    cout << "-------------------------------------------" << endl;
    cout << "1. Graj" << endl;
    cout << "2. Pomoc" << endl;
    cout<<"3. Wyjscie"<<endl; //dodatkowa opcja
    cout << "-------------------------------------------" << endl;
}

int main()
{
     // wyrzuciłem etykiete od goto
        cout << "Witaj - poczatek";
        menu();
        for(;;)  //rownie dobrze moze byc petla while

        {


        cout << "Twoj wybor: ";
        cin >> wybor;
         if ( !cin.good())  //opis w komentarzu do posta 
        {
            cout <<"Nie podano liczy"<<endl;
            cin.clear();  //naprawia flago bledu
            cin.sync(); // czysci strumien wejsciowy 
            continue;
        }
        switch (wybor)
        {
            case 1:
                {
                    cout << "funkcja gry";
                }
            break;
            case 2:
                {
                    cout << "pomoc";
                }
            break;
            case 3:
                {
                    exit(0); // wyjście z programu
                }
                break;
            default:
                {
                    cout << "zle wpisales gamoniu";
                    Sleep(1000);
                    system("cls");
                    continue; //     dodalem continue zeby petla for wykonala sie jeszcze raz
                }
        }

        }
    return 0;
}

 

---

PS

Metoda cin.good(); sprawdza czy flagi obie flagi błędu mają dobre wartości

---

Nie rozumiem zbytnio ale w praktyce to pocwicze i zobacze jak to dziala, natomiast jezeli teraz wpisze poprawna wartosc czyli "1" lub "2" to i tak znowu mi kaze wpisac co wybieram, wiec funkcja nieskonczona to chyba nie jest dobry pomysl, lub trzeba dac jej jakis warunek, nie wiem, masz moze pomysl? :)

 

PS : Bardzo dziekuje za pomoc i poswiecony czas ;)

---

Nie dokońca rozumiem co chcesz osiągnąć. Może napisze w punktach jak ja to rozumiem.

 

1 Program ma mieć 3 opcje ( Gry , Pomocy  i naturalnie opcje wyjścia)

2 Ma ograniczyć urzytkownikawi wpisanie niepoprawnych danych ( wydawało mi się że o to chodzi w pytaniu)

 

Co jeszcze ma robić?  Jak ma działać ?

---

Jezeli wpiszesz "1" ze chcesz grac to kaze ponownie wybrac jedna z opcji, a powinno sie wyswietlic jedynie "funkcja gry" po wpisaniu "1" :)

tak samo z "2", wyskakuje ponowna opcja wyboru, a powinno tylko wyskoczyc "pomoc"

---

Looz.  

Zaraz wstawie kod.

---

#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int wybor;
string nickGracza;

void menu()
{
    cout << "Wybierz jedna z opcji:" << endl;
    cout << "-------------------------------------------" << endl;
    cout << "1. Graj" << endl;
    cout << "2. Pomoc" << endl;
    cout<<"3. Wyjscie"<<endl; //dodatkowa opcja
    cout << "-------------------------------------------" << endl;
}

int main()
{
     // wyrzuciłem etykiete od goto
        cout << "Witaj - poczatek";
        menu();
        bool kontrolka =1; // tworze zmienna pomocnicza i nadaje jej wartosc 1
        while (kontrolka ) //zmienilem na pentle while

        {


        cout << "Twoj wybor: ";
        cin >> wybor;

        if ( !cin.good())
        {
            cout <<"Nie podano liczby"<<endl;
            cin.clear();  //naprawia flago bledu
            cin.sync(); // czysci strumien wejsciowy
            continue;
        }
        switch (wybor)
        {
            case 1:
                {
                    cout << "funkcja gry";
                    kontrolka=0;
                }
            break;
            case 2:
                {
                    cout << "pomoc";
                    kontrolka=0;
                }
            break;
            case 3:
                {
                    exit(0); // wyjście z programu
                }
                break;
            default:
                {
                    cout << "zle wpisales gamoniu";
                    Sleep(1000);
                    system("cls");
                    continue; //     dodalem continue zeby petla for wykonala sie jeszcze raz
                }
        }

        }
    return 0;
}

 

---

wszystko dziala jak nalezy, bardzo dziekuje! :)

---

Nie ma problemu :)

Answer 2

#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int wybor;
string nickGracza;

void menu()
{
    cout << "Wybierz jedna z opcji:" << endl;
    cout << "-------------------------------------------" << endl;
    cout << "1. Graj" << endl;
    cout << "2. Pomoc" << endl;
    cout << "-------------------------------------------" << endl;
}

int main()
{
    cout << "Witaj - poczatek";
    do{
        menu();
        cout << "Twoj wybor: ";
        cin >> wybor;
        switch (wybor)
        {
            case 1:
                {
                    cout << "funkcja gry" << endl;
                }break;
            case 2:
                {
                    cout << "pomoc" << endl;
                }break;
            default:
                {
                    cout << "zle wpisales gamoniu\n" << endl;
                }break;
        }
    }while(wybor < 1 || wybor > 2);
    return 0;
}

 

Comments

Jeszcze lepszym rozwiązaniem jest:

#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int wybor;
string nickGracza;

void menu();

bool main_menu ()
{
        menu();
        cout << "Twoj wybor: ";
        cin >> wybor;
        switch (wybor)
        {
            case 1:
                {
                    cout << "funkcja gry" << endl;
                }   return false;
            case 2:
                {
                    cout << "pomoc" << endl;
                    return false;
                }
            default:
                {
                    cout << "zle wpisales gamoniu\n" << endl;
                }   return true;
        }
}

int main()
{
    cout << "Witaj - poczatek";
    while(main_menu());
    return 0;
}

void menu()
{
    cout << "Wybierz jedna z opcji:" << endl;
    cout << "-------------------------------------------" << endl;
    cout << "1. Graj" << endl;
    cout << "2. Pomoc" << endl;
    cout << "-------------------------------------------" << endl;
}

 

---

nie do końca bo przy kompilowaniu mam errory :P pracuje na win 7

 

"error: stray '\240' in program"

---

która linia?

Answer 3

Chyba wystarczy objąć całego maina (ewentualnie część) niekończącą się pętlą while o w taki sposób:

#include <iostream>
#include <windows.h>
#include <cstdlib>

using namespace std;

int wybor;
string nickGracza;

void menu()
{
    cout << "Wybierz jedna z opcji:" << endl;
    cout << "-------------------------------------------" << endl;
    cout << "1. Graj" << endl;
    cout << "2. Pomoc" << endl;
    cout << "-------------------------------------------" << endl;
}

int main()
{
    while(true)
    {
        cout << "Witaj - poczatek";
        menu();
        cout << "Twoj wybor: ";
        cin >> wybor;
        switch (wybor)
        {
            case 1:
                {
                    cout << "funkcja gry";
                }
            break;
            case 2:
                {
                    cout << "pomoc";
                }
            break;
            default:
                {
                    cout << "zle wpisales gamoniu";
                    Sleep(1000); // odczekanie sekundy
                    system("cls"); // wyczyszczenie konsoli
                }
        }
    }

    return 0;
}

 

Comments

po wczytaniu danych możesz sprawdzić czy dane pasują do formatu za pomocą funkcji cin.good  

jeżeli nie będą pasowały to wyczyść strumień popraw flagi błędów i zastosuj continue jak na przykładnie jaki podałem niżej

---

zbytnio nie rozumiem :/ poczytalem troche o cin.good ale nie rozumiem jak to dziala takze chyba poszukam innego rozwiazania

---

Twoja decyzja. Jednak jeśli zamirzasz uczyć się cpp to Cie to nie ominie bo to podstawa.

Jak chcesz moge dopisać fragment kodu o którym gadamy i z grubsza opowiedzieć co się dzieje w komentarzach

---

Jezeli moglbys to by bylo fajnie :)

---

(Zakładam że pracujesz pod windowsem. Na linukcie sprawa jest podobne z różnicą w 1 detalu  )

 

Za wprowadzanie danych odpowiedzialny jest obiekt cin. Posiada od 2 flagi błędu ( dwie zmienne bool które są ustawione na 0 przy tworzeniu obiektu cin). Jeżeli wprowadzanie danych się niepowiedzie z powodu złych danych wprowadzonych przez  urzytkownika to pierwsza z nich zmienia swoją wartość co jest sygnałem dla programisty że coś jest nie tak)

Jeśli chcesz dalej wprowadzać dane to musisz naprawić obiekt cin. Robisz to za pomocą dwuch metod

cin.clear(); //ustawia flagi błędu na wartości początkowe

cin.sync();// czyści buffor tzn usuwa ze strumienia wszyszystko ( co jest przydatne kiedy urzytkownik do zmiennej int chce zapisać "hwergyw")

 

przykład zastosowanie tych metod podam w kodzie programu który wstawiłem ( zedytuje go)

Answer 4

Spróbuj użyć funkcji goto https://msdn.microsoft.com/pl-pl/library/b34dt9cd.aspx

Comments

Co za głupi pomysł.