dziwny przeskok w switchu c++

Question

Witam mam kłopot w mojej pierwszej grze na konsoli chodzi o dziwny ruch z stron c++ opisałem działanie programu w komentarzach

fragment bo cała gra jest dobra tylko kłopot jest w menu + visual studio nie pokazuje żadnych błędów 

 switch (op1)
    {
    case 1:
    {
        if (Zapis.good() == false)
        {

            cout << "Zapis nie istneje";
            Sleep(2500);
            break;
        }
        else if (Zapis.good() == true) 
        {
            Zapis.open("Daneapk.txt", ios::in);
            cout << "Wczytuje grę"; // Wyświetla się  napis następnie przeskakuje do        drugiej opcji 
            while (getline(Zapis, linia))
            { 
                switch (nr_linii)
                {
                case 1: nick = atoi(linia.c_str()); break;
                case 2: Exp = atoi(linia.c_str()); break;
                case 3: kasa = atoi(linia.c_str()); break;
                case 4: day = atoi(linia.c_str()); break;
                    nr_linii++;
                } 

                Sleep(500);
                
                break;
            }

        }
    }
    case 2:
    {

        Zapis.open("Daneapk.txt", ios::out);

        cout << "Nie używaj spacji !" << endl;
        cout << "Podaj nickname :"; cin >> nick;
        Zapis << nick
            ; Zapis.close();
        break;   // czyli to tej i teraz nadpisuje mi tamten zapis gry 
    }

 

Comments

A to

nick = atoi(linia.c_str());

na pewno dobrze jest?

Po co ten Sleep w linii 27? No i ten break w 29... skoro przerywasz pętle po pierwszej iteracji, to po co w ogóle pętla?

Answer 1

Trochę "pogubiłeś się" z tymi break-ami w case 1  

Przykład jak to poprawić

   switch (op1)
   {
       case 1:
       {
           if (Zapis.good() == false)
           {
               cout << "Zapis nie istneje";
               Sleep(2500);
           }
           else if (Zapis.good() == true) 
           {
               Zapis.open("Daneapk.txt", ios::in);
               cout << "Wczytuje grę";
               
               int nr_linii=0;
               while (getline(Zapis, linia))
               { 
                   switch (++nr_linii)
                   {
                        case 1: nick = atoi(linia.c_str()); break;
                        case 2: Exp  = atoi(linia.c_str()); break;
                        case 3: kasa = atoi(linia.c_str()); break;
                        case 4: day  = atoi(linia.c_str()); break;
                   } 
     
                   Sleep(500);
               }
     
           }
           
           break;
       }
       case 2:
       {
           Zapis.open("Daneapk.txt", ios::out);
     
           cout << "Nie używaj spacji !" << endl;
           cout << "Podaj nickname :"; cin >> nick;
           Zapis << nick;
           Zapis.close();
           break;
       }

 

 

P.S. nick to chyba string, więc czemu masz taki zapis, co chciałeś osiągnąć?

case 1: nick = atoi(linia.c_str()); break;

 

Comments

Bardzo Dziękuję za pomoc nie wiem o czym myślałem że zrobiłem takie błędy

Answer 2

Przestań obwiniać c++ i dodaj break za pierwszym casem, i za każdym z którego chcesz wyskoczyć. Bez break'a po casie switch poleci do końca, lub do pierwsza napotkanego break'a.