[SOLUCIONADO] Error usando Switch y Case

[AORV]

Al hacer este programa usando switch y case, no me deja compilarlo me aparece [Error] case label not within a switch statement y [Error] break statement not within loop or switch, marca en cada case ese error, aqui mi codigo:

Código:

#include <stdio.h>
int main (void)
{
	int n,opcion;
	printf("\tCasa rural\n");
	printf("\n");
	printf("Habitaciones\n");
	printf("n1. Azul\n");
	printf("n2. Roja\n");
	printf("n3. Verde\n");
	printf("n4. Rosa\n");
	printf("n5. Gris\n");
	printf("Ingresa el numero de la habitacion: ");
	scanf("%d", &opcion);
	switch(opcion)
	printf("\n");
	
	{
	case 1:
	if (n==1){
		printf("Habitacion 1: Tiene 2 camas y se encuentra en la primera planta\n");
		break;
			}
	
	case 2:
	if (n==2){
		printf("Habitacion 2: Tiene 1 cama y se encuentra en la primera planta\n");
		break;
	}
	case 3:
	if (n==3){
		printf("Habitacion 3: Tiene 3 camas y se encuentra en la segunda planta\n");
		break;
	}
	case 4:
	if (n==4){
		printf("Habitacion 4: Tiene 2 camas y se encuentra en la segunda planta\n");
		break;
	}
	case 5:
	if (n==5){
		printf("Habitacion 5: Tiene 1 cama y se encuentra en la tercera planta\n");
		default:
			printf("Error ingresa de nuevo");
	}
	
		}
	return 0;
}

A ver si me ayudan, muchas gracias

[CalgaryCorpus]

tienes un printf entre el switch() y la primera "{". No pueden haber instrucciones entremedio.

[AORV]

gracias, si era eso pero ahora al ejecutarlo, ingresando el numero no me da ningun resultado.

[vangodp]

Es normal.
Observe la linea: if ( n == 1 ) {
n nunca le has asignado un valor.
Es mas...eso es una mala practica. Crear una variable e no darle un valor inicial aun que sea un 0 para tener control. Estas comparando el residuo de la memoria que hay en la variable n con 1, y a menos que algún programa buen samaritano te haya dejado un uno antes nunca vas a tener un 1 ahí.

Sin embargo si haces:
n=1;
if ( n == 1 ) {
//blablabla
}
...eso ya es otra cosa.

Observe linea a linea e intente prever que es lo que pasa.

Ademas ese problema es fácil de saber pero otros vas a tener que aprender a usar el debuguer si o si.
Dale un valor a ese n o quita lo de ahí.

Otra cosa que veo, el break debería estar fuera de if no dentro. Lo correcto seria:

Código C:

#include <stdio.h>
int main ( void ) {
    int n, opcion;
    printf ( "\tCasa rural\n" );
    printf ( "\n" );
    printf ( "Habitaciones\n" );
    printf ( "n1. Azul\n" );
    printf ( "n2. Roja\n" );
    printf ( "n3. Verde\n" );
    printf ( "n4. Rosa\n" );
    printf ( "n5. Gris\n" );
    printf ( "Ingresa el numero de la habitacion: " );
    scanf ( "%d", &opcion );
    
    //Eso lo hago para dar un valor a n...lo puedes quitar después si quieres o cambia por 2,3,4....
    n = 1;   //solo puedes elegir 1 en el menú sino te va fallar las otras opciones n==2.... n==4... porque n vale uno n==1
    
    
    
    printf ( "\n" );
    
    switch ( opcion ) {
        case 1:
            if ( n == 1 ) {
                printf ( "Habitacion 1: Tiene 2 camas y se encuentra en la primera planta\n" );
            }
            
            break;
            
        case 2:
            if ( n == 2 ) {
                printf ( "Habitacion 2: Tiene 1 cama y se encuentra en la primera planta\n" );
            }
            
            break;
            
        case 3:
            if ( n == 3 ) {
                printf ( "Habitacion 3: Tiene 3 camas y se encuentra en la segunda planta\n" );
            }
            
            break;
            
        case 4:
            if ( n == 4 ) {
                printf ( "Habitacion 4: Tiene 2 camas y se encuentra en la segunda planta\n" );
            }
            
            break;
            
        case 5:
            if ( n == 5 ) {
                printf ( "Habitacion 5: Tiene 1 cama y se encuentra en la tercera planta\n" );
            }
            
            break;
            
        default:
            printf ( "Error ingresa de nuevo" );
            break;
    }
    
    
    
    
    return 0;
}
 

Observe los break fuera de las sentencias if.

Un truco que puedes usar es poner printf("Dentro de case1"); dentro de case 1 =D para saber si has entrado en ese caso. printf te sirve para dar una depurada chapucera pero efectiva y ver que camino ha tomado tu programa. Si has entrado a un if pon printf("dentro de if"); ....

Código C:

switch ( opcion ) {
        case 1:
            printf("Dentro de case1");
            if ( n == 1 ) {
                printf("Dentro de if.\nn vale 1\n");
                printf ( "Habitacion 1: Tiene 2 camas y se encuentra en la primera planta\n" );
            }
            printf("No hemos entrado a if porque n no es igual que 1.\n");
            break;
        case 2:
            ...
 

Una vez que vea que el programa va como tu quieres les puedes quitar esas lineas o simplemente comentarlas con un //
Default también lleva el break;

[jc_moj]

Hola

Una duda. ¿Qué objetivo tiene el if dentro del case y la variable n? La opción que pulsó el usuario ya la selecciona el switch con la variable opcion.

Saludos

[leosansan]

Iniciado por jc_moj

Hola

Una duda. ¿Qué objetivo tiene el if dentro del case y la variable n? La opción que pulsó el usuario ya la selecciona el switch con la variable opcion.

Saludos

Efectivamente todos esos if y el uso de la variable n, tal como está el código, están de más.

Saludos!.

[AORV]

muchas gracias ya quedo, la verdad es que se me paso eso de que cambie n por opcion, ya le puse n ya que practicamente opcion esta demas, sin embargo no entiendo la parte que dices de poner printf("Dentro de case1") es necesaria? y el break probe poniendolos fuera y dentro de if y me da el mismo resultado. gracias perdon por saber tan poco :(

[vangodp]

No son necesarios! Es que por ejemplo si se imprime el mensaje "Dentro de case1" sabes que has entrado en dicho case. pero si no se imprimiera "Dentro de if...." es que no has entrado en tal if lo que dicho de otro modo tu programa haría "nada".

[jc_moj]

Hola

Me parece que te has liado un poco con el switch.

Cuando se ejecuta el programa, se ejecuta el case adecuado según la variable opción, lo que hace el break es detener la ejecución dentro del case y saltar fuera del switch, es decir, si no lo pones, seguiría ejecutándose el código del siguiente case. Por eso, si lo pones dentro del if, el break solo se ejecuta si entra en el if, por lo que debes sacarlo fuera para que se ejecute siempre.

Tal y como creo que debería funcionar el programa, no necesitas los if dentro del case, el case ya se encarga de seleccionar la opción adecuada según la opción que escogió el usuario.

Saludos

[AORV]

muchas gracias a todos, la verdad es que se muy poco, algun dia quisiera ser como ustedes.

[vangodp]

http://v1.minidosis.org/C++/ XD