Symfony — Repository Как избежать двойного кода в этом случае?

Question

[symnoob]

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

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

public function save(User $user): void
    {
        $this->getEntityManager()->persist($user);
        $this->getEntityManager()->flush();
    }

хотелось бы создать какой нибудь абстрактный класс для этого дела, но как быть с тайп-хинтингом?

Comments

[tukreb]

А что не так? Это нормально, правда flush() делать надо в контроллере в handler или прямо в action контроллера. Иначе вы будете делать бесполезные записи в БД, когда это можно сделать одним коммитом.

[symnoob]

этот метод должен присутствовать во всех CRUD Контроллерах/Репозиториях. А не очень охота иметь один и тот же метод везде.
по сути только один Тайпхинт и отличается.

[tukreb]

symnoob, ну если очень хочется, то сделайте родительский класс с save() и принимайте mixed объект.

[D3lphi]

Вот поэтому языку и не хватает дженериков.

[Максим Федоров]

D3lphi, да чего уже, можно забыть
Psalm в руки и поехали

[D3lphi]

Максим Федоров, в такой манере можно "оправдать" любой недостаток любого языка.
В джаве runtime дженерики не нужны, можно забыть, Class<T> в конструктор и поехали.
В пхп аннотации, кстати, тоже не нужны. Зря в восьмерку добавили. Костыль в виде doctrine/annotations и поехали.

[Максим Федоров]

D3lphi, я не писал, что не нужны — сам бы хотел :(

я написал, что можно забыть

[D3lphi]

Максим Федоров, хорошо, я вас не так понял. Ответ на мой комментарий, как мне показалось, выглядел как "дженерики не нужны".

Answer 1

[Максим Федоров]

flush() нужно вынести, потому что вы можете заперсистить 100 объектов и сделать один flush — одна череда обращений к Бд. В вашем же случае будет 100 и более обращений к БД

Чтобы сделать универсальный метод для персиста в репозиториях, раз уж вы решили так делать (многие считают это верным) — можно сделать универсальный код (трейт к примеру), в котором будет проверка через classMetadata, что сохраняемый объект нужного типа для репозитория (тут не забываем про наследование сущностей, is_a()и вот это все) и если придет не тот объект — кидать InvalidArgumentException

НО! Если вы репозитории размещаете в своем домене/пакете/неймпсейсе, то вам не избежать дублирования кода, и это не так уж и страшно... Связанность и паутина наследований страшнее. Репозиториев все равно мало во всем коде относительно всей базы кодовой.

Comments

[symnoob]

tommy-vercetti не совсем понятно почему именно в контроллере? Операция по сути проходит с бд... как-бы логичней и выкинуть все в Repository?

[symnoob]

tommy-vercetti супер, спасибо