Есть ли принципальное отличие между инъекцией Repository в функцию, и получением его же через entityManager(см. код)?

Question

[Роман]

Данные можно получить так:

public function list(ItemRepository $itemRepository)
    {
        $items = $itemRepository->findAll();

или так

public function list()
    {
        $entityManager = $this->getDoctrine()->getManager();
       $items = $entityManager->getRepository(Item::class)->findAll();

Есть ли какое-то принципиальное отличие между этими двумя подходами?

В чем именно преимущество DI (первого варианта) в данной ситуации? И есть ли оно вообще?

Спасибо.

Answer 1

[sl0]

Если нужно просто получить некоторые данные из репозитория, то первый вариант лучше.
А вот если надо полученные данные как-то изменять, то лучше второй вариант, т.к. сразу можно дернуть persist и flush из em.

Comments

[BoShurik]

Первый вариант выигрывает в любом случае:

public function list(EntityManagerInterface $entityManager, ItemRepository $itemRepository)
{
    $items = $itemRepository->findAll();
    // ...
    $entityManage->flush();
}

public function list()
{
    $entityManager = $this->getDoctrine()->getManager();
    $itemRepository = $entityManager->getRepository(Item::class);

    $items = $itemRepository->findAll();
    // ...
    $entityManage->flush();
}

По сигнатуре метода сразу понятно что за зависимости нужны. Если там EntityManagerInterface, то сразу понятно, что там манипуляции с данными
Гораздо проще мокать зависимости для тестов, как и сказал @quiex
Это занимает банально меньше символов :)

Answer 2

[montray]

Попробуйте написать тесты для этого класса/метода и поймете, в чем разница.

Comments

[Роман]

т.е. если не писать тесты - второй вариант лучше?

а если писать тесты - лучше первый?

так?

[montray]

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

[BoShurik]

montray, мокать можно только интерфейс?

[montray]

BoShurik, задайте этот вопрос здесь на хабре или спросите у гугла.

[BoShurik]

montray, зная ответ на этот вопрос меня удивляет, почему никакой принципиальной разницы?

[montray]

BoShurik, принципиальной разницы нет для человека, которому было адресовано это высказывание. Прочтите еще раз ответ, который был помечен решением и первый комментарий к моему ответу, и тогда поймете, почему ее нет.

[BoShurik]

montray, понял вас. Из фразы "Но я сейчас обратил внимание, что вы инжектите класс, а не интерфейс. В таком случае никакой принципиальной разницы для вас нет." сделал вывод, что на разницу влияет то, что инжектится, а не позиция автора вопроса