Regex test early on, remove redundant check

[xoliver]

Hi! I noticed that the regex already enforces that there should only be an @ between two lumps of text, so an early check with the regex will:

• Save a redundant check
• Speed up the process by saving the following checks

[houd1ni]

Hi! Thank you for the idea! But is it really more performant to run the big regex than to create an array of two strings? Any tests ?

[xoliver]

Hi @houd1ni . Thanks for your comment. This is my thinking:

• For valid email addresses, all the checks need to happen. In that respect, my proposal spares an if so it should be the same performance or very marginally better
• For invalid email addresses in cases that are specifically sought, such as missing "@" or components that are too long, the old way should be faster
• For everything else (trailing spaces, invalid chars, etc), my suggestion will be faster as the regex will discard them upfront

I'm not a JS benchmarking expert (or JS experienced!) but I found this out and put out an example here, based on the input used in the unit tests: https://www.measurethat.net/Benchmarks/Show/27639/0/email-validator-pr-80-benchmark

My proposal seems to be 50% faster