Use only preg_replace() or combine preg_match() and preg_replace()?

Question

I'm need to replace string with some pattern in array if needed pattern exists:

$patterns = [
   'PATTERN#1' => 'REPLACE#1',
   'PATTERN#2' => 'REPLACE#2',
];

$string = 'SOME STRING TO PREG_REPLACE';

Which way will be faster:

// PREG_REPLACE ONLY FOR MATCHED PATTERN:
foreach ($patterns as $pattern => $replace) {
    if (preg_match($string, $pattern)) {
        preg_replace($pattern, $replace, $string);
        break;
    }
}

or

// PREG_REPLACE FOR ALL PATTERNS:
foreach ($patterns as $pattern => $replace) {
    preg_replace($pattern, $replace, $string);
    break;
}

Only replace will be faster - it doesn't need to match twice. Have you tested it? Is the simpler solution too slow? — Bergi
– Bergi, Commented Mar 31, 2013 at 15:04

Explosion Pills · Accepted Answer · 2013-03-31 15:19:29Z

4

preg_replace takes array arguments for pattern and replacement.

preg_replace(array_keys($patters), array_values($patterns), $string);

answered Mar 31, 2013 at 15:19

Explosion Pills

192k56 gold badges341 silver badges417 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Use only preg_replace() or combine preg_match() and preg_replace()?

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related