依赖注入：将调用图与构造图分离

Question

我正在尝试练习依赖注入原则，但我在做时遇到了一些困难所以。

我有一个功能可以定期访问我的数据库，检索产品列表，然后针对这些产品运行一系列测试以确定它们的安全性。

如果发现一个或多个产品不安全，我需要通过实例化和分派 ProductRecall 对象来发出召回。

该函数看起来像这样：（伪代码）

void SafteyInspector::CheckProductSaftey()
{
  database.getProducts( list_of_products )

  foreach( list_of_products as p )
  {
    if ( !runBatteryOfTests( p ) )
      unsafe_products.insert( p );
  }

  if ( !unsafe_products.empty() )
  {
    ProductRecall * recall = 
          new ProductRecall( unsafe_products );  // Oh no!
    recall->dispatch();
  }
}

问题是我在调用图的中间“新建”了一个 ProductRecall 对象。 这违反了依赖注入的原则。 正如所写，如果不创建 ProductRecall 对象，我就无法测试 CheckProductSafey() 。

但是，我无法将 ProductRecall 对象传递到我的 SafetyInspector 对象中，因为 SafetyInspector 是确定不安全产品列表的对象。

我对所有事情都使用构造函数注入，并且我想继续这样做。 另请注意，我可能随时发出多个 ProductRecalls，因此我不一定在构造时将单个 ProductRecall 对象传递到 SafetyInspector 中。

有什么建议么？ 谢谢！

Answer 1

我认为问题可能出在您的 ProductRecall 实现上。 具体来说，如果您可以对新创建的对象调用 dispatch()，则意味着许多实际功能要么隐藏在 ProductRecall 类中，要么 >ProductRecall 类具有静态成员和/或单例，可以为其提供所需的一切。

我建议创建一个名为 ProductRecallDispatcher 的类来处理实际调度的复杂性。 然后，您将创建这些对象之一，并将其​​传递给 SafetyInspector 的构造函数。 这将使您的 CheckProductSafety 函数如下所示：

void SafteyInspector::CheckProductSaftey()
{
  database.getProducts( list_of_products )

  foreach( list_of_products as p )
  {
    if ( !runBatteryOfTests( p ) )
      unsafe_products.insert( p );
  }

  if ( !unsafe_products.empty() )
  {
    recallDispatcher.recall( unsafe_products );
  }
}

Answer 2

我认为您实际上需要想出某种 ProductRecallFactory 来代替。 注入容器支持某种工厂风格的接口是相当常见的，或者您可能只是希望让您的 SafetyInspector 容器感知。

编辑：通过“容器感知”，我指的是按照 COM 的 IObjectWithSite 的方式实现一些接口，以便您的对象可以回调到其父对象。 这是一种较弱的依赖注入形式，它部分地撤销了控制反转。 如果您手动执行依赖项注入，请务必注入工厂对象。