error: case label ''3'' not within a switch statement

Question

Siemanko, mam taki problem. Mianowicie nie wiedzieć czemu wyskakuje mi błąd podany w temacie pytania. Sprawdziłem go w google, jednak w większości problem dotyczył średnika postawionego tuż za instrukcją switch, jednak u mnie go nie ma. Tutaj jest mój kod:

#include <iostream>
#include <fstream>
#include <cstdlib>

using namespace std;
string imie, nazwisko;
int nr_telefonu;

int main()
{

   fstream plik;

   plik.open("wizytowka.txt", ios::in);

   if(plik.good()==false)
     {

       cout<<"Brak takiego pliku!";
       exit(0);
     }
     string linia;
     int nr_linii=1;
     while(getline(plik,linia))
     {
        switch(nr_linii)

        case '1':
            {
            imie=linia;
             break;
            }

        case '2':
            {
            nazwisko=linia;
             break;
            }
        case '3':
            {
             nr_telefonu=atoi(linia.c_str());
              break;
            }

         nr_linii++;
     }


   plik.close();

   cout<<imie<<endl;
   cout<<nazwisko<<endl;
   cout<<nr_telefonu<<endl;
    return 0;
}

 

Answer 1

Musisz ująć wszystkie case'y w klamry:

switch(sth)
{
   case a:
      break;
   case b:
     break;
}

 

EDIT: Patrycjerz mnie uprzedził. :)

 

Answer 2

Wydaje mi się ze próbujesz porównać int nr_lini z '1', '2', '3' czyli charami zamień to na 1, 2, 3 czyli będzie case 1: i brak klamerek

switch(arg)
{
    case 1:
break;
case 2:
 break;

}

 

Answer 3

A niby jak wygląda instrukcja switch?

switch(zmienna)
{
case wartosc_1:
    //jakiś kod
    break;
    
case wartosc_2:
    //jakiś kod
    break;
    
case wartosc_n:
    //jakiś kod
    break;
    
default:
    //jakiś kod
    break;
}

Z kolei porównywanie liczby do znaku.

Answer 4

Posłucham waszych wszystkich rad, tj usunąłem apostrofy z case'ów, dodałem case 4 z defaultem i ująłem wszystkie case'y w klamry, jedank problem nie zniknął

Comments

Wrzuć kod po zmianie

---

a nie, jednak mój błąd, zapomniałem usunąć jednej klamry, teraz jest ok, trzeba było po prostu wstawić tego defaulta i wziąć wszystkie case'y w klamry, dzięki wszystkim za pomoc

---

Tylko o klamry chodziło, label default jest opcjonalny. ;)