Как получить из фабрики коллекцию сущностей либо саму сущность?

Question

[ragnar_ok]

Нужно решить проблему получения из фабрики коллекции сущностей, либо самой сущности. Решил нижеуказанным образом.

Это нормально, что AbstactEntityFactory::get возвращает Collection|EntityInterface? В следствие этого не имеет type hint. Предложите лучшее решение проблемы.

Возможно я двигаюсь не в том направлении?

PHP 7.1.

interface EntityFactoryInterface
{
    /**
     * @param mixed[] $state
     */
    public function create(array $state): EntityInterface;
}
 
abstract class AbstactEntityFactory
{
    /**
     * Отстуствует TYPE HINT. Вместо него phpdoc.
     * @param mixed[] $state
     *
     * @return Collection|EntityInterface
     */
    public function get(array $state)
    {
        return $this->create($state);
    }
 
    /**
     * @param mixed[] $state
     */
    protected function createCollection(array $state): Collection
    {
        $collection = new Collection();
 
        foreach ($state as $value) {
            $collection->push($this->create($value));
        }
 
        return $collection;
    }
}
 
class FooFactory extends AbstactEntityFactory
{
    /**
     * {@inheritdoc}
     */
    public function create(array $state): EntityInterface
    {
        return Foo::fromState($state);
    }
 
    /**
     * {@inheritdoc}
     */
    public function get(array $state): Collection
    {
        return $this->createCollection($state);
    }
}
 
class BarFactory extends AbstactEntityFactory
{
    /**
     * {@inheritdoc}
     */
    public function create(array $state): EntityInterface
    {
        return Bar::fromState($state);
    }
}

$fooFactory = new FooFactory();
$foo = $factory->get($state); // Collection|EntityInterface[]
$barFactory = new BarFactory();
$bar = $factory->get($state); // EntityInterface

Answer 1

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

1. ты путаешь/миксуешь фабрику и билдер
2. должен быть определен абстрактный метод AbstactEntityFactory::create()
3. что мешает реализовать Collection::__construct(array $entities)?
4. фабричный метод EntityInterface::fromState() заявлен непосредственно в модели зачем создавать отдельный класс?

Comments

[ragnar_ok]

3, 4. Существует также сущность Baz. Эта сущность имеет свойства $bar и $foo, которые соответствуют одноименным сущностям из листинга кода в моем вопросе. Отдельный класс нужен для маппинга этих сущностей. Чтобы из массива получить сущность, которая включает в себя другие связанные сущности. По крайней мере, мне кажется, что он нужен. Предложите вариант лучше?

https://pastebin.com/dEnh28WT

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

ragnar_ok, данные классы фабрик бесполезны т.к. являются обертками EntityInterface::fromState(). Не надо изобретать велосипед, используй готовый DI пакет, например, Pimple

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

ragnar_ok, по поводу коллекции

$collection = new Collection(array_map('Foo::fromState', $fooStates));

[ragnar_ok]

Антон Шаманов, но обертка позволяет использовать создавать в цикле сущности (как в BazMapper::map из моего примера): "определяет общий интерфейс для создания объектов в суперклассе, позволяя подклассам изменять тип создаваемых объектов". Так ли она бесполезна?

spoiler

В целом я понимаю, что лучше избавится от фабрик и использовать конкретный механизм. Уточняю для понимания.

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

ragnar_ok, есть шаблон фабричный метод -FactoryMethod, а есть фабрика - SimpleFactory или StaticFactory. Ты пытаешься реализовать оба шаблона, что в данном случае
это не имеет смысла.

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

ragnar_ok, AbstactEntityFactory объекта передаешь имя класса-наследника EntityInterface и вызываешь колбек [$entityClass, 'fromState'] либо ставляешь фабрики и заменяешь fromState на конструктор принимающий массив свойств.

[ragnar_ok]

Антон Шаманов, избавился от метода AbstactEntityFactory::get. Объявил фабричный метод для создания коллекции AbstactEntityFactory::createEntityCollection. Правильно понял?

https://pastebin.com/eU6HrR4v

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

ragnar_ok, нет. либо фабрика, причем абстрактная завязанная на EntityInterface, либо интерфейс EntityInterface объявляет фабричный метод fromState(). + CollectionBuilder в конструктор которого передается callback-фабрика и класс-коллекции (т.е. Collection).

abstract class Entity implements EntityInterface
{ 
    public function __construct(array $state)
    { 
           ...
    }
}

Чем тебя не устраивает такой вариант?