met sneller en korter kun je twee verschillende dingen bedoelen.



om het sneller te maken:

* plaats achter de $item['title']=''; een return; de rest hoeft toch niet meer gedaan te worden.
*doe eerst een $x=$item['menu_1']; en ga dan los op de $x;
*doe eerst een strtolower op de $x en dan laat de i weg in de regex
*laat de .* weg ( heeft in deze regexen geen waarde)
*voeg de regexen samen

$x=strtolower($item['menu_1']);
if ( preg_match("/(baby|bed|bad)/",$x) ) {
$item['title']='';return;
}

dit is in een testje 30% sneller. maar zal sterk afhangen van het aantal items dat weggestreept gaat worden ( door de toegevoegd return). Als je de $x maar 1 keer gebruikt is dit ook niet meer efficiënter ( en maakt de strtolower ook nog maar weinig uit.

if ( preg_match("/(baby|bed|bad)/",strtolower($item['menu_1']);) ) {
$item['title']='';return;
}

dus strtolower en x=$item['menu_1'] helpen alleen als je de regexen 1 voor 1 doet, bijvoorbeeld omdat je ze niet eenvoudig kunt samenvoegen:

preg_match ("/^bed/"...
preg_match ("/bad$/"...

om het qua programmeerwerk eenvoudiger te maken kun je de waardes in een array zetten en ze dan 1 voor 1 doorlopen met een foreach en dan is strpos weer sneller dan preg_match.

$b=array('bed','bad','baby');
$x=strtolower($item['menu_1']);
foreach ( $b as $k=>$v ) {
if ( preg_match('/'.$v.'/',$x) ) {
#if ( strpos($x,$v) !== false ) {
$item['title']='';return;
}
}

of als je alleen 'eenvoudige' patronen hebt

$b=array('bed','bad','baby');
$c=join('|',$b);
function pixmania_cb(&$item) {
global $c;
$x=strtolower($item['menu_1']);
foreach ( $b as $k=>$v ) {
if ( preg_match('/('.$c.')/',$x) ) {
$item['title']='';return;
}
}