注入提供者服务或容器。良好的做法

Question

我的问题更多的是怀疑什么是最佳解决方案。

想象一下，我们需要实现一个处理程序，这取决于来自实体的一个参数。 让我解释一下

OPTION1

• 服务/供应商/ HSBCProvider
• 服务/供应商/ BarclaysProvider
• 服务/ BankHandler
• 控制器/ BankController

BankHandler

class BankHandler { 
    private $container; 
    function __contructor($container) { 
    $this->container = $container; 
    } 
    function create($account) { 
    $provider = $this->container->get('service.provider.'.$account->getName()); 
    $provider->do(); 
    } 
} 

BankController行动

public function createAction($id) { 
    $account = $repository->getAccount($id); 
    $bankHandler = $this->get('service.bank_handler'); 
    $bankHandler->create($account); 
} 

OPTION 2

• 服务/提供者/ HSBCProvider
• 服务/提供者/ BarclaysProvider
• 控制器/ BankController

BankController行动

public function createAction($id) { 
    $account = $repository->getAccount($id); 
    $bankProvider = $this->get('service.provider.'.account->getName()); 
    $bankProvider->do(); 
} 

使用此选项的BankHandler类不是不再需要

我简化所有的逻辑里面的创建和行动。

我不喜欢选项1因为我正在注射容器。 我不喜欢选项2 cos动作控制器有太多的逻辑（胖控制器？）。

还有其他更好的解决方案吗？

Answer 1

与所有提供者创建注册表服务，将其注入处理程序并从中获取提供程序。沿着线的东西...

class BankProviderRegistry 
{ 
    /** 
    * @var array|BankProviderInterface[] 
    */ 
    private $providers = array(); 

    public function __construct(array $providers = array()) 
    { 
     foreach($providers as $name => $provider) { 
      $this->addProvider($name, $provider); 
     } 
    } 

    /** 
    * Add provider to registry 
    * 
    * @param string $name 
    * @param BankproviderInterface $provider 
    */ 
    public function addProvider($name, BankProviderInterface $provider) 
    { 
     $this->provider[$name] = $provider; 
    } 

    /** 
    * Get provider by name 
    * 
    * @param string $name 
    * @return BankproviderInterface 
    */ 
    public function getProvider($name) 
    { 
     if (!isset($this->providers[$name])) { 
      throw new \Exception(sprintf('Provider "%s" is not registered', $name)); 
     } 

     return $this->providers[$name]; 
    } 
} 

您既可以在你的服务，如添加这些..

app.registry.bank_provider: 
    class: AppBundle\Registry\BankProviderRegistry 
    arguments: 
     - 
      'a name': '@app.provider.a_name' 
      'another name': '@app.provider.another_name' 

..，在你的DI扩展，在编译过程，也许别的地方我没有想到。

然后你通过这个到您的处理程序，并获得供应商如..

class BankHandler 
{ 
    private $registry; 

    public function __contruct(BankProviderRegistry $registry) { 
     $this->registry = $registry; 
    } 

    public function create($account) { 
     $provider = $this->registry->getProvider($account->getName()); 
     $provider->do(); 
    } 
}