Find a Quick Match Email Regular Expression

Question

solution:[a-zA-Z0-9.!#$%&'*+-/=?\^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)* is a good choice

I am using a regular expression like the below to match email addresses in a file:

email = re.search('(\w+-*[.|\w]*)*@(\w+[.])*\w+',line)

When used on a file like the following, my regular expression works well:

[email protected]    huofenggib  wrong in get_gsid
[email protected]   rouni816161 wrong in get_gsid

But when I use it on a file like below, my regular expression runs unacceptably slowly:

9b871484d3af90c89f375e3f3fb47c41e9ff22  [email protected]
e9b845f2fd3b49d4de775cb87bcf29cc40b72529e   [email protected]

And when I use the regular expression from this website, it still runs very slowly.

I need a solution and want to know what's wrong.

Define "slow". And are you certain it's the regular expression that is the bottleneck? — Martijn Pieters, Jun 15 '12 at 17:00
it's my code http://pastebin.com/6vUBxLZV you can try it,when it runs,it stagnate in regular expression match. — kuafu, Jun 15 '12 at 17:10

score 1 · Answer 1 · answered Jun 15 '12 at 17:08

That's a problem with backtracking. Read this article for more information.

You might want to split the line and work with the part containing an @:

pattern = '(\w+-*[.|\w]*)*@(\w+[.])*\w+'
line = '9b871484d3af90c89f375e3f3fb47c41e9ff22  [email protected]'
for element in line.split():
    if '@' in element:
        g = re.match(pattern, element)
        print g.groups()

score 0 · Accepted Answer · answered Jun 15 '12 at 17:09

Generally when regular expressions are slow, it is due to catastrophic bactracking. This can happen in your regex because of the nested repetition during in the following section:

(\w+-*[.|\w]*)*

If you can work on this section of the regex to remove the repetition from within the parentheses you should see a substantial speed increase.

However, you are probably better of just searching for an email regex and seeing how other people have approached this problem.

score 0 · Answer 3 · edited May 23 '17 at 12:07

0

It's always a good idea to search StackOverflow to see if your question has already been discussed.

Using a regular expression to validate an email address

This one, from that discussion, looks like a good one to me:

[a-zA-Z0-9.!#$%&'*+-/=?\^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*

edited May 23 '17 at 12:07

Community

1
1

answered Jun 15 '12 at 17:13

steveha

74,789
21
92
117

yes, your works,and mine maybe like other said is with the catastrophic bactracking – kuafu Jun 16 '12 at 00:43

Find a Quick Match Email Regular Expression

3 Answers3