Является ли данный код нарушением принципа Dependency Inversion?

Question

[Nikita]

Создание объекта сущности в контроллере будет ли являться нарушением DI?
Пример из демо symfony:
https://github.com/symfony/demo/blob/e5e5a8fff0483...

public function new(Request $request): Response
    {
        $post = new Post();
        $post->setAuthor($this->getUser());
...

Answer 1

[Daria Motorina]

Тут нет нарушения, потому что нет дополнительной бизнес логики в виде разных подвидов поста, поэтому можно ограничиться созданием поста локально, в пределах экшена контроллера. Нарушение DI присутствует, когда явно видно, что классы зависят от реализации там, где пора создавать абстракцию

Comments

[Nikita]

Спасибо, за ответ.
А почему тогда в данном случае https://github.com/symfony/demo/blob/e5e5a8fff0483... PostRepository передается параметром?

public function index(PostRepository $posts): Response
    {
        $authorPosts = $posts->findBy(['author' => $this->getUser()], ['publishedAt' => 'DESC']);

[Daria Motorina]

Nikita, тут это уместно, потому что можно очертить экшен как зависимость от репозитория и инстанцировать его вне экшена в сервис контейнере нормально, но репозитории не так часто инжектят в методы, чаще инжектят EntityManager (репозиторий будет получен уже внутри метода и связность кода будет ниже). В демке симфони просто не заморачивались с расширяемостью многих вещей, потому что это одноразовый проект, плюс ко всему есть принципы kiss и yagni (условно - не надо писать слишком гибкий код с первой попытки, потому эта гибкость может оказаться неоправдана с точки зрения бизнеса)

Answer 2

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

не будет, но лучше использовать абстрактную фабрику
p.s. new зарезервировано php и не может быть именем метода

Comments

[Дмитрий Беляев]

Мне кажется, тут достаточно фабричного метода, ведь создается лишь одна сущность Post
Абстрактная фабрика все таки про набор связных сущностей

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

Дмитрий Беляев, мы обсуждаем абстрактный пример, а не конкретную реализацию т.ч. остается только гадать. я предположил что имеется базовый интерфейс моделей symfony, поэтому и abstract factory.

[DevMan]

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

new зарезервировано php и не может быть именем метода

рили?
https://3v4l.org/9VPr5