В
Может, еще какие есть?
Ладно, надо с этим что-то решать)) Я вижу такие варианты:
- Трейты - ок
- Эвенты внутри сущностей не ок, поэтому сделаем сервис
Size: a a a
2020 November 05
СП
У тебя DTO простые) Конечно там не ОК
В
У тебя DTO простые) Конечно там не ОК
Я про пакет в целом же
СП
В сущности есть логика, там где логика - есть события :)
AB
Понятно что создание без конструктора в бизнес логике это правктика плохая. Но в ORM это очень помогает. Чтоб не городить на каждую сущность фабрику, в которой генерировать id и проставлять createdAt
СП
Понятно что создание без конструктора в бизнес логике это правктика плохая. Но в ORM это очень помогает. Чтоб не городить на каждую сущность фабрику, в которой генерировать id и проставлять createdAt
Как раз в ORM это помогает логике. Сущность не создаётся заново при получении её из БД. И это логично.
AB
Я разве спорю? 🙂
AB
Я к тому что вся магия под капотом ORM никак не мешает
2020 November 06
В
@predvoditelev, а что ты думаешь насчет того, чтобы сделать и трейт, и сервис?
СП
@predvoditelev, а что ты думаешь насчет того, чтобы сделать и трейт, и сервис?
трейт - я понимаю как использовать.
СП
а сервис - не понимаю :)
В
$service->pushEvent($event);
$service->flush();
))
$service->flush();
))
СП
$service->pushEvent($event);
$service->flush();
))
$service->flush();
))
Технически сам сервис - понимаю. Не понимаю, как в проекте это применять.
В
На мой взгляд, в 90% случаев сервис подойдет. А там, где по каким-то причинам класс имеет зависимости и не создается через фабрики/контейнеры, придется трейт юзать.
В
Технически сам сервис - понимаю. Не понимаю, как в проекте это применять.
В твоем примере - вынести создание эвентов из DTO
СП
В
Да, но не имплементировать
EventDispatcherInterface. Представить это как нечто абсолютно от него отдельное.СП
Да, но не имплементировать
EventDispatcherInterface. Представить это как нечто абсолютно от него отдельное.А почему не реализовывать?
Зачем события без диспетчера?
Зачем события без диспетчера?
В
Диспетчер будет в конструкторе. А этот сервис - не диспетчер