jQuery email validation regexp problems

Question

I'm currently working on my own validation, but I have one small problem. I can't get email validation to work. I have tried some different expressions, but can't get it to work. Maybe i did something wrong with the layout?

var email = $('.email');    
function valemail(){
    var filter = /^([\w-\.]+)@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.)|(([\w-]+\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})(\]?)$/;
    if (filter.test(email)){
        return true;
    } else {
        return false;
    }
}

$('#registerform').submit(function() {
    if (valemail()) {
        return true;
    } else {
        return false;
    }
});

Don't do regex email validation beyond a very basic /[^@\s]+@[^@\s]+\.[^@\s]+/ check. A more complex regex will either reject valid addresses or be so complex that it is unmaintainable, both not useful. Keep it simple. Check mail addresses by trying to send actual mail there. — Tomalak
– Tomalak, Commented Feb 16, 2013 at 15:56
I would not write my own validation script when this plugin is available. However, simply take a look at the email regex contained within it. — Sparky
– Sparky, Commented Feb 16, 2013 at 16:14

score 2 · Accepted Answer · 2013-02-16 15:56:56Z

2

Try this :

function IsEmail(email) {
  var regex = /^([a-zA-Z0-9_\.\-\+])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/;
  return regex.test(email);
}

The the following calls:

   $('#registerform').submit(function(){
      if(IsEmail($(this).val())==false){
          //do something validation not satisfied
        }else{
            //do something
        }
      });

edited Feb 16, 2013 at 15:56

answered Feb 16, 2013 at 15:51

user2004812

Sign up to request clarification or add additional context in comments.

Comments

Sparky · Accepted Answer · 2013-02-16 16:31:11Z

I would not write my own validation script when the jQuery Validate plugin is available.

However, you could simply take a look at the email regex contained within it and use that...

function email(value) {
    // contributed by Scott Gonzalez: http://projects.scottsplayground.com/email_address_validation/
    return /^((([a-z]|\d|[!#\$%&'\*\+\-\/=\?\^_`{\|}~]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])+(\.([a-z]|\d|[!#\$%&'\*\+\-\/=\?\^_`{\|}~]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])+)*)|((\x22)((((\x20|\x09)*(\x0d\x0a))?(\x20|\x09)+)?(([\x01-\x08\x0b\x0c\x0e-\x1f\x7f]|\x21|[\x23-\x5b]|[\x5d-\x7e]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(\\([\x01-\x09\x0b\x0c\x0d-\x7f]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF]))))*(((\x20|\x09)*(\x0d\x0a))?(\x20|\x09)+)?(\x22)))@((([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])([a-z]|\d|-|\.|_|~|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])*([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])))\.)+(([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])([a-z]|\d|-|\.|_|~|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])*([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])))$/i.test(value);
}

$('#registerform').submit(function() {
    var result = email($('#myemailfield').val());
    if (result) {
        alert('passed'); // <-- your code if passed
    } else {
        alert('failed'); // <-- your code if failed
    }
});

Simple Demo: http://jsfiddle.net/m8baZ/

jfriend00 · Accepted Answer · 2013-02-16 16:24:36Z

0

When you do this:

var email = $('.email');
if (filter.test(email)){

email is a jQuery object, not the string the user typed. You need to get the actual string out of the input field and use that with your regex like this:

if (filter.test(email.val())){

answered Feb 16, 2013 at 16:24

jfriend00

711k104 gold badges1.1k silver badges1k bronze badges

Collectives™ on Stack Overflow

jQuery email validation regexp problems

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related