Why does this regex match too much? (Doesn't stop at slash) [duplicate]

Answer 1

The dot is greedy and matches as many characters as possible. Either make it lazy:

$pattern = str_replace(':any', '(.+?)', $pattern);

or keep it from matching slashes:

$pattern = str_replace(':any', '([^\/]+)', $pattern);

Answer 2

Use a non-greedy modifier to make the + match as few characters as possible instead of as many as possible:

$pattern = str_replace(':any', '(.+?)', $pattern);
                                   ^

You probably also want to add delimiters round your regular expression and anchor it to the start of the string:

$pattern = '#^/foo/:any/#';

Answer 3

Your code is rather confusing and misleading and if run it, it outputs a warning:

Warning: preg_match(): Unknown modifier '(' in php shell code on line 1

What I think is wrong is:

$pattern = '/foo/:any/';
#should be
$pattern = '/foo\/:any/';

because you need to escape a forward slash in regexp.

After this is fixed the script returns:

(
  [0] => foo/bar/this/that
  [1] => bar/this/that
)

Which is an expected result. As you match foo/ and everything afterwards with (.*). If you want to match anything until the next forward slash you have some possibilities:

$pattern = '/foo/(.*?)/'     #non greedy
$pattern = '/foo/([^\/]*)/'  #not matching any forward slash
$pattern = '@foo/:any/@'     #or using different start and end markers, e.g. @