alternative to if(preg_match() and preg_match())

Question

I want to know if we can replace if(preg_match('/boo/', $anything) and preg_match('/poo/', $anything))
with a regex..

$anything = 'I contain both boo and poo!!';

for example..

To tell you truth, it's already a regex – Your Common Sense Apr 29 '13 at 14:15 — Your Common Sense, Apr 29 '13 at 14:15
@YourCommonSense I meant only one regex – iguider Apr 29 '13 at 14:47 — iguider, Apr 29 '13 at 14:47

Pudge601 · Accepted Answer · 2013-04-29T14:21:18.233

From what I understand of your question, you're looking for a way to check if BOTH 'poo' and 'boo' exist within a string using only one regex. I can't think of a more elegant way than this;

preg_match('/(boo.*poo)|(poo.*boo)/', $anything);

This is the only way I can think of to ensure both patterns exists within a string disregarding order. Of course, if you knew they were always supposed to be in the same order, that would make it more simple =]

EDIT After reading through the post linked to by MisterJ in his answer, it would seem that a more simple regex could be;

preg_match('/(?=.*boo)(?=.*poo)/', $anything);

score 2 · Answer 2 · answered Apr 29 '13 at 14:04

2

By using a pipe:

if(preg_match('/boo|poo/', $anything))

answered Apr 29 '13 at 14:04

sroes

14,663
1
53
72

1

it is insufficient for replacement – Your Common Sense Apr 29 '13 at 14:11
1

I think this replaces: if(preg_match('/boo/', $anything) **or** preg_match('/poo/', $anything)) – iguider Apr 29 '13 at 17:47

score 1 · Answer 3 · edited May 23 '17 at 11:49

You can use the logical or as mentioned by @sroes:

if(preg_match('/(boo)|(poo)/,$anything)) the problem there is that you don't know which one matched.

In this one, you will match "I contain boo","I contain poo" and "I contain boo and poo". If you want to only match "I contain boo and poo", the problem is really harder to figure out Regular Expressions: Is there an AND operator? and it seems that you will have to stick with the php test.

Ben Carey · Answer 4 · 2013-04-29T14:16:51.850

You can achieve this by altering your regular expression, as others have pointed out in other answers. However, if you want to use an array instead, so you do not have to list a long regex pattern, then use something like this:

// Default matches to false
$matches = false;

// Set the pattern array
$pattern_array = array('boo','poo');

// Loop through the patterns to match
foreach($pattern_array as $pattern){
    // Test if the string is matched
    if(preg_match('/'.$pattern.'/', $anything)){
        // Set matches to true
        $matches = true;
    }
}

// Proceed if matches is true
if($matches){
    // Do your stuff here
}

Alternatively, if you are only trying to match strings then it would be much more efficient if you were to use strpos like so:

// Default matches to false
$matches = false;

// Set the strings to match
$strings_to_match = array('boo','poo');

foreach($strings_to_match as $string){
    if(strpos($anything, $string) !== false)){
        // Set matches to true
        $matches = true;
    }
}

Try to avoid regular expressions where possible as they are a lot less efficient!

it's not me who's downvoting anyone of you.. :/ – iguider Apr 29 '13 at 14:43 — iguider, Apr 29 '13 at 14:43
@IderAghbal No probs, glad I could help :-) – Ben Carey Apr 29 '13 at 18:05 — Ben Carey, Apr 29 '13 at 18:05

score 0 · Answer 5 · answered Apr 29 '13 at 14:14

0

To take conditions literally

if(preg_match('/[bp]oo.*[bp]oo/', $anything))

answered Apr 29 '13 at 14:14

Your Common Sense

156,878
40
214
345

alternative to if(preg_match() and preg_match())

5 Answers5