Jquery Value match Regex [duplicate]

Question

I am trying to validate the input for E-Mail via JQuery:

My JQuery

<script>
/* <![CDATA[ */
  jQuery(function(){
   $( ".mail" ).keyup(function() {
   var VAL = $(this).val();
   var email = new RegExp(^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$);

   if(VAL.test(email)){
   alert('Great, you entered an E-Mail-address');
   }
   });
  });
  /* ]]> */
  </script>

This won't alert even though I entered [email protected]. I already tried .test() and .match(), what did I do wrong?

H. Pauwelyn · Accepted Answer · 2016-07-29 08:09:15Z

109

Pass a string to RegExp or create a regex using the // syntax
Call regex.test(string), not string.test(regex)

So

jQuery(function () {
    $(".mail").keyup(function () {
        var VAL = this.value;

        var email = new RegExp('^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$');

        if (email.test(VAL)) {
            alert('Great, you entered an E-Mail-address');
        }
    });
});

edited Jul 29, 2016 at 8:09

H. Pauwelyn

14.4k28 gold badges93 silver badges164 bronze badges

answered Feb 12, 2014 at 12:09

Arun P Johny

389k68 gold badges532 silver badges532 bronze badges

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

8 Comments

James Donnelly Over a year ago

You can use this.value instead of $(this).val().

Mariano Over a year ago

Take into account this will not accept lowecase letters.

Muhammad Musavi Over a year ago

It's a case sensitive expression so [email protected] isn't accepted while [email protected] is approved. You can modify it to '^[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,4}$' to solve the problem but there is another issue, in this Regex abc@abcd without any dot is accepted! I suggest this one instead: /^\w+@[a-zA-Z_]+?\.[a-zA-Z]{2,3}$/g but none of these expressions accept an email address when there is a hyphen in domain name like: [email protected], visit this page for the best Regex: link

CertainPerformance Over a year ago

When constructing a RE with new RegExp, if you pass a sting, you'll need to double-escape backslashes. Eg you have \. in your code, but that's just an unnecessary escape character - '\.' === '.'. Best to avoid new RegExp unless you need to dynamically construct a RE from a variable.

James Donnelly Over a year ago

@leo848 $(this).val() just wraps this.value - it's one extra function call to get to the same piece of data, making it unnecessary.

|

user21926user21926 · Accepted Answer · 2014-02-12 12:06:49Z

14

Change it to this:

var email = /^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$/i;

This is a regular expression literal that is passed the i flag which means to be case insensitive.

Keep in mind that email address validation is hard (there is a 4 or 5 page regular expression at the end of Mastering Regular Expressions demonstrating this) and your expression certainly will not capture all valid e-mail addresses.

answered Feb 12, 2014 at 12:06

user21926

3 Comments

JAYD3V Over a year ago

var email = new RegExp('^[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}$', 'i');

JAYD3V Over a year ago

I prefer to change it like this

user21926 Over a year ago

I don't see anything different other than that you are using a string instead of the (arguably preferred) regular expression literal syntax?

Collectives™ on Stack Overflow

Jquery Value match Regex [duplicate]

2 Answers 2

8 Comments

3 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

3 Comments

Linked

Related