2

Por que ponga lo que ponga en el prompt el resultado es que mi instrumento es de viento? no encuentro el error.

let musica = prompt("Que instrumento tocas?");

if (musica === "saxofón" || "flauta" || "clarinete" || "trompeta" || "oboe") {
  console.log(`${musica} es un instrumento de viento`);
} else if (musica === "guitarra" || "arpa" || "violín" || "piano") {
  console.log(`${musica} es un instrumento de cuerda`);
} else if (musica === "timbal" || "tambor" || "platillos" || "bombo") {
  console.log(`${musica} es un instrumento de percusion`);
} else if (musica === "bajo electrico" || "guitarra eléctrica" || "theremín" ||
  "sintetizado") {
  console.log(`${musica} es un instrumento electrico`);
} else {
  console.log(`${musica} tiene un error vuelvelo a completar`)
}

6
  • 7
    Esta sintaxis: musica === "saxofón" ||"flauta" || "clarinete" || "trompeta" || "oboe" no es correcta, en todo caso debe ser: musica === "algo" || musica === "otro" ........
    – user128299
    Commented el 18 nov. 2020 a las 16:46
  • @BetaM es cierto, pero ojala fuese correcta, acortaria bastante las cosas si javascript algun dia la decide implementar, pero por el momento veo que lo mejor que el usuario podria hacer para comparar eso es simplemente clasificando informacion en diferentes array.
    – Layer
    Commented el 18 nov. 2020 a las 16:48
  • 1
    Debe ser una tarea, esta solo haciendo un promtp y comparando valores. Quizás aun no pasan a arrays. @Riven
    – Excorpion
    Commented el 18 nov. 2020 a las 16:50
  • Puedes usar arreglos: if(['saxofón', 'flauta', 'clarinete', 'trompeta', oboe'].includes(musica)) { console.log('es de viento'); }
    – Triby
    Commented el 18 nov. 2020 a las 17:01
  • @Triby Como le dije a Riven, puede que sea una tarea, y el nivel requerido es ese, sabemos que se puede mejorar bastante, pero es algo que le han pedido.
    – Excorpion
    Commented el 18 nov. 2020 a las 17:02

2 respuestas 2

3

Entre cada '||' debes poner otra vez el comparador

musica = prompt("Que instrumento tocas?");

if (musica == "saxofón" || musica == "flauta" || musica == "clarinete" || musica == "trompeta" || musica == "oboe") {
  console.log(`${musica} es un instrumento de viento`);
} else if (musica == "guitarra" || musica == "arpa" || musica == "violín" || musica == "piano") {
  console.log(`${musica} es un instrumento de cuerda`);
} else if (musica == "timbal" || musica == "tambor" || musica == "platillos" || musica == "bombo") {
  console.log(`${musica} es un instrumento de percusion`);
} else if (musica == "bajo electrico") {
  console.log(`${musica} es un instrumento electrico`);
} else {
  console.log(`${musica} tiene un error vuelvelo a completar`)
}

3
  • Exacto, esta es la respuesta que BetaM sugirio
    – Excorpion
    Commented el 18 nov. 2020 a las 17:06
  • Será colocar la comparación de nuevo, pues el comparador solo sería el ===
    – user128299
    Commented el 18 nov. 2020 a las 17:33
  • 1
    Es preferible usar siempre el operador de comparación estricto ===, aún cuando sepamos de antemano que los valores a comparar son del mismo tipo, esto como un consejo de buena práctica. Saludos Commented el 18 nov. 2020 a las 17:44
1

Podrías crear arrays independientes por cada tipo de instrumento y comporbar si el valor introducido esta en alguna de ellas.

let instrumento = prompt("Que instrumento tocas?");

let instrumentosViento = ["saxofón", "flauta", "clarinete", "trompeta", "oboe"];
if (instrumentosViento.includes(instrumento)) {
  console.log(`${instrumento} es un instrumento de viento`);
}

2
  • Y esta es la respuesta que sugerí xd, de todas maneras es valida, pero aun no sabemos si el usuario puede usar arrays. o bueno, almenos pensaba hacer exactamente lo mismo
    – Layer
    Commented el 18 nov. 2020 a las 17:19
  • Esta respuesta me gusta más porque reduce mucho el uso de los operadores de cortocircuito ||. +1 Commented el 18 nov. 2020 a las 17:45

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.