Błąd ze switchem.

pytanie zadane 24 października 2016 w C i C++ przez kamilando Nowicjusz (150 p.)

Witam, zrobiłem taki program: Mam jednak problem. Nie wiem czemu, ale przy case 2 w switchu, wyświetla się błąd i program się nie kompiluje. Czy wie ktoś o co chodzi?

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

using namespace std;

int i;
int time;
int opcje;

int main()
{
    odliczanie:
    cout << "Wpisz liczbe, od ktorej chcesz rozpoczac odliczanie:" << endl;
    cin >> i;
    cout << "Wpisz w milisekundach, ile czasu system ma zaczekac, przed pokazaniem nastepnej cyfry" << endl;
    cin >> time;

    for ( i; i >0; i-- )
    {
        cout << i << endl;
        Sleep(time);
        system( "cls" );
    }

    cout << "Aby wylaczyc program, wybierz 1.\n Aby zaczac od nowa, wybierz 2." << endl;

     switch (opcje)
     case 1:
           {
                for(; getch() != 27; );
                cout << "Dziekuje za skorzystanie z programu, [ESC] aby wyjsc." << endl;
                break;
           }
     case 2:           <------------ TU JEST BŁĄD
           {
                goto odliczanie;
                break;
           }

    return 0;
}

komentarz 24 października 2016 przez QizmoPL Stary wyjadacz (11,440 p.)

Zacznijmy od tego blok na kod, dlaczego deklarujesz zmienne globalnie, uzywasz przestrzeni nazw, zamiast drugiego fora uzyj while, po co uzywasz goto, jest to stara juz nie uzywana petla(http://www.algorytm.edu.pl/instrukcja-iteracyjna-ptla/skok-bezwarunkowy-goto.html),

komentarz 24 października 2016 przez plkpiotr Stary wyjadacz (12,420 p.)

Jeśli byłbyś tak łaskaw, to sformatuj ten kod, który wkleiłeś na forum jako kod źródłowy, ułatwi nam to czytanie bez wrzucania go do IDE : ) Fakt, faktem, że podejście do problemu mogło być nieco inne: zmienne globalne, goto, niedobrze to wygląda ; )

1 odpowiedź

odpowiedź 24 października 2016 przez plkpiotr Stary wyjadacz (12,420 p.)
edycja 24 października 2016 przez plkpiotr

Najlepsza

Zabrakło klamerek przy warunku wielokrotnego wyboru switch. Wszystkie instrukcje znajdujące się wewnątrz niego powinny zostać otoczone nawiasami klamrowymi.

Tak zadziała:

switch (opcje)
{
    case 1: {
        for (; getch() != 27; );
        cout << "Dziekuje za skorzystanie z programu, [ESC] aby wyjsc." << endl;
        break;
    }
    case 2: {
        goto odliczanie;
        break;
    }
}