Otra cosa que podrías hacer es ahorrarte todas esas divisiones de validación por una sola y ordenar un poco tu código.
Código HTML:
Ver original <label>E-mail
</label><input name="strEmail" class="form-control" placeholder="e-mail"> <!-- Validacion -->
<div class="alert alert-danger oculto" id="error"></div>
Código Javascript
:
Ver original// JavaScript Document
function validarEmail( email ) {
expr = /^([a-zA-Z0-9_.-])+@(([a-zA-Z0-9-])+.)+([a-zA-Z0-9]{2,4})+$/;
if ( !expr.test(email) )
return false;
else
return true;
}
function validarusuarioalta() {
const email = $("input[name=strEmail]").val();
const pass = $("input[name=strPassword]").val();
const name = $("input[name=strNombre]").val();
// Ocultamos el div del error
$("#error").hide("slow");
//Si el campo de texto del e-mail no posee contenido
if ( !email ){
$( "#error" ).html( "Por favor introduce un correo electrónico." );
$( "#error" ).show( "slow" );
return false; // Detenemos la función en este punto
}
//Se evalúa su formato. Si no es válido, se muestra el mensaje de error respectivo
if ( !validarEmail( email )){
$( "#error" ).html( "El correo electrónico introducido no es correcto." );
$( "#error" ).show( "slow" );
return false; // Esto detiene la función en este punto
}
if ( !pass ){
$( "#error" ).html( "Por favor introduce una contraseña." );
$( "#error" ).show( "slow" );
return false; // Esto detiene la función en este punto
}
if ( !name ){
$( "#error" ).html( "Por favor introduce tu nombre." );
$("#error").show("slow");
return false; // Esto detiene la función en este punto
}
return true;
}