regular expression goes into infinite loop
java, regex
Solution
To address the first part of your question, you should read up on catastrophic backtracking. Essentially, what is happening is there are too many ways to match your regular expression with your string, and the parser is continually back tracking to try and make it work.
In your case, it was probably the nested repitition: `(\s*[a-z]+)*` Which likely caused some very very strange loops. As Qtax has adeptly pointed out, it's hard to tell without more information.
The second part of your question is, unfortunately, impossible to answer. It's basically the Halting problem. Since Regular Expressions are essentially of a finite state machine whose input is a string, you cannot create a general solution which predicts which regular expressions will backtrack catastrophically, and which will not.
As far as some tips for making your regular expressions run faster? That's a big can of worms. I've spent a lot of time studying regular expressions on my own, and some time optimizing them, and here's what I've found generally helps:
- Compile your regular expressions outside of your loops, if your language supports it.
- Whenever possible, add anchors when you know they're useful. Especially the `^` for the beginning of the string. See also: Word Boundaries
- Avoid nested repetition like the plague. If you have to have it (which you will), do your best to provide hints to the engine to short circuit any unintended backtracking.
- Take advantage of flavor constructs to speed things up. I'm partial to Non-Capturing groups and possessive quantifiers. They don't appear in every flavor, but when they do, you should use them. Also check out Atomic Groups
- I always find this to be true: The longer your regular expression gets, The more trouble you're going to have making it efficient. Regular expressions are a great and powerful tool, they're like a super smart hammer. Don't fall into the trap of seeing everything as a nail. Sometimes the string function you're looking for is right under your nose.
Hope this helps you. Good luck.
Problem
I am parsing (species) names of the form: ``` Parus Ater H. sapiens T. rex Tyr. rex ``` which normally have two terms (binomial) but sometimes have 3 or more. ``` Troglodytes troglodytes troglodytes E. rubecula sensu stricto ``` I wrote ``` [A-Z][a-z]*\.?\s+[a-z][a-z]+(\s*[a-z]+)* ``` which worked most of the time but occasionally went into an infinite loop. It took some time to track down that it was in the regex matching and then I realised it was a typo and I should have written ``` [A-Z][a-z]*\.?\s+[a-z][a-z]+(\s+[a-z]+)* ``` which performs properly. My questions are: - why does this loop happen? - is there a way I can check for similar regex errors before running the program? Otherwise it may be difficult to trap them before the prgram is distributed and cause problems. [Note: I don't need a more general expression for species - there is a formal 100+ line regex specification for Species names - this was just an initial filter]. NOTE: The problem arose because although most names were extracted precisely into 2 or occasionally 3/4 terms (as they were in italics) there were a few false positives (like `"Homo sapiens lives in big cities like London"`) and the match fails at "L".] NOTE: In debugging this I have found that the regex was often completing but being very slow (e.g. on shorter target strings). It is valuable that I found this bug through a pathological case. I have learnt an important lesson!