Regular expression not working

[tshad]

I have a regular expression that works fine for emails but it also allows angle brackets "<" and ">".

It crashes the page if there are angle brackets in the email.

My expression is: @"[A-Z0-9._%+-]+@[A-Z0-9.-]+\.[A-Z]{2,4}"

I would assume this would not work since I say only allow certain letters and none are angle brackets.

It allows "tom <tom@aol.com>" to go through fine.

Why does it work and what can I do to fix it?

Thanks,

Tom

 

[MakeItSo]

The cuplrit is the period in your second set: [A-Z0-9.-]+
. is a special character in Regex meaning "any character". You are looking for a literal period, hence escape it:
[A-Z0-9\.-]+

Does that do the trick?

Cheers,
MakeItSo

“Knowledge is power. Information is liberating. Education is the premise of progress, in every society, in every family.” (Kofi Annan)
Oppose SOPA, PIPA, ACTA; measures to curb freedom of information under whatever name whatsoever.

 

[tshad]

Didn't work.

I have it as:

private const string Regex_Email = @"[A-Z0-9\._%+-]+@[A-Z0-9\.-]+\.[A-Z]{2,4}";

and the code is:

reg = new Regex(Regex_Email, RegexOptions.IgnoreCase);
result = (reg.IsMatch(email));

and the result comes back as true.

 

[MakeItSo]

I think I know the problem:
<tom@aol.com> goes through because tom@aol.com is a match.

Maybe you should rethink your approach and first check for < > addresses, maybe replacing their "@" with [at]" before testing for mail addresses?

“Knowledge is power. Information is liberating. Education is the premise of progress, in every society, in every family.” (Kofi Annan)
Oppose SOPA, PIPA, ACTA; measures to curb freedom of information under whatever name whatsoever.

 

[johnny9864]

This is the best article you can find on this

http://www.regular-expressions.info/email.html

It even gives the "official" accepted regular expression