Получаются тяжелые конструкторы с большим количеством инжекшен параметров?

Question

[magary4]

public function __construct(
        Request $request,
        $router,
        Storage $storage,
        $user,
        Session $session,
        TranslatorInterface $translator,
        DocumentRepository $documentRepository,
        \Swift_Mailer $mailer,
        EngineInterface $templating,
        $checkout_email,
        WishlistOrderManager $wishlistOrderManager
    )
    {
        $this->request = $request;
        $this->router = $router;
        $this->storage = $storage;
        $this->user = $user;
        $this->session = $session;
        $this->translator = $translator;
        $this->documentRepository = $documentRepository;
        $this->mailer = $mailer;
        $this->templating = $templating;
        $this->checkout_email = $checkout_email;
        $this->wishlistOrderManager = $wishlistOrderManager;
    }

так и хочется инжектить контейнер и оттуда доставать все после ижекшена

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

Comments

[dev400]

инжектите в нужный метод

[magary4]

не совсем понял

через calls: чтоли?

так такое же количество параметров будет

Answer 1

[Алексей Скобкин]

Дело не только в независимости от контейнера, а в очевидности наличия зависимостей. И в минимизации ответственности класса. Зачем ему знать о существовании чего-то, что ему не нужно?

Comments

[magary4]

__constructor( $container ) {
$this->mailer = $container->get('nailer');
$this->templating = $container->get('templating');
$this->repository= $container->get('repository');
. . .
}

вот так тоже вполне все очевидно

[Александр Евгеньевич]

magary4:
$container->get('nailer');

У вас класс зависит от контейнера. А то, что есть в контейнере или нет, он не знает, пока не спросит.
О том, есть 'nailer' или нет вы узнаете только во время выполнения специфической задачи. Самое простое - у вас может быть 10 методов, и вам придется их всех проверить, иначе во время жизни проекта может случится баг.

[Алексей Скобкин]

magary4: Отнюдь. Особенно глядя в конфигурацию контейнера будет очень очевидно, что именно нужно вашему сервису, когда ему передаётся только контейнер.

Answer 2

[Александр Евгеньевич]

так ли действительно нужно быть независимым от контейнера?

Да. SOLID.

так ли действительно нужно быть независимым от контейнера?

Нет, если это необходимо, и обусловлено спецификой задачи, контейнер - это тот же класс.

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