Какой тут паттерн нужен?

Question

[gog69]

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

class Detail {
     function create($data) {
          ......
         
         $this->nestedDatail = new Detail()->create( $data->nestedDetailData );
         return $this;
     }
}

(консткруктор занят dependency injection)

Answer 1

[Иван Корюков]

А нужен ли тут паттерн? Возможно стоило бы описать недостатки того что есть, хотелки, и спросить как этого добиться.

Comments

[gog69]

ну хотелку я описал
как ее добиться?

[Иван Корюков]

Нужен отдельный построитель объекта - фабрика или билдер, который будет получать данные, и отдавать объект.
Рекурсия данных должна обходиться как раз этим билдером. DetailBuilder->create($data_chunk) должен создавать объект, проходиться по nested_data_chunck и для каждого, в только что созданный объект записывать $obj->nested_detail = $this->create($nested_data_chunk);

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

Возможно для рекурсивного обхода данных можно было бы приспособить какой-нибуль рекурсивный итератор, или даже шаблон Посетитель.

Answer 2

[Andrey Kulikovsky]

Ваш класс очень похож на объект бизнес-логики (domain entity).
Предпочтительно использовать при создании объекта и установки значений конструктор (entity всегда должна быть валидной) и постараться избавиться от внедрения зависимости через конструктор, например так:

public function updatePassword($plainPassword, PasswordEncoderInterface $encoder) {
    $this->password = $this->encoder->encode($plainPassword);
}

Если у нас множество вариантов создания объекта, удобно использовать именованные конструкторы, такого вида:

public static function create($userName, $email) {
    $user = new self();
    $user->username = $username;
    // ...
    return $user;
}

public static function createFromMyVariant($uid) {
    $user = new self();
    // ...
    return $user;
}

Comments

[gog69]

Спасибо
сейчас буду пробовать

А скажите в чем отличие этого подхода от фабрики?

[gog69]

мне еще не понятно кто должен отвечать за извлечение данных из БД?
это нужно вынести из тела класса или

public static function createId($id) {
$user = new self();
$user->subitems = $this->storage->find($id);
return $user;
}

[Andrey Kulikovsky]

gog69: По сути ни чем не отличается от фабрики. Но в данном случае соблюдается паттерн "Информационный эксперт".

Извлечение из БД.
Тут множество вариантов, начиная от ручками собирать объект или active record, заканчивая использованием полноценной ORM с DataMapper - Doctrine.
Лично я предпочитаю - Doctrine, тогда наши сущности максимально абстрагированы от СУБД.

[gog69]

Здесь нет ни доктрины ни активрекорд
есть только $this->storage
его и надо использовать
как это сделать по-уму? внутри этого класса или это должно быть изолировано?

[Andrey Kulikovsky]

gog69: Изолировано, для этого есть замечательный паттерн Repository. Его обязанность извлекать объекты из хранилища, сохранять, удалять. Таким образом, наши domain-object изолирован и мы не завязаны на реализации хранилища.

И ещё очень рекомендую прочитать книгу Эванса по DDD, там в часности очень хорошо разложен материал на эту тему.

[gog69]

ну так мой $this->storage - это и есть репозиторий
я правильно к нему обращаюсь?

[gog69]

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

[Andrey Kulikovsky]

Репозиторий должен возвращать уже готовый объект/объекты.

class MyRepository
{

    private $storage;


    public function get($id) {
        $data = $this->storage->fetch($id);
        $object = MyObject::createFromArray($data);
        //... Ещё какой-то код, связи например
        return $object;
    }


    public function save(MyObject $object) {

        $data = [];
        // Тут свой код, создаем например массив $data, все что нужно для сохраниение в наш $storage
        $this->storage->save($data);
    }
    
    // Аналогично удаление, выборки, если нужно

}

[Andrey Kulikovsky]

PS. забыл в конструктор проинжектить $storage

[gog69]

т.е из контроллера я делаю $this->get('myrepository')->get(123) ???
а не так как сейчас $this->get('MyObject')->create(123) который при создании извлекает данные?

[gog69]

правильно понял?

[gog69]

И там же в репозитории
вместо
//... Ещё какой-то код, связи например

я могу сделать
$object->setRelatedObjects( $this->getRelatedObjects($object->getId()) )
?

т.е репозиторий он и как фабрика? если он наполняет свойства обьекта?

[Andrey Kulikovsky]

да, все верно.

[gog69]

Спасибо огромное за толкое разьяснение

если можно, то еще один вопрос
как сделать для этого обьекта lasy-loading?

class MyClass {

function getChildren() {
$this->repository->load( .. );
}
}

чет мне кажется что надо репозиторий дергать

$myObject = $this->get('myrepository')->get(123);
$myObject->setChildren($this->get('myrepository')->getChildren($myObject));

а сам класс MyClass полностью изолирован и в него ничего не инжектится