Почему локатор сервисов является Anti-Pattern в следующем примере?

Question

У меня есть приложение MVC с хорошо известной моделью домена, с объектами, репозиториями и уровнем обслуживания.

Чтобы избежать необходимости создавать классы обслуживания внутри моих контроллеров и, таким образом, беспорядок мои контроллеры с логикой, которая им не подходит, я создал помощник, который действует как своего рода Service Locator, но после прочтения я понял что многие разработчики:

• http://blog.tfnico.com/2011/04/dreaded-service-locator-pattern.html
• http://blog.ploeh.dk/2010/02/03/ServiceLocatorIsAnAntiPattern.aspx
• http://underground.infovark.com/2010/06/18/the-service-locator-pattern-is-the-new-global-variable/
• http://www.andyfrench.info/2011/05/service-locator-anti-pattern_17.html

Скажите, что Service Locator на самом деле является анти-шаблоном. Но я считаю, что моя реализация не является анти-шаблоном.

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

Вот мой код:

<?php 

namespace App\Controller\Action\Helper; 
use Zend_Controller_Action_Helper_Abstract as Helper, 
    Doctrine\ORM\EntityManager; 

/** 
* Service Locator Helper 
* @author JCM 
*/ 
class Service extends Helper { 

    /** 
    * The actual EntityManager 
    * @var \Doctrine\ORM\EntityManager 
    */ 
    private $entityManager; 

    /** 
    * Services Namespace 
    * @var string 
    */ 
    private $ns; 

    /** 
    * @param EntityManager $entityManager 
    * @param string $ns The namespace where to find the services 
    */ 
    public function __construct(EntityManager $entityManager, $ns) 
    { 
     $this->entityManager = $entityManager; 
     $this->ns = $ns; 
    } 

    /** 
    * @param string $serviceName 
    * @param array $options 
    * @param string $ns 
    */ 
    public function direct($serviceName, array $options = array(), $ns = null) 
    { 
     $ns = ((!$ns) ? $this->ns : $ns) . '\\'; 
     $class = $ns . $serviceName; 

     return new $class($this->entityManager, $options); 
    } 

    /** 
    * @param EntityManager $entityManager 
    */ 
    public function setEntityManager(EntityManager $entityManager) 
    { 
     $this->entityManager = $entityManager; 
    } 

    /** 
    * @return \Doctrine\ORM\EntityManager 
    */ 
    public function getEntityManager() 
    { 
     return $this->entityManager; 
    } 

    /** 
    * @param string $name 
    */ 
    public function __get($name) 
    { 
     return $this->direct($name); 
    } 
} 

Регистрация Действие Helper с Front Controller:

//inside some method in the bootstrap 
HelperBroker::addHelper(new App\Controller\Action\Helper\Service($entityManager, '\App\Domain\Service')); 

И как я использую этот помощник в моих контроллеров:

//Some Controller 
$myService = $this->_helper->service('MyService'); //returns an instance of the class App\Domain\Service\MyService 
$result = $myService->doSomethingWithSomeData($this->getRequest()->getPost()); 
//etc... 

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

Answer 1

Я не думаю, что то, что вы создали, реализует шаблон локатора сервисов. Если бы это было так, там был бы глобальный «реестр».

Что я вижу в основном заводского класса (App\Controller\Action\Helper\Service) с зависимостями, которые вводятся через ctor. Поэтому ваш класс не знает, откуда взялись его зависимости, и он также не несет ответственности за их создание (что хорошо!).

Исправьте меня, если я ошибаюсь. :)

BTW: Это также является причиной того, почему вы не должны проходить вокруг контейнера для инъекций. Он превращается в локатор сервисов.