1

En esta funcion para calcular el IMC, independientemente del valor que ingrese, se ejecuta el "default" y no logro ver el error. Probe con decimales y sin decimales y aparece lo mismo.

 var altura = parseInt(prompt("Ingresa tu altura"));
var peso = parseInt(prompt("Ingresa tu peso"));

function calcularIMC (altura,peso){
  return peso/altura**2;
} 

var imc = calcularIMC(altura,peso);

switch(imc){
  case (imc<=18.5):
    alert("Bajo peso");
    break;

  case ((imc>=18.5) && (imc<=24.9)):
    alert("Peso normal");
    break;

  case ((imc>=25) && (imc<=29.9)):
    alert("Peso sobrepeso");
    break;

  default: 
    alert("Obesidad");
    break;

}
2
  • 1
    solo debes cambiar el switch(imc){ por switch(true){ y eso deberia solventar tu problema!
    – Bryro
    el 20 sep. 2019 a las 19:57
  • muchas gracias!!!
    – Maguii Ta
    el 20 sep. 2019 a las 20:03

2 respuestas 2

4

el error se da porque switch toma la expresion que se le pase y la compara con el case y tu caso ninguna va concidir ya que estas usando otra expresion en el case la forma mas simple de solventar esto es pasar true para que evalue el valor de case y si es true coincida true == true

ejemplo:

 var altura = parseInt(prompt("Ingresa tu altura"));
var peso = parseInt(prompt("Ingresa tu peso"));

function calcularIMC (altura,peso){
  return peso/altura**2;
} 

var imc = calcularIMC(altura,peso);

switch(true){
  case (imc<=18.5):
    alert("Bajo peso");
    break;

  case ((imc>=18.5) && (imc<=24.9)):
    alert("Peso normal");
    break;

  case ((imc>=25) && (imc<=29.9)):
    alert("Peso sobrepeso");
    break;

  default: 
    alert("Obesidad");
    break;

}

2
  • Pero ¿no es una aberración eso de switch(true)? ¿No debería cambiarse toda la estructura switch() por una serie de if .. else if ... else? ¿O es práctica común lo de switch(true) en el mundo javascript?
    – abulafia
    el 20 sep. 2019 a las 20:05
  • @abulafia if.elseif...else deberia ser la mejor solucion para tratar el problema pero tampoco es tan descabellado utilizar switch(true)
    – Bryro
    el 20 sep. 2019 a las 20:15
3

En JavaScript se puede utilizar el switch(true) y luego evaluar cada condición en el case.

En este caso, que son unos "casos predefinidos" la estructura de if ... else if ... else es otra opción, ya que son también pocos casos.

En el caso de que fuesen muchos casos, por legibilidad yo utilizaría la estructura del switch(true) ... case.

Ejemplo usando una estructuraif ... else if ... else:

var altura = parseInt(prompt("Ingresa tu altura"));
var peso = parseInt(prompt("Ingresa tu peso"));

function calcularIMC (altura,peso){
  return peso/altura**2;
} 

var imc = calcularIMC(altura,peso);

if(imc<=18.5){
  alert("Bajo peso");
}else if (imc <=24.9){
  alert("Peso normal");
}else if (imc<29.9){
  alert("Peso sobrepeso");
}else{
  alert("Obesidad");
}
  

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.