Se supone que la siguiente función verifica dos entradas: nombre y mensaje. No estoy seguro de qué está mal aquí, pero primero estoy tratando de ver si la entrada está vacía; si no es así, quiero verificarlo con una expresión regular. El mensaje solo quiero ver si está vacío o no. Si cualquiera de los dos está vacío, quiero devolver un mensaje solicitando ingresar la entrada correcta. No estoy seguro de si debería usar else ifs aquí o separar las declaraciones if por completo. Si estoy haciendo esto completamente loco, también me gustaría saberlo. ¡Gracias!
JavaScript:
function validate() { let regName = /^[a-zA-Z]+ [a-zA-Z]+$/; let name = document.querySelector("[name='user-name']"); let msg = document.querySelector("[name='user-message']"); if (name) { name = name.value; } else if (!regName.test(name)){ document.querySelector("[name='user-name']"); return "Please enter your first & last name."; } if (msg) { msg = msg.value; } else if (msg.trim() == "") { document.querySelector("[name='user-message']"); return "Please enter a message"; } }Su lógica if y else if es incorrecta. Solo está realizando la prueba de expresiones regulares cuando document.querySelector() no puede encontrar el elemento de entrada. else if solo se ejecuta cuando falla la condición anterior.
Debe poner ese cheque en el mismo if eso verifica que se encontró la entrada.
function validate() { let regName = /^[a-zA-Z]+ [a-zA-Z]+$/; let name = document.querySelector("[name='user-name']"); let msg = document.querySelector("[name='user-message']"); if (name && !regName.test(name.value)) { return "Please enter your first & last name."; } if (msg && msg.value.trim() == "") { return "Please enter a message"; } }