error C2361: inicialización de 'encontrado' se salta por 'defecto' etiqueta

Question

Duplicar posibles:
Why can't variables be declared in a switch statement?

Tengo un extraño error en mi código de abajo:

char choice=Getchar(); 
switch(choice) 
{ 
case 's': 
    cout<<" display tree "; 
    thetree->displaytree(); 
    break; 

case 'i': 
    cout<<" enter value to insert "<<endl; 
    cin>>value; 
    thetree->insert(value); 
    break; 
case 'f' : 
    cout<< "enter value to find "; 
    cin>>value; 
    int found=thetree->find(value); 
    if(found!=-1) 
     cout<<" found = "<<value<<endl; 
     else 
      cout<< " not found " <<value <<endl; 
     break; 
default: 
    cout <<" invalid entry "<<endl;; 
    } 

El compilador de Visual Studio 2010 dice que:

1>c:\users\daviti\documents\visual studio 2010\projects\2-3-4\2-3-4\2-3-4.cpp(317): error C2361: initialization of 'found' is skipped by 'default' label 
1>   c:\users\daviti\documents\visual studio 2010\projects\2-3-4\2-3-4\2-3-4.cpp(308) : see declaration of 'found' 

Creo que he escrito correctamente declaraciones de ruptura y predeterminadas, entonces, ¿dónde está el error?

Answer 1

Necesitas tener ya sea incluir su case 'f': con un corsé de ámbito:

case 'f' : 
{ 
    cout<< "enter value to find "; 
    cin>>value; 
    int found=thetree->find(value); 
    if(found!=-1) 
     cout<<" found = "<<value<<endl; 
    else 
     cout<< " not found " <<value <<endl; 
    break; 
} 

o colocar la declaración de found fuera de las switch

Answer 2

Debe declarar las variables internas de switch 's case dentro de las llaves. es decir

case 'f' : 
{ 
    ... 
    int found=thetree->find(value); 
    ... 
} 

Answer 3

La semántica de un switch son los de una goto: case s no haga introduzca un nuevo alcance. Por lo tanto, found está disponible en su default: case (aunque en realidad no tiene acceso). Saltar sobre una inicialización no trivial es ilegal, por lo que su código se vuelve ilegal.

Dada la complejidad de su case 'f':, la mejor solución es probablemente para descomponerla en una función separada. De no ser así, puede poner todo el caso en {...}, creando un ámbito separado, o renunciar a la inicialización, escribiendo:.

int found; 
found = thetree->find(value); 

(Menciono esto para la integridad Es no la solución que lo haría recomendación.)