Можно ли в Symfony инициализировать в конструкторе сервисы если их не используешь?

Question

[Di Lee]

Есть сервис в Symfony - SomeService (см. ниже)
И есть клиентский код который его использует (см. ниже)

В клиентском коде есть несколько разных вызовов разных методов сервиса SomeService и каждый метода может использовать общие сервисы на всех, а также и свой набор сервисов.

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

Как правильно решать проблему?

Делать общий родитель с общим конструктором и потом в классах потомках дополнять конструктор своими сервисами, используя конструкторы с вызовом внутри parent::__construct()? Решение возможно сомнительное - будет та же повторяемость кода(перечисление сервисов) в параметрах конструкторов потомков.

Или есть более правильное решение?

class SomeService
{
    private ServiceFirst  $serviceFirst;

    private ServiceSecond $serviceSecond;

    public function __construct(
           ServiceFirst $serviceFirst, 
           ServiceSecond $serviceSecond, 
           SomeCommonService $someCommonService
   )  {
        $this->serviceFirst = $serviceFirst;
        $this->serviceSecond = $serviceSecond;
        $this->someCommonService = $someCommonService;
    }
    
    public function methodFirst()
    {
        $this->someCommonService->doSomethingCommon();
        $this->serviceFirst->doSomethingFirst();

        return $this->commonMethod();
    }

    public function methodSecond()
    {
        $this->someCommonService->doSomethingCommon();
        $this->serviceSecond->doSomethingSecond();
        
        return $this->commonMethod();
    }

    private function commonMethod(): string
    {
        // some code
        
        return 'some string';
    }
}

И есть какой то клиентский код который его использует

// клиентский код1

$someResultString1 = $someService->methodFirst();
//...

// клиентский код2
$someResultString2 = $someService->methodSecond();

Answer 1

[voronkovich]

Под ваше описание подходит Service Subscriber

Что такое "подписчик сервисов"?

Это сервис, который получает свои зависимости в виде локатора (https://en.wikipedia.org/wiki/Service_locator_pattern). При этом, локатор реализует ленивую загрузку, что позволяет не создавать экземпляры сервисов без необходимости.

Когда его следует использовать?

Когда у сервиса много зависимостей, но не все они используются им одновременно.
Примером, может служить AbstractController, который зависит от Twig, Doctrine, Router и т.д.
Очевидно, что не для каждого запроса нужен Twig (json api), и не в каждом запросе есть обращение к БД.
См. https://github.com/symfony/symfony/blob/e1581a0937...

Как его создать?

Реализовать интерфейс ServiceSubscriber:

<?php

use Psr\Container\ContainerInterface;
use Symfony\Contracts\Service\ServiceSubscriberInterface;

class SomeService 
{
    public function __constructor(
        private ContainerInterface $container,
    ) {
    }
    
    public static function getSubscribedServices(): array
    {
        return [
            ServiceFirst::class => ServiceFirst::class,
            ServiceSecond::class => ServiceSecond::class,
            SomeCommonService::class => SomeCommonService::class,
        ];
    }
    
    public function methodFirst()
    {
        $this->container[SomeCommonService::class]->doSomethingCommon();
        $this->container[ServiceFirst::class]->doSomethingFirst();

        return $this->commonMethod();
    }
    
    public function methodSecond()
    {
        $this->container[SomeCommonService::class]->doSomethingCommon();
        $this->container[ServiceSecond::class]->doSomethingSecond();
        
        return $this->commonMethod();
    }

    private function commonMethod(): string
    {
        // some code
        
        return 'some string';
    }
}

Однако, я соглашусь с ответами выше. В подавляющем большинстве случаев лучше будет разбить на несколько сервисов.

Comments

[Di Lee]

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

Answer 2

[Дмитрий Гординский]

Разбить по отдельным сервисам*, в том числе и содержимое commonMethod вынести в отдельную структуру, которую использовать в сервисах с methodFirst и methodSecond.

*это не обязательно должны быть сервисы