Тонкости ООП в PHP. Почему перезаписываются данные?

Question

[Дмитрий Батулин]

Добрый день!

Столкнулся со странным поведением PHP. Ну или со своей не полной компетентностью в данном вопросе.
Прошу помощи у знающих людей.
Заранее благодарен.

Суть ситуации.
Есть образный статический класс Items
у него 2 статические функции
Items::preload() - формирует массив элементов и записывает в приватное свойство класса
Items::getItems() - возвращает список элементов сгенерированных первой функцией

Теперь, что работает не так как мне хотелось бы.

Items::preload();
$list = array(
    'action1' => 'addOne',
    'action2' => 'addTwo'
);
foreach( $list as $key => $action ){
    $itemList = Items::getItems();
    foreach( $itemList as $id => $item ){
        $itemList[ $id ] = doSomethinkWithItem( $action, $item );
    }
}

function doSomethinkWithItem( $action, $item ){
    $item->{$action} = 'someText';
}

class Items {
    private static $arItems;

    public static function preload(){
        // $arItems выборка из базы
        foreach($arItems as $ID => $data){
            $one = new \stdClass();
            $one->id  = $ID;
            $one->name = $data["NAME"];

            self::$arItems[ $ID ] = $one;
        }
    }

    public static function getItems(){
        return self::$arItems;
    }
}

Проблема:
При повторном получении списка элементов методом $itemList = Items::getItems();
у элементов сохраняются свойства после обработки первым методом, а мне необходимо, что бы на момент начала работы со списком, он был неизменный и изначально заданный. Т.к. во время работы с ним, какие то характеристики меняются, какие то удаляются элементы вообще.

Кто может подсказать что не так со структурой и как сделать так, что бы изначально сформированный список не изменялся вообще и отдавался всегда один и тот же?

Answer 1

[Melkij]

По вашей функции doSomethinkWithItem очевидно, что аргумент $item является объектом.
Объекты передаются всегда по ссылке. Для создания копии объекта - есть оператор clone

Comments

[Дмитрий Батулин]

То, что передается по ссылке, я понял, спасибо про то, что указал в чем именно дело.
Но я что-то не очень могу представить как можно скопировать статический класс?
Можешь подсказать?
$itemList = clone Items::getItems(); - понятное дело выдает ошибку, что это не объект

[Melkij]

Клонировать каждый объект коллекции в отдельности. Например,
public static function getItems(){
return array_map(function($el) {
return clone $el;
}, self::$arItems;
}

Статический класс здесь значения не имеет. По ссылке передаются сами stdClass

[Дмитрий Батулин]

Ага. методом проб тоже до этого дошел.
Твой подход мне больше нравится.
я ж сделал клонирование сам в цикле:
$itemList = Items::getItems();
foreach( $itemList as $id => $item ){
$item_clone = clone $item;

Но в итоге переделал на возвращение функции.
Еще раз спасибо тебе

Answer 2

[Anton B]

Вы реализацию getItems покажите, видимо функцию возвращает не массив, а ссылку.

Comments

[Дмитрий Батулин]

class Item {
    private static $arItems;

    public static function preload(){
        // $arItems выборка из базы
        foreach($arItems as $ID => $data){
            $one = new \stdClass();
            $one->id  = $ID;
            $one->name = $data["NAME"];

            self::$arItems[ $ID ] = $one;
        }
    }

    public static function getItems(){
        return self::$arItems;
    }
}