799

My regex pattern looks something like

<xxxx location="file path/level1/level2" xxxx some="xxx">

I am only interested in the part in quotes assigned to location. Shouldn't it be as easy as below without the greedy switch?

/.*location="(.*)".*/

Does not seem to work.

random
  • 9,774
  • 10
  • 66
  • 83
publicRavi
  • 2,657
  • 8
  • 28
  • 34

9 Answers9

1579

You need to make your regular expression lazy/non-greedy, because by default, "(.*)" will match all of "file path/level1/level2" xxx some="xxx".

Instead you can make your dot-star non-greedy, which will make it match as few characters as possible:

/location="(.*?)"/

Adding a ? on a quantifier (?, * or +) makes it non-greedy.

Note: this is only available in regex engines which implement the Perl 5 extensions (Java, Ruby, Python, etc) but not in "traditional" regex engines (including Awk, sed, grep without -P, etc.).

Daniel Vandersluis
  • 91,582
  • 23
  • 169
  • 153
88

location="(.*)" will match from the " after location= until the " after some="xxx unless you make it non-greedy.

So you either need .*? (i.e. make it non-greedy by adding ?) or better replace .* with [^"]*.

  • [^"] Matches any character except for a " <quotation-mark>
  • More generic: [^abc] - Matches any character except for an a, b or c
hakre
  • 193,403
  • 52
  • 435
  • 836
sepp2k
  • 363,768
  • 54
  • 674
  • 675
  • 8
    [^"]* is also probably faster with most regex engines because it does not need to lookup the pattern after the current pattern. – Jean Vincent Jul 21 '12 at 10:34
  • 2
    @Kip: You're probably right, but the `.*?` notation is more general than `[^"]*` – Bondax Sep 02 '15 at 07:45
  • how about if I want to include the delimiter character using [^"]* – Frohlich Nov 28 '16 at 11:46
  • 1
    not at all, if you don't know what ^ and [ ] mean here. Most people will understand .* – Vincent Gerris Jun 04 '19 at 14:37
  • Another benefit of this approach is that it supports matching over line-breaks which the the dot "." does not support unless PCRE_DOTALL / DOT_MATCH_ALL or similar flags/modifiers. – hakre Oct 27 '21 at 16:11
45

How about

.*location="([^"]*)".*

This avoids the unlimited search with .* and will match exactly to the first quote.

One Man Crew
  • 9,420
  • 2
  • 42
  • 51
user193690
  • 1
  • 2
  • 2
  • 1
    Due to [discrepancies in grep](https://stackoverflow.com/questions/23454172/non-greedy-matching-with-grep) the above should be the preferred pattern if portability is a concern. – vhs Aug 13 '18 at 05:44
37

Use non-greedy matching, if your engine supports it. Add the ? inside the capture.

/location="(.*?)"/
codenheim
  • 20,467
  • 1
  • 59
  • 80
21

Use of Lazy quantifiers ? with no global flag is the answer.

Eg,

enter image description here

If you had global flag /g then, it would have matched all the lowest length matches as below. enter image description here

Uddhav P. Gautam
  • 7,362
  • 3
  • 47
  • 64
4

The other answers here fail to spell out a full solution for regex versions which don't support non-greedy matching. The greedy quantifiers (.*?, .+? etc) are a Perl 5 extension which isn't supported in traditional regular expressions.

If your stopping condition is a single character, the solution is easy; instead of

a(.*?)b

you can match

a[^ab]*b

i.e specify a character class which excludes the starting and ending delimiiters.

In the more general case, you can painstakingly construct an expression like

start(|[^e]|e(|[^n]|n(|[^d])))end

to capture a match between start and the first occurrence of end. Notice how the subexpression with nested parentheses spells out a number of alternatives which between them allow e only if it isn't followed by nd and so forth, and also take care to cover the empty string as one alternative which doesn't match whatever is disallowed at that particular point.

Of course, the correct approach in most cases is to use a proper parser for the format you are trying to parse, but sometimes, maybe one isn't available, or maybe the specialized tool you are using is insisting on a regular expression and nothing else.

tripleee
  • 175,061
  • 34
  • 275
  • 318
3

Here's another way.

Here's the one you want. This is lazy [\s\S]*?

The first item: [\s\S]*?(?:location="[^"]*")[\s\S]* Replace with: $1

Explaination: https://regex101.com/r/ZcqcUm/2


For completeness, this gets the last one. This is greedy [\s\S]*

The last item:[\s\S]*(?:location="([^"]*)")[\s\S]* Replace with: $1

Explaination: https://regex101.com/r/LXSPDp/3


There's only 1 difference between these two regular expressions and that is the ?

Ste
  • 1,729
  • 1
  • 17
  • 27
2

Because you are using quantified subpattern and as descried in Perl Doc,

By default, a quantified subpattern is "greedy", that is, it will match as many times as possible (given a particular starting location) while still allowing the rest of the pattern to match. If you want it to match the minimum number of times possible, follow the quantifier with a "?" . Note that the meanings don't change, just the "greediness":

*?        //Match 0 or more times, not greedily (minimum matches)
+?        //Match 1 or more times, not greedily

Thus, to allow your quantified pattern to make minimum match, follow it by ? :

/location="(.*?)"/
Mohammad Kanan
  • 4,452
  • 10
  • 23
  • 47
0
import regex
text = 'ask her to call Mary back when she comes back'                           
p = r'(?i)(?s)call(.*?)back'
for match in regex.finditer(p, str(text)):
    print (match.group(1))

Output: Mary