Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

PHP array_search consistently returns first key of array [duplicate]

I recently noticed trouble when using the array_search function in my code. I am searching the array "$allcraftatts" for the value "sharp". I tried to isolate the problem by setting up a two line experiment:

$testcopy=$allcraftatts;
$testsharp=array_search("sharp", $testcopy);

Using "print_r(get_defined_vars());" later on, I get this result:

[testcopy] => Array
                (
                    [0] => 0
                    [1] => 0
                    [2] => 0
                    [3] => 0
                    [4] => 0
                    [5] => 0
                    [6] => Sharp Stone
                    [7] => Sharp Stones
                    [8] => stone
                    [9] => object
                    [10] => sharp
                    [11] => hard
                    [12] => 0
                    [13] => 0
                    [14] => 0
                    [15] => 0
                    [16] => 0
                    [17] => 0
                    [18] => 0
                )

[testsharp] => 0

I made sure that I do not modify these variables at any other time.

Now, if I change my code to

$testcopy=$allcraftatts;
unset($testcopy[0]);
$testsharp=array_search("sharp", $testcopy);

it returns "1".

This leads me to believe that it always returns the first key in the array.

It baffles me! This is one of those bugs that makes you fear something wrong with the language itself. However doubtful this is, I actually was eventually driven to looking at the PHP source for something wrong there, but unfortunately could not understand it.

Seeing that it is a function simple as this, I will most definitely be completely humiliated by the inevitably simple answer, but at this point, I just want an answer.

like image 724
Aaron David Fairbanks Avatar asked Nov 06 '12 21:11

Aaron David Fairbanks


2 Answers

array_search is using == to compare values during search

FORM PHP DOC

If the third parameter strict is set to TRUE then the array_search() function will search for identical elements in the haystack. This means it will also check the types of the needle in the haystack, and objects must be the same instance.

Becasue the first element is 0 the string was converted to 0 during search

Simple Test

var_dump("sharp" == 0); //true
var_dump("sharp" === 0); //false

Solution use strict option to search identical values

$testsharp = array_search("sharp", $testcopy,true);
                                               ^---- Strict Option

var_dump($testsharp);

Output

10
like image 155
Baba Avatar answered Sep 28 '22 02:09

Baba


If any key before the searched one is numeric zero, then that key is returned, because it is performing a "loose" match dominated by the array's data type, and "sharp" (if converted to int) counts as zero. Using a strict checking, the correct value is found.

Otherwise, by executing

$testcopy = array_map('strval', $testcopy);

so that values are translated to strings, it works also with "loose" check.

like image 42
LSerni Avatar answered Sep 28 '22 02:09

LSerni