Tu código se ve bastante bien, pero hay algunas sugerencias que podrían mejorar su estructura y claridad. Aquí van algunas mejoras:
- Es preferible usar un único mensaje de alerta para errores y personalizarlo según sea necesario. Esto facilita el mantenimiento y la consistencia.
- Utiliza constantes para mensajes que se utilizan varias veces. Esto facilita la gestión de cambios y la localización de errores.
- Usa convenciones de nombres, como camelCase para variables.
- Agrega comentarios para explicar el propósito de ciertas secciones de tu código.
También te recomiendo agregar validación para asegurarte de que la entrada del usuario sea un número válido para la edad.
Aquí te paso una versión de código con mis indicaciones que te puede servir:
// Mensajes
const welcomeMessage = "¡Bienvenida y bienvenido a nuestro sitio web!";
const errorMessage = "¡Error! Completa todos los campos";
// Mostrar mensajes
alert(welcomeMessage);
// Validación de campos
let nombre = prompt("¿Cuál es tu nombre de usuario?");
let edad = prompt("¿Cuál es tu edad?");
if (!nombre || !edad) {
// Validación de campos vacíos
alert(errorMessage);
} else {
// Mostrar información
console.log("Nombre de usuario:", nombre);
console.log("Edad:", edad);
// Validación de edad
if (isNaN(edad) || edad < 0) {
alert("La edad ingresada no es válida");
} else if (edad >= 18) {
alert("¡Puedes obtener tu licencia de conducir!");
} else {
alert("Necesitas ser mayor de edad");
}
}
Con estas modificaciones, el código es más claro y sigue buenas prácticas de programación. Además, se ha agregado una validación para asegurarse de que la edad ingresada sea un número válido. De mi parte un saludo y espero haberte ayudado!
