PHP array values and strpos function

Question

I have been using this piece of code to compare two values, I am getting the matched items exactly, The only problem is with non matched items.

$filter1 = "red,green,blue,yellow";         
$parts1 = explode(',', $filter1);

$filter2 = "red,green,blue";        
$parts2 = explode(',', $filter2);

for($i=0; $i< count($parts1); $i++)
{
  for($j=0; $j< count($parts2); $j++)
  {
    if(strpos($parts1[$i],$parts2[$j]) !== false)
    {
      $match[] = $parts1[$i];
    }
    else
    {
      $nomatch[] = $parts2[$j];
    }
  }
}

print_r($match);
echo "<br>";
print_r($nomatch);

And what i am getting as result is

Array ( [0] => red [1] => green [2] => blue ) 
Array ( [0] => green [1] => blue [2] => red [3] => blue [4] => red [5] => green [6] =>     
red [7] => green [8] => blue )

Array 1 is giving the exact matched values but array 2 is giving absurd results instead of yellow.

Have you tried to use array_diff() and array_intersect()? — complex857
– complex857, Commented Oct 2, 2012 at 12:57
I dont need to use array_diff or any other function because i dont need exact match i want containing items like animals and animal are same . so thats why i use strpos() function. — h_h
– h_h, Commented Oct 2, 2012 at 12:59
Then array_udiff and array_uintersect is the way to go (-: — complex857
– complex857, Commented Oct 2, 2012 at 13:04

PiTheNumber · Accepted Answer · 2012-10-02 13:43:06Z

2

Here you go:

for($i=0; $i< count($parts1); $i++)
{
  $matched = false;
  for($j=0; $j< count($parts2); $j++)
  {
    if(strpos($parts1[$i],$parts2[$j]) !== false)
    {
      $match[] = $parts1[$i];
      $matched = true;
      break;
    }
  }
  if (!$matched)
  {
    $nomatch[] = $parts1[$i];
  }
}

Demo: http://codepad.org/J6lmOUVO

edited Oct 2, 2012 at 13:43

answered Oct 2, 2012 at 13:00

PiTheNumber

23.7k17 gold badges114 silver badges191 bronze badges

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

1 Comment

abidibo Over a year ago

I'd add a break; after the line $matched = true, not essential but more powerful

Michael Berkowski · Accepted Answer · 2012-10-02 13:03:22Z

1

Because you are nesting loops, for each iteration of the outer loop you must iterate over every element of the inner loop. The non-matching extraneous values you see are all those extra inner iterations.

If you must allow partial matches with strpos(), use the following:

foreach ($parts1 as $p) {
  // Flag that the current value has been matched
  $matched = FALSE;
  foreach ($parts2 as $p2) {
    if (strpos($p, $p2) !== FALSE) {
      $matches[] = $p;
      $matched = TRUE;
    }
  }
  // If the loop was proceed with no match, add to non-matches
  if (!$matched) {
    $nomatch[] = $p;
  }
}

var_dump($matches);
array(3) {
  [0] =>
  string(3) "red"
  [1] =>
  string(5) "green"
  [2] =>
  string(4) "blue"
}
var_dump($nomatch);
array(1) {
  [0] =>
  string(6) "yellow"
}

edited Oct 2, 2012 at 13:03

answered Oct 2, 2012 at 12:58

Michael Berkowski

271k47 gold badges450 silver badges395 bronze badges

1 Comment

h_h Over a year ago

I explained above that i cant use array_dif or array intersect function because i dont need exact matches.

Collectives™ on Stack Overflow

PHP array values and strpos function

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related