Error with parser recognising regex

Question

I am trying to add a bit of JavaScript validation to my application to check that an email address is formatted correctly.

This is the code that I am using to perform the validation checks:

var bValid = true;
allFields.removeClass( "ui-state-error" );
bValid = bValid && checkLength( email, "email", 6, 80 );
bValid = bValid && checkLength( password, "password", 5, 16 );
bValid = bValid && checkRegexp( name, /^[a-z]([0-9a-z_])+$/i, "Username may consist of a-z, 0-9, underscores, begin with a letter." );
// From jquery.validate.js (by joern), contributed by Scott Gonzalez: http://projects.scottsplayground.com/email_address_validation/
var reg = /^(([^<>()[]\.,;:s@"]+(.[^<>()[]\.,;:s@"]+)*)|(".+"))@(([[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}])|(([a-zA-Z-0-9]+.)+[a-zA-Z]{2,}))$/;
bValid = bValid && checkRegexp(email, reg, "eg. smith@hotmail.com");
bValid = bValid && checkRegexp( password, /^([0-9a-zA-Z])+$/, "Password field only allow : a-z 0-9" );
if ( bValid ) {
    //DO ajax call
    $( this ).dialog( "close" );
}

And I am getting this error:

enter image description here

I have tried numerous examples of regex expressions but all get caught by the parser.

@isi, its JavaScript, not C#. So the `/.../` literals are fine (placing an `@` in front of it will result in more errors): the problem is most probably as M42 indicated. — Bart Kiers, Jun 17 '14 at 11:41
What I don't understand is why is the server giving an error for your JavaScript code? Isn't JavaScript supposed to run on the client browser? — Chirag Bhatia - chirag64, Jun 17 '14 at 11:57

Toto · Answer 1 · 2014-06-17T11:58:52.687

2

Escape the ] inside character class:

var reg = /^(([^<>()[\]\.,;:s@"]+(.[^<>()[\]\.,;:s@"]+)*)|(".+"))@(([[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}])|(([a-zA-Z-0-9]+.)+[a-zA-Z]{2,}))$/;
//            here __^             here __^

Not sure but javascript may want to escape also [ :

var reg = /^(([^<>()\[\]\.,;:s@"]+(.[^<>()\[\]\.,;:s@"]+)*)|(".+"))@(([[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}.[0-9]{1,3}])|(([a-zA-Z-0-9]+.)+[a-zA-Z]{2,}))$/;
//           here __^              here __^

edited Jun 17 '14 at 11:58

answered Jun 17 '14 at 11:32

Toto

89,455
62
89
125

i think escaping is just another issue, but the first issue is according missing quotes for the regex string, or is the "/" a syntactical shorthand in C# to indicate a regex? – isaias-b Jun 17 '14 at 11:37
2

@isi: Not for javascript. – Toto Jun 17 '14 at 11:39
tried to add escapes where you mentioned but got the same error? – brian4342 Jun 17 '14 at 11:45
@M42 any other ideas? – brian4342 Jun 17 '14 at 11:56
Thanks, Though I am getting a bit confused with what needs escaping, could you add the escape characters in your example? – brian4342 Jun 17 '14 at 12:01
@BrianPeach: Look at http://stackoverflow.com/questions/399078/what-special-characters-must-be-escaped-in-regular-expressions – Toto Jun 17 '14 at 13:18

score 0 · Answer 2 · edited May 23 '17 at 10:33

Based on what Chirag64 wrote, it looks like the server might be mistakenly running your JavaScript code? Line 104 is perhaps just the first line where the compiler/interpreter has an error due to it being the wrong language? What programming language is your server expecting?

You should try copying and pasting the problem code into a separate HTML file and see if it will run there when you load it directly into a browser (no server). I suspect that it will.

Error with parser recognising regex

2 Answers2