2

En este ejercicio de Javascript trato de validar la edades y hay algo que falla y es que el condicional no se comporta como debería según el rango de edad... ¿alguna sugerencia? ¿estará mal planteada la lógica? ¿Debería usar else if en vez de solo if?

me refiero a que si el usuario ingresa por ejemplo 20 años de edad, salta el mensaje "necesitas autorización de un adulto", por ejemplo.

let edad = prompt("Ingresar edad");
    
    if (edad <= 6) {
        alert("No puede ingresar a la sala")
    }
    if (edad <= 13) {
        alert("Necesitas ingresar con un adulto/a");
    }
    if (edad >13){
        alert("Necesitas autorizacion de un adulto/a");
    }
    if (edad >= 18) {
        alert("Bienvenido al club")
    }

6
  • 2
    ¿Cuá es el comportamiento esperado entonces?
    – Excorpion
    Commented el 30 dic. 2021 a las 17:58
  • 1
    Ten en consideración que el valor ingresado en el prompt es un string. Convierte la variable edad a número y vuelve a probar Commented el 30 dic. 2021 a las 17:59
  • 2
    @PauloUrbanoRivera estoy probando el codigo, y veo que funciona como se espera, lo que no queda claro, es el problema que el menciona. Dice que no se comporta, pero no menciona un caso de error.
    – Excorpion
    Commented el 30 dic. 2021 a las 18:00
  • 1
    me refiero a que si el usuario ingresa por ejemplo 20 años de edad, salta el mensaje "necesitas autorización de un adulto", por ejemplo Commented el 30 dic. 2021 a las 18:12
  • 1
    Tu condicional esta mal entonces... if (edad >13) Si la edad es mayor a 13... 20 es mayor a trece, si, salta... agregale otra condicion... edad > 13 && edad < 18
    – Excorpion
    Commented el 30 dic. 2021 a las 18:14

3 respuestas 3

3

Basado en el comentario provisto por el OP

me refiero a que si el usuario ingresa por ejemplo 20 años de edad, salta el mensaje "necesitas autorización de un adulto", por ejemplo

Para corregir el mal comportamiento, es solo corregir el 3er IF.
El cual solo consideraba un edad > 13 siendo 20 mayor a 13... faltaba un limite superior, el cual es edad < 18

let edad = prompt("Ingresar edad");

if (edad <= 6) {
  alert("No puede ingresar a la sala")
}
if (edad <= 13) {
  alert("Necesitas ingresar con un adulto/a");
}
if (edad > 13 && edad < 18) {
  alert("Necesitas autorizacion de un adulto/a");
}
if (edad >= 18) {
  alert("Bienvenido al club")
}

2
  • Si ingresas un número menor o igual que 6, aparecerán dos alertas. Habrá que esperar a ver si esto es lo que quiere OP.
    – Triby
    Commented el 30 dic. 2021 a las 19:03
  • En ese caso la corrección es similar, pero una vez preguntado el error de caso, el que menciona es el ya dicho, si el tiene o no otro comportamiento no esperado, es el quien lo debe mencionar, por que ya fue consultado 1 vez.
    – Excorpion
    Commented el 30 dic. 2021 a las 19:06
1

Esta respuesta está basada en la idea de que solo espera mostrar un mensaje en cada rango de edades, por las respuestas que ha dado entiendo eso, pero podría no ser lo esperado

Yo hubiera utilizado else-if. Te evita el rango superior. Si una no es válida, pasa a la siguiente ya con esa condición superada. Además, en cuanto una es válida deja de comprobar todo el resto de condiciones, mejorando ligeramente la eficiencia.

Otra cosa que mejora es que consigues que no muestre más de un mensaje. En el ejemplo, con un valor de 6 tu código te mostraría los dos primeros mensajes, ya que 6 es menor que 6 pero también menor que 13. No sé si tiene sentido eso en tu programa, habrá veces en que sí.

Por último, yo metería un casteo a número (con poner un + delante de la expresión te lo hace) por dejarlo más bonito. Sé que puede parecer que con que JS lo haga por detrás en cada comparación es suficiente, pero siempre es mejor tener el control de tu programa.

Ah, y usa const si no vas a cambiar el valor de una variable.

const edad = +prompt("Ingresar edad");
    
if (edad <= 6) {
    alert("No puede ingresar a la sala")
} else if (edad <= 13) {
    alert("Necesitas ingresar con un adulto/a");
} else if (edad <= 17){
    alert("Necesitas autorizacion de un adulto/a");
} else {
    alert("Bienvenido al club")
}

1

Deberias de utilizar if/elseif para que la segunda y tercera condicion no se cumplan:

  let edad = prompt("Ingresar edad");
    
    if (edad <= 6) {
        alert("No puede ingresar a la sala")
    }
    else if (edad <= 13) {
        alert("Necesitas ingresar con un adulto/a");
    }
    else if (edad < 18){
        alert("Necesitas autorizacion de un adulto/a");
    }
    else if (edad >= 18) {
        alert("Bienvenido al club")
    }

7
  • Esta respuesta supone un resultado y no sabemos cuál será ese resultado esperado
    – Alfabravo
    Commented el 30 dic. 2021 a las 18:01
  • @Alfabravo tiraste -1 al post y la respuesta de una ? sin esperar un comentario al respecto ?
    – Excorpion
    Commented el 30 dic. 2021 a las 18:02
  • @Alfabravo no hay que ser tan metodico para entender lo que intenta hacer. Claramente hay un problema de logica donde si se cumple la segunda igual la tercera. Siquiera probaste el codigo?
    – Einer
    Commented el 30 dic. 2021 a las 18:02
  • Tan lo probé que cambié el bloque en la pregunta por un snippet. Pero no, el resultado esperado no está en la pregunta.
    – Alfabravo
    Commented el 30 dic. 2021 a las 18:06
  • @Alfabravo VUELVO Y PREGUNTO: Realmente se necesita ser tan dotado para entender lo que el OP no logra hacer? Entiendo lo que quieres decir y veo que quieres que se cumplan las reglas pero ya a los niveles que lo quieres llevar son exagerados.
    – Einer
    Commented el 30 dic. 2021 a las 18:09

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.