Куда утрачивается ссылка на параметр в методе?

Question

[MaxPyane]

Имею не мало опыта работы с PHP, но в этот раз напоролся на не очевидный для меня подводный камень с ссылками который никогда до этого не встречал, может баг, а может я туплю. В общем простой метод, который играет роль заполнителя внешнего хранилища $storage (простая переменная-объект). Проблема возникает когда я внутри метода обращаюсь к сторонней библиотеке (ReflectionDocBlock), она мне парсит док коммент откуда я в свою очередь выдергиваю некоторые ха-ки и добавляю в $storage. Иногда она не может спарсить (например плохо сделанный док коммент), и именно в этот момент происходят какие-то аномалии хотя ни ошибок, ни исключений она не кидает.

private function parseClassAttrsIn(object &$storage, object $source)
{
    $storage->name = 'test';
    // Смотрим что внутри
    var_dump($storage); // object(classStorage)#2513 (1) { ["name"]=> string(4) "test" }

    // Вызываем метод который просто обращается к инстансу ReflectionDocBlock,
    // ничего в нем магического нет
    $docBlock = $this->reflectionDocBlock($source->getDocComment());

    // Опять смотрим что внутри
    var_dump($storage); // string(20) "object(classStorage)"
}

Что здесь произошло o_O? Единственное что осталось от моего $storage'a это описательная стринга, мол здесь был объект classStorage...
Я попробовал работать без ссылки и возвращать $storage, проблема уходит, но это не вариант.

private function parseAttrsIn(object $source, object $storage)
{
    $storage->name = 'test';
    // [...]
    return $storage;
}

Этот $storage удобно передавать ссылкой другим методам и заполнять его ха-ками. Я знаю что PHP очень умный и оптимизированный внутри, и параметры, даже если там обычная переменная, все равно передается по ссылке, и при обращении к нему внутри функции/метода мы все так же работаем с ссылкой, но как только мы задумали его изменить, то он копируется и превращается в локальную переменную. Без амперсанда я теряю $storage, т.к внутри метода изменяю его динамически объявляя свойства. Как я понимаю, амперсандом я говорю PHP в лоб не копировать, а продолжать работать через ссылку.
Понять что за аномалия там происходит я не могу, достаточно перестать обращаться к внешней библиотеке (закомментировать) и ссылки не ломаются. У кого какие идеи?

Comments

[Дмитрий]

Создать клон, забрать с него доку, вернуть старый пришедший по ссылке объект

[MaxPyane]

Дмитрий, ну это по сути то же самое, что я сделал во втором примере, возвращать объект return $storage; и снаружи метода parseAttrsIn() принимать его. В данном случае клон незачем, мне нужен всегда обновленный объект со старыми данными то же. Как оказывается ссылки в PHP просто для галочки, в сложных ситуациях они просто становятся непредсказуемыми и падают. Придется избавиться от ссылок, и добавить на пару строк больше.

Answer 1

[Антон Шаманов]

передавать объекты по ссылке как минимум не имеет смысла т.к. объект переданный в метом все равно указывает на тот же объект и отзязать его можно только через клонирование.

Comments

[Антон Шаманов]

Неизменяемые объекты в PHP

[MaxPyane]

Антон Шаманов,

передавать объекты по ссылке как минимум не имеет смысла

Как минимум это не всегда правда. Цитирую самого себя:

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

В PHP очень хитро замудренная система работы с памятью, поэтому единственный и очевидный вариант, это избавиться от ссылок и не вмешиваться в их работу, т.к они не работают точь в точь как в том же C++. В данном случае я получил совсем неожиданный результат (в PHP мануале предупреждают). Решением будет второй вариант который я предложил в проблеме. Придется на пару строк увеличить код каждого метода и место где он вызывается.

[Антон Шаманов]

MaxPyane, еще раз повторяю object $source равносильно object &$source, чтобы отвязать объект нужно использовать clone

private function parseAttrsIn(object $source, object $storage)
{
    $storageClone = clone $storage;
    $storageClone->name = 'test';
    // ...
    return $storageClone;
}

[MaxPyane]

Антон Шаманов, это вообще не суть вопроса, отвязывание происходит все равно, поэтому не нужно мне рассказывать басни про клонирование которого у меня нигде нет. Проблема явно вылазит при вызове той сторонней библиотеки. Есть другая либа (BetterReflection), она то же использует функционал ReflectionDocBlock, и стоит мне только обратиться к ее методам (пр. getDocBlockTypes()), она в свою очередь обращается к ReflectionDocBlock чтобы сделать то же самое, что я делаю напрямую, то ссылки опять ломаются, причем здесь клонирование? Это явно попахивает багом, искать его мне лень, скорее всего он глубинный в движке PHP, поэтому тема закрыта, нашел обходной путь и решил без ссылок явно возвращая и принимая $storage.

[Антон Шаманов]

MaxPyane, код в студию)))

[MaxPyane]

Антон Шаманов, я еще в описании проблемы альтернативное решение описал, не собираюсь повторяться. Хочешь попытаться воспроизвести проблему, пытайся, может получится. Тема закрыта.