Preg Match

[jaymc]

Can anyone tell me whats wrong with this, I dont think its formatted correct

[code]$patterns = "/\msn.com | \hotmail.com | \hotmail.co.uk/";
if (preg_match($patterns, $string)) {echo "yes";}[/code]

[roopurt18]

I think you need to use grouping:
[code]<?php
$patterns = "/(\\msn.com)|(\\hotmail.com(.uk)?)/";
if (preg_match($patterns, $string)) {echo "yes";}
?>[/code]

[HuggieBear]

I don't think so either.  Something like this for alternation...

[code]<?php
$pattern = "/(msn|hotmail)\.(com|co\.uk)/";
if (preg_match($pattern, $string)){
  echo "Yes";
}
?>[/code]

Although this will include msn.co.uk

Regards
Huggie

[jaymc]

[quote author=roopurt18 link=topic=119182.msg487787#msg487787 date=1166486062]
I think you need to use grouping:
[code]<?php
$patterns = "/(\\msn.com)|(\\hotmail.com(.uk)?)/";
if (preg_match($patterns, $string)) {echo "yes";}
?>[/code]
[/quote]

That works but it doesnt match hotmail.co.uk...

Huh?

And too the other guy, you cant return msn.co.uk as valid..

[jaymc]

Ive just used

$patterns = "/(\\@msn.com)|(\\@hotmail.com)|(\\@hotmail.co.uk)/";

Working fine

[HuggieBear]

In that case, I'd have thought that the following would be better.  It's certainly slightly shorter.

[code=php:0]$pattern = "/@((msn|hotmail)\.com|hotmail\.co\.uk)/";[/code]

Regards
Huggie