Using strpos() to match part of an array, PHP

Question

I need some help with strpos().

Need to build a way to match any URL that contains /apple-touch but also need to keep specifics matching, such as "/favicon.gif" etc

At the moment, the matches are listed out individually as part of an array:

<?php 

$errorurl = $_SERVER['REQUEST_URI'];
$blacklist = array("/favicon.gif", "/favicon.png", "/apple-touch-icon-precomposed.png", "/apple-touch-icon.png", "/apple-touch-icon-72x72-precomposed.png", "/apple-touch-icon-72x72.png", "/apple-touch-icon-114x114-precomposed.png", "/apple-touch-icon-114x114.png", "/apple-touch-icon-57x57-precomposed.png", "/apple-touch-icon-57x57.png", "/crossdomain.xml");

 if (in_array($errorurl, $blacklist)) { // do nothing }
    else { // send an email about error }

?>

Any ideas?

Many thanks for help

jeroen · Accepted Answer · 2012-10-11 02:30:16Z

1

Instead of a regex, you could also remove all occurrences of your blacklist items with str_replace and compare the new string to the old one:

if ( str_replace($blacklist, '', $errorurl) !== $errorurl )
{
  // do nothing
}
else
{
  // send an email about error
}

answered Oct 11, 2012 at 2:30

jeroen

91.8k22 gold badges117 silver badges132 bronze badges

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

Comments

SDC · Accepted Answer · 2012-10-17 13:33:40Z

0

If you want to use regex for this, and you want a single regex string that will capture all the values in your existing blacklist plus match any apple-touch string, then something like this would do it.

if(preg_match('/^\/(favicon|crossdomain|apple-touch.*)\.(gif|png|xml)$/',$_SERVER['REQUEST_URI']) {
    //matched the blacklist!
}

To be honest, though, that's far more complex than you need.

I'd say you'd be better off keeping the specific values like favicon.gif etc in the blacklist array you already have; it'd make it a lot easier when you come to adding more items to the list.

I'd only consider using regex for the apple-touch values, since you want to block any variant of them. But even with that, it would likely be simpler if you used strpos().

answered Oct 17, 2012 at 13:33

SDC

14.2k2 gold badges39 silver badges49 bronze badges

2 Comments

nooblag Over a year ago

Thanks for your suggestion. How to use strpos() then?

SDC Over a year ago

in fact, I meant substr() :) -- something like if(substr($uri,0,11)=='apple-touch') {....}

Collectives™ on Stack Overflow

Using strpos() to match part of an array, PHP

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related