Можно ли считать DI контейнер подобием фабрики или что я делаю не так?

Question

[Алексей Верховцев]

Делаю инъекцию в конструктор контроллера, а потом просто вызываю метод.
Так выглядит мой сервис-провайдер

public function register()
    {
        //
        $this->app->bind(CampaignService::class, function($app) {
            $entityManager = $app->make('registry')->getManager();
            $doctrineCampaignRepository = new DoctrineCampaignRepository($entityManager, $entityManager->getRepository(Campaign::class));
            $doctrineCampaignTypeRepository = new DoctrineCampaignTypeRepository($entityManager);
            $doctrineCampaignCategoryRepository = new DoctrineCampaignCategoryRepository($entityManager);
            return new CampaignService($doctrineCampaignRepository, $doctrineCampaignTypeRepository, $doctrineCampaignCategoryRepository);
        });
    }

Впервые работаю с сервис-провайдерами и контейнерами, но верно ли я делаю это? Выглядит как подобие фабрики, разве нет? Или оно так и есть или я использую не по назначению это, укажите на ошибки, пожалуйста

Answer 1

[Кирилл Несмеянов]

Метод bind как раз и регистрирует "фабрику". Т.е. при каждом резолве (получения) сервиса из контейнера - он будет создаваться по определяемой схеме.

$container->call(function(CampaignService $c): void {
    echo \spl_object_id($c); // 1
});

$container->call(function(CampaignService $c): void {
    echo \spl_object_id($c); // 2
});

Если сервис должен быть константным, то имеет смысл bind заменить на singleton

$container->call(function(CampaignService $c): void {
    echo \spl_object_id($c); // 1
});

$container->call(function(CampaignService $c): void {
    echo \spl_object_id($c); // 1
});

Comments

[Алексей Верховцев]

Значит ли это, что я делаю это не совсем верно? Получается, что мне нужно создать отдельную фабрику, где создать все эти классы, а в сервисе я лишь буду биндить CampaignService::class на созданную фабрику или же мой код тоже имеет право на жизнь? Заранее спасибо

[Кирилл Несмеянов]

Значит ли это, что я делаю это не совсем верно?

Всё зависит от того, что надо реализовать ;)

Получается, что мне нужно создать отдельную фабрику, где создать все эти классы, а в сервисе я лишь буду биндить CampaignService::class на созданную фабрику или же мой код тоже имеет право на жизнь?

Опять же всё зависит от архитектуры и задач. Если не требуется ничего другого, то написанный вами код - самый простой вариант создания фабрик.

С другой стороны я не вижу вообще необходимости в фабрике в данном примере, по-этому и предложил вариант с методом "singleton"