Почему то strpos работает не правильно?

Question

[tvoyadres]

PHP 7.3

$user_ip = 94301750;
if ( strpos('1334268210|3322954437', $user_ip) !== false ) {
echo 'Yes';
}

Почему оно срабатывает ни понимаю, ведь так не должно быть.

Answer 1

[nokimaro]

Коротко - не передавайте в strpos число (int), a передавайте строку (sting) и проблем не будет

Разврёнуто
Читаем документацию https://www.php.net/strpos

needle
Если параметр needle не является строкой, он преобразуется в целое число и трактуется как код символа. Это поведение устарело с PHP 7.3.0, и полагаться на него крайне не рекомендуется. В зависимости от предполагаемого поведения, параметр needle должен быть либо явно приведен к строке, либо должен быть выполнен явный вызов chr().

Если прогнать ваш код то видим ошибку

Deprecated: strpos(): Non-string needles will be interpreted as strings in the future. Use an explicit chr() call to preserve the current behavior in /in/2JE4h on line 4

Чтобы код отрабатывал как ожидается, $user_ip должен иметь тип string

так
$user_ip = (string)94301750;
или так
$user_ip = '94301750';

В противном же случае вместо "94301750" в strpos уходит результат chr(94301750) что есть строка "6"
и получается в итоге strpos('1334268210|3322954437', '6') отсюда и результат Yes

Comments

[tvoyadres]

О большое спасибо!

[tvoyadres]

И еще хотел спросить в PHP 5.6 такая проблема могла возникнуть или это проявляется только в PHP 7.3

[Илья]

tvoyadres, Попробуй в песочнице запусти и узнаешь) sandbox.onlinephpfunctions.com

[nokimaro]

tvoyadres, судя по информации, то в версии 5.6 - 7.4.4 оно работает так, как в моём ответе - int преобразуется в chr(int).

В будущих версиях PHP поведение будет другое, если передать число - int будет автоматически приведено к типу string, о чём написано в документации.

Чтобы избежать проблем, и не зависеть от версии РНР, рекомендую делать как в ответе, всегда явно передавать в функцию строку.

Answer 2

[Дмитрий Ким]

https://www.php.net/manual/ru/function.strpos.php:

Если параметр needle не является строкой, он преобразуется в целое число и трактуется как код символа. Это поведение устарело с PHP 7.3.0, и полагаться на него крайне не рекомендуется. В зависимости от предполагаемого поведения, параметр needle должен быть либо явно приведен к строке, либо должен быть выполнен явный вызов chr().

$user_ip = "94301750";
if ( strpos('1334268210|3322954437', $user_ip) !== false ) {
    echo 'Yes';
}
// ...