Problemas con el Switch... Case

hcarretero · #1 (**permalink**) 21/03/2009, 14:01

Hola a todos.

Estoy empezando a programar en PHP. Tengo una situación con la sentencia Switch... Case. Explico, en una primera parte el usuario debe de escoger entre 10 opciones disponibles (por ejemplo escogo la no. 7), en una segunda parte el usuario debe de escoger entre 3 opciones diponibles (por ejemplo 3). El proceso del switch ejecuta primero la opcion 7 y despues la opcion 3, sin embargo tambien ejecuta la opcion 3 de la parte 8, y la opcion 3 de la parte 9 y la opcion 3 de la parte 10. ¿Que debo de hacer para corregir este error...?
Mil Gracias.

David · #2 (**permalink**) 21/03/2009, 14:09

Primero debes mostrarnos cuál es la estructura switch que tienes para que podamos saber por qué está dando problemas.

Lectura recomendada: http://www.php.net/switch

hcarretero · #3 (**permalink**) 21/03/2009, 17:25

Claro... Gracias.... Es la siguiente.... (Estas son las 3 primeras opciones...)

switch($destino){
case "1": // Bloque 1 (Destino=Tijuana)
switch($categoria){
case "1":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Tijuana", " ";
echo "Categoría: Gran Turismo"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$gta)+($ninos*$gtn))*$estanci a);
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
case "2":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Tijuana", " ";
echo "Categoría: Turismo"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$ta)+($ninos*$tn))*$estancia) ;
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
case "3":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Tijuana", " ";
echo "Categoría: Cinco Estrellas"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$cincoa)+($ninos*$cincon))*$e stancia);
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
default:
echo "Opción no valida...";
}
case "2": // Bloque 2 (Destino=Ciudad Juárez)
switch($categoria){
case "1":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Ciudad Juárez", " ";
echo "Categoría: Gran Turismo"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$gta)+($ninos*$gtn))*$estanci a);
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
case "2":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Ciudad Juárez", " ";
echo "Categoría: Turismo"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$ta)+($ninos*$tn))*$estancia) ;
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
case "3":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Ciudad Juárez", " ";
echo "Categoría: Cinco Estrellas"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$cincoa)+($ninos*$cincon))*$e stancia);
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
default:
echo "Opción no valida...";
}
case "3": // Bloque 3 (Destino=Monterrey)
switch($categoria){
case "1":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Monterrey", " ";
echo "Categoría: Gran Turismo"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$gta)+($ninos*$gtn))*$estanci a);
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
case "2":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Monterrey", " ";
echo "Categoría: Turismo"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$ta)+($ninos*$tn))*$estancia) ;
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
case "3":
echo "Has seleccionado lo siguiente:"," ";
echo "Destino: Monterrey", " ";
echo "Categoría: Cinco Estrellas"," ";
echo "Estancia: $estancia noches"," ";
echo "Adultos: $adultos"," ";
echo "Niños: $ninos"," ";
$importe=((($adultos*$cincoa)+($ninos*$cincon))*$e stancia);
printf("El total a pagar es de $ %.2f",$importe);
echo " "," ";
break;
default:
echo "Opción no valida...";
}

3du4rd0 · #4 (**permalink**) 22/03/2009, 04:20

Cita:

Iniciado por hcarretero

Claro... Gracias.... Es la siguiente.... (Estas son las 3 primeras opciones...)

switch($destino){
case "1": // Bloque 1 (Destino=Tijuana)
switch($categoria){
...

Se debe que te falta detener el case con break;

Prueba asi:

Código PHP:

  <?php 
switch($destino){ 
case "1": // Bloque 1 (Destino=Tijuana) 
switch($categoria){ 
case "1": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Tijuana", "<br>"; 
echo "Categoría: Gran Turismo","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$gta)+($ninos*$gtn))*$estanci a); 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
case "2": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Tijuana", "<br>"; 
echo "Categoría: Turismo","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$ta)+($ninos*$tn))*$estancia) ; 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
case "3": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Tijuana", "<br>"; 
echo "Categoría: Cinco Estrellas","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$cincoa)+($ninos*$cincon))*$e stancia); 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
default: 
echo "Opción no valida..."; 
} 
break; ///<<< estas te faltan 
case "2": // Bloque 2 (Destino=Ciudad Juárez) 
switch($categoria){ 
case "1": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Ciudad Juárez", "<br>"; 
echo "Categoría: Gran Turismo","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$gta)+($ninos*$gtn))*$estanci a); 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
case "2": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Ciudad Juárez", "<br>"; 
echo "Categoría: Turismo","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$ta)+($ninos*$tn))*$estancia) ; 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
case "3": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Ciudad Juárez", "<br>"; 
echo "Categoría: Cinco Estrellas","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$cincoa)+($ninos*$cincon))*$e stancia); 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
default: 
echo "Opción no valida..."; 
} 
break; ///<<< estas te faltan 
case "3": // Bloque 3 (Destino=Monterrey) 
switch($categoria){ 
case "1": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Monterrey", "<br>"; 
echo "Categoría: Gran Turismo","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$gta)+($ninos*$gtn))*$estanci a); 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
case "2": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Monterrey", "<br>"; 
echo "Categoría: Turismo","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$ta)+($ninos*$tn))*$estancia) ; 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
case "3": 
echo "Has seleccionado lo siguiente:","<br>"; 
echo "Destino: Monterrey", "<br>"; 
echo "Categoría: Cinco Estrellas","<br>"; 
echo "Estancia: $estancia noches","<br>"; 
echo "Adultos: $adultos","<br>"; 
echo "Niños: $ninos","<br>"; 
$importe=((($adultos*$cincoa)+($ninos*$cincon))*$e stancia); 
printf("El total a pagar es de $ %.2f",$importe); 
echo " ","<br>"; 
break; 
default: 
echo "Opción no valida..."; 
} 
break; ///<<< estas te faltan 
default: 
//// este te servira para dar error. 
} 
 
?>

Es un problema comun te puede servir Notepad2 este te colorea el codigo y te muestra si cierras bien las { } es muy util. Aparte distingues mejor el codigo.

hcarretero · #5 (**permalink**) 22/03/2009, 13:07

Muchas gracias.... hago las modificaciones y te cuento.....

Holo 3du4rd0, muchas gracias... si me funcionaron tus modificaciones... mil gracias... Me sugieres que utilice el Notepad2, yo utilizo el ConText, cual es mejor...?

3du4rd0 · #6 (**permalink**) 22/03/2009, 23:38

Ya vi el ConText pero aun asi me gusta mas el notepad2 ya que es igual al notepad que trae windows y no consume nada, y el coloreo es muy util.

Igual descargalo y pruebalo.

Saludos

rafaconpu · #7 (**permalink**) 23/03/2009, 02:09

Básicamente como bien han explicado ya y veo que solucionado este link, cada vez que se utilice un switch ... case .... para evitar que tras ejecutar la opción correspondiente siga ejecutándose el código, hay que poner siempre break;

Otro caso distinto es que a partir de un case se tenga que ejecutar los siguientes "cases" en tal caso no se pone break;

En tu caso tienes dos switch anidados. Por cada uno de ellos deberás tener los correspondientes break; y dicho sea de paso, tampoco estaría mal tener el código tabulado para una mejor claridad del mismo.

Saludos.