0

Estoy trantando de validar un correo en un input en html usando regexp, no logro que me lo valide correctamente que fallo podría estar realizando ? Cuando quito el foco siempre me sale email incorrecto aunque coloque bien el patrón.

document.getElementById('email').addEventListener('focusout', function () {
   
  var regExpEmail = new RegExp('^w+@[a-zA-Z_]+?.[a-zA-Z]{2,3}$');
  if (regExpEmail === true) {
    console.log('Email correcto');
  } else {
  console.log('Email incorrecto');
  }

7
  • qué error te da?, qué email te falla? has probado regexr.com?
    – Iria
    Commented el 19 dic. 2020 a las 14:23
  • el if /else, creo que no coge el if y directamente sale el else todo el rato como si el if no funcionara correctamente. Commented el 19 dic. 2020 a las 14:25
  • a bote pronto, veo un fallo, pero igual no es lo que tu tienes en mente... hay correos electrónicos que terminan en .co.uk (emails de Reino Unido). Según veo tu regular expression no pasarían
    – Iria
    Commented el 19 dic. 2020 a las 14:25
  • estoy probando el típico ejemplo de [email protected], no necesito nada mas complicado Commented el 19 dic. 2020 a las 14:26
  • 3
    No reinventes la rueda, con emailregex tienes lo que buscas. A veces es mejor usar las herramientas ya desarrolladas por expertos, y en el campo de las expresiones regulares hay bastantes expertos y muchos coinciden que el patrón que se muestra en esa web es 99.99% funcional. Saludos Commented el 19 dic. 2020 a las 15:07

4 respuestas 4

2

Aunque en las respuestas ya se solucionen el problema, quiero dar mi aporte observando tu codigo:

  1. Esta mal escrito, porque falta al final cerrar })
  2. La exprecion regular que estas usando tambien esta mal porque valida incorrectamente un correo segun regexr
  3. RegExp no devuelve un booleano porque es una funcion constructor RegExp
  4. Para validar un RegExp se utiliza el test() donde esta ultima si devuelve un booleano

Como desconozco aun usar expreciones regulares, usare un ejemplo para validar un correo usando esto validar-correo-usando-exprecion-regular

document.getElementById("email").addEventListener("focusout", (e) => {
  var regExpEmail = /^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,4})+$/.test(e.target.value);
  
  if (regExpEmail) {
    console.log("Email correcto");
  } else {
    console.log("email incorrecto");
  }
});
<input type="email" id="email" />

1
  • Muchas gracias!!!!!! ha sido el unico que me ha logrado funcionar :) Commented el 19 dic. 2020 a las 16:41
1

Estas comparando la expresion regular con un true, eso no funciona, debes comparar el valor del input con la expresion regular usando test.

 <input type="email" id="email" />
        <script>
          document.getElementById("email").addEventListener("focusout", (e) => {
            var regExpEmail = new RegExp("^w+@[a-zA-Z_]+?.[a-zA-Z]{2,3}$");
            console.log(e.target.value); // Valor actual del input
            if (regExpEmail.test(e.target.value)) {
              console.log("Email correcto");
            } else {
              console.log("email incorrecto");
            }
          });
        </script>
2
  • nose si he hecho algo diferente pero ahora ya funciona correctamente, lo que si coloco incorrectamente el correo funciona igualmente y no sale email incorrecto Commented el 19 dic. 2020 a las 14:29
  • Lo siento, modifico la respuesta, la forma correcta no era con search, sino con test, expresionRegular.test(valor)
    – fabran99
    Commented el 19 dic. 2020 a las 14:33
1

Según https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp estás creando un objeto RegExp, no un boolean, cuando compruebas (https://www.w3schools.com/js/js_comparisons.asp), estás igualando RegExp a true, en valor y tipo cosa que no funciona, porque true es boolean. Lo que tienes que hacer es comparar la expresion regular con el email y si es identico o no ir a un lado u otro. Tienes ejemplos de lo que quieres hacer en https://stackoverflow.com/questions/6603015/check-whether-a-string-matches-a-regex-in-js

0

Como bien responde Iria, estas comparado regExpEmail === true lo que siempre sera falso, ya que al usar === estas comparando un dato varchar con uno boolean.

De todas formas no estas comparando ningun email con la expresion regular. La expresion regular ^w+@[a-zA-Z_]+?.[a-zA-Z]{2,3}$ segun reger.com no valida [email protected]. He usado esta expresion regular, y si que funciona como creo que esperas ^[a-z0-9._%+-]+@[a-z0-9.-]+\.[a-z]{2,}$

   
  var regExpEmail = new RegExp('^[a-z0-9._%+-]+@[a-z0-9.-]+\.[a-z]{2,}$');
  var email = "[email protected]";
  if (regExpEmail.test(email)) {
    console.log('Email correcto');
  } else {
  console.log('Email incorrecto');
  }

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.