次の例でサービスロケータがアンチパターンであるのはなぜですか？

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

サービスロケータが実際にアンチパターンであるとします。しかし、私の実装は反パターンではないと私は思う。

サービスロケータをアンチパターンと見なす理由は、依存関係を隠すためですが、私は唯一の依存関係（Entity Managerを挿入します。この依存関係はおそらく変更されません。サービス・インタフェースを使用して、サービス・ロケータをインスタンス化するときに、サービス・クラスが必要とします。ここで

は私のコードです：

フロントコントローラとアクションヘルパーの登録

<?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); 
    } 
} 

：

//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を介して注入されます。だからあなたのクラスは、その依存関係がどこから来たのか分からず、それを作成する責任もありません（これは良いことです）。

私が間違っている場合は私を修正してください。 :)

ところで、依存性注入コンテナを渡してはいけない理由もそうです。それはサービスロケータに変わります。