Странное поведение класса PHP. Есть ли идеи?

Question

[cat_crash]

Собственно проблема со странным поведением внутри _constructor и дальнейшим присвоением значения.

Сам код:

Class ResultIterator extends \ArrayIterator {

	public function __construct($_array,$object){

		if(is_array($_array) && count($_array)>0) {
            foreach ($_array as $v){
            	$value=Util::ArrayToClass($object,$v);
            	$this->append($value);
            	unset($value);
            }
        }
        var_dump($this);
	}
}

возвращает n одинаковых экземпляров данных (var_dump($this)):

bject(OpswatPHP\ResultIterator)#784 (1) {
["storage":"ArrayIterator":private]=>
array(10) {
[0]=>
object(OpswatPHP\Group\GroupResult)#59 (10) {
["group_id":"OpswatPHP\Group\GroupResult":private]=>
string(9) "103455134"
["group_name":"OpswatPHP\Group\GroupResult":private]=>
string(15) "[EXT] Customers"
}
[1]=>
object(OpswatPHP\Group\GroupResult)#59 (10) {
["group_id":"OpswatPHP\Group\GroupResult":private]=>
string(9) "103455134"
["group_name":"OpswatPHP\Group\GroupResult":private]=>
string(15) "[EXT] Customers"
....
}

но если поставить var_dump во внуть цикла:

public function __construct($_array,$object){

		if(is_array($_array) && count($_array)>0) {
            foreach ($_array as $v){
            	$value=Util::ArrayToClass($object,$v);
            	$this->append($value);
            	var_dump($value);
            	unset($value);
            }
        }
        
	}

то выводится правильные данные (разные значения)

object(OpswatPHP\Group\GroupResult)#59 (10) {
["group_id":"OpswatPHP\Group\GroupResult":private]=>
string(22) "0051a00000393LXdefault"
["group_name":"OpswatPHP\Group\GroupResult":private]=>
string(7) "Default"
}
object(OpswatPHP\Group\GroupResult)#59 (10) {
["group_id":"OpswatPHP\Group\GroupResult":private]=>
string(9) "226066674"
["group_name":"OpswatPHP\Group\GroupResult":private]=>
string(9) "[ZZZ] XXX"
}

куда копать ?

Comments

[cat_crash]

После долгих дебагов и опытов склоняюсь думать что это глубокий баг PHP.
Немного переписав логику кода проблема исчезла. Рабочий вариант если кто то заинтересуется:

public function __construct($_array,$object){
		
		if(is_array($_array) && count($_array)>0) {
			$_tmp=array();
            foreach ($_array as $v){
            	$_tmp[]=serialize(Util::ArrayToClass($object,$v));
            	
            }

            //Ugly hack because looks like PHP 7.x have a bug w alreasy assigned values
            foreach($_tmp as $item){
            	$this->append(unserialize($item));
            }
        }
	}

т.е сериализуем то что возвращается превращая объект в строку и еще раз циклом де-сериализуем и вуаля - все работает

[Adamos]

После беглого знакомства с вопросом есть подозрение, что этот "глубокий баг" создавали вы сами, зачем-то вызывая unset.

[Евгений Самсонов]

Adamos, Решение "бага" тоже доставляет)

[Adamos]

Евгений Самсонов, на него я как-то и не смотрел... да, ЩИ, конечно...

[Кирилл Несмеянов]

Adamos, а не смущает, что в аргумент "ArrayToClass" передаётся один и тот же объект всегда? Очевидно, что он там где-то внутри патчится и возвращается этот же самый объект. А потом опять передаётся и опять патчится (уже пропатченный). Ну типа не смущает, что идентификатор у объекта всегда одинаковый? var_dump об этом как бы явно пишет строкой "#59"...

[cat_crash]

Adamos, хотелось бы в это верить но увы не в этом проблема.
изначальный код был

foreach ($_array as $v){
                $this->append(Util::ArrayToClass($object,$v));
            }

(прув: https://github.com/khvalov/opswat-php/blob/master/...
те без всякой лапши. Лапша появилась в результате поисков возможных воркэраундов.

Answer 1

[Кирилл Несмеянов]

В PHP все объекты (и ресурсы) всегда передаются по указателю, в отличии от скаляров (и массивов), которые передаются по значению.

Передавая один и тот же объект в каждом "тике" цикла и возвращая его же - ты всегда будешь работать с одним и тем же объектом и патчить его же. Ваш Капитан Документация.

Так что отвечая на вопрос - копать в сторону изучения передачи аргументов в PHP. Это не больно. Т.е. в сторону документации.

P.S. Если переписать конкретно этот код на "человеческий" PHP, то выглядеть он должен так:

class ResultIterator extends \ArrayIterator
{
    public function __construct(array $array, object $object)
    {
        foreach ($array as $value) {
            $this->append(Util::ArrayToClass(clone $object, $value));
        }
    }
}

Но вообще проблема не в нём, а в "ArrayToClass", который занимается манки-патчингом и заставляет клонировать объекты.

Comments

[cat_crash]

Благодарю за исчерпывающий ответ и указание в какую сторону "читать"