PHP7 и foreach. Коварное изменение?

Question

[Андрей Павленко]

Как говорится в документации, теперь в PHP7 foreach работает с копией массива, а не с самим массивом.
То есть, раньше этот код работал, а теперь - нет (если не ошибаюсь):

<?php
foreach ($arr as $k=>$v)
{
if ($some_condition==$v) {
   unset($arr[$k]);
}
}

К сожалению, в коде есть некоторые места, которые используют подобную конструкцию.
Получается, нужно с помощью поиска по проекту найти все foreach и заменить $k=>$v на $k=>&$v
Поможет ли это?
Или нужно еще какой-то способ, к примеру (переписать на array_filter)?
Как нужно изменить выше указанный код и нужно ли его изменять вообще?

Comments

[romy4]

Александр А: это не уязвимость

[Робот]

Александр А: если у меня написан нормальный безопасный код, как версия языка сделает его уязвимым?

[Робот]

Александр А: Ну переписана и что, в каком месте будет уязвимость в trim() например? ))

[Андрей Павленко]

Это не уязвимость, об этом сообщили в документации. Поэтому я спросил, нужно ли переписывать подобный код, которого у меня, к сожалению, немало и, уверен, что немало его и у других.

Answer 1

[nelson]

Читайте официальную документацию по переезду, там всё написано.
php.net/manual/en/migration70.incompatible.php

А именно, по вашему вопросу про foreach:

foreach by-value operates on a copy of the array
When used in the default by-value mode, foreach will now operate on a copy of the array being iterated rather than the array itself. This means that changes to the array made during iteration will not affect the values that are iterated.

Перевод: в режиме foreach($arr as $v) - движок сначала сделает копию массива и будет проводить итерацию по копии. Даже если во время цикла вы сделаете $arr = [], то цикл всё равно пробежит до конца исходного массива, по всем элементам.

То есть код теперь эквивалентен такому:

$arr_copy = $arr;
foreach($arr_copy as $k=>$v) {
...
}

за тем исключением, что к переменной $arr_copy вы не имеете прямого доступа.

Это не значит, что переменная $arr внутри цикла начинает ссылаться на копию. Поэтому Ваш код будет работать.
Вот если бы вы работали в цикле не только с текущим элементом, а например, и со следующими относительно текущего, изменяя или удаляя их, тогда поведение программы изменилось бы. Но я никогда не встречал подобного использования цикла foreach.

Comments

[Adamos]

Для такого использования все равно потребуется целочисленный индекс текущего элемента и контроль границ массива, так что foreach просто неактуален.

[Андрей Павленко]

Большое спасибо, что разобрались и объяснили!

Answer 2

[romy4]

Приведённый код будет нормально работать в php7.

Comments

[romy4]

Суть. В данном примере копия делается с $arr для итерации по foreach, а $arr остаётся самим собой.

[Алексей Уколов]

https://3v4l.org/3s8Hs#v5410 - наглядная демонстрация

[Андрей Павленко]

То есть, я могу изменять все, как было?
Ну уж накрутили они в документации - а примерв я не нашел. Спасибо за помощь!

Answer 3

[65536]

в общем то и раньше не стоило менять массив из форыча по нему

Comments

[Adamos]

Почему? Например, у словаря сменить кодировку - законная добыча foreach.

[nelson]

Присоединяюсь к вопросу. Кто-нибудь знает почему?

[65536]

nelson: теперь вроде очевидно почему))

[nelson]

65536: пока не совсем...

[Андрей Павленко]

Согласен, немного код с запахом. Сейчас есть смысл использовать array_filter или array_map. Но факт в том, что код есть.
К счастью, никаких нарушений для его работоспособности не происходит.