8

在使用带有构造函数注入的依赖注入框架时使用可选参数是否被认为是一种不好的做法？

例子：

public class ProductsController
{
    public ProductsController(IProductService productService = null, IBackOrderService = null)
    {
    }
}

我已将这两个参数都指定为可选参数，但我的 DI 框架将始终注入这两个依赖项。如果我向我的控制器添加一个需要新依赖项的新操作，那么将新依赖项设为可选是否会很糟糕？即使现有的测试不需要新的依赖项，我也可能会破坏数十个单元测试。

编辑
人们似乎对我的问题感到困惑。我从不在我的 Web 应用程序中手动构建 ProductsController。这由控制器工厂（自动注入依赖项）处理。

我不喜欢这样的单元测试：

[Test]
public void Test1()
{
    var controller = new ProductsController(new MockProductService(), new MockBackOrderService());
}

现在我决定向我的控制器添加一个新的操作方法。此新操作需要新的依赖项，但现有操作都不需要。现在我必须回去修改 100 个不同的单元测试，因为我添加了一个新参数。我可以通过使参数可选来避免这种情况，但我想知道这是否是一个坏主意。我的直觉说不，因为它唯一影响的是单元测试。

4

3 回答 3

11

我完全同意所有情况下的公认答案，即定义一个依赖意味着没有它实现将无法工作。

但是，如果您有一些不一定需要依赖项但您希望能够在该依赖项已加载的情况下配置某些东西的情况。好的...？这听起来有点奇怪，但它是一个有效的元编程用例——你认为工厂模式可能会有所帮助..但即使是工厂也可能需要一些、没有或全部依赖项，所以工厂不能解决这个问题。

特征标志的现实世界问题。如果关闭了某个功能，则您不需要该依赖项..或者甚至无法创建它，因为没有具体的实现。所以它关闭了。它编译，一切正常。但是后来有人打开了这个功能，突然间我们需要这种依赖。

我找到了一种方法来做到这一点——最好的部分是我通过学习另一种不太知名的依赖注入技术才意识到如何做到这一点（我正在使用 Microsoft.Extensions.DependencyInjection）

为单个接口注入多个具体实现

services.AddTransient<IWarehouseRepo, ActionRepository>();
services.AddTransient<IWarehouseRepo, EventRepository>();
services.AddTransient<IWarehouseRepo, AuditRepository>();
services.AddTransient<IWarehouseRepo, ProRepository>();
services.AddTransient<IWarehouseRepo, SuperWarehouseRepository>();
services.AddTransient<IWarehouseRepo, InferiorWarehouseRepository>();
services.AddTransient<IWarehouseRepo, MonthlyStatisticsRepository>();

我最近才知道这是完全有效的，但要让它工作，你的构造函数需要看起来像这样..

public WarehouseHandler(IEnumerable<IWarehouseRepo> repos)

所以这太酷了！我可以根据任何标准选择我需要的存储库。但这对可选依赖项有何帮助？

因为这种类型的构造函数会给你0个或更多的依赖。因此，如果您不添加任何依赖项，您可以使用条件语句在构造函数中分支 

  if (repos.Count() == 0)
    return;

这是空引用安全的，不需要默认值，易于调试，易于阅读且易于实现。

您现在也可以将此技术用作功能切换机制！

于 2019-03-29T15:00:07.090 回答
6

我不认为这是一个好主意。构造函数注入意味着依赖项是必需的。如果参数之一为空，您甚至应该添加引发的警戒线。

我认为问题在于您的单元测试。例如，我只有一个地方创建控制器并模拟支持对象（controllerContext、HttpContext、Request、Response 等）。然后，如果我在构造函数中添加一个新参数，我只需要在单元测试中的一个地方更改它。

也许您应该考虑在单元测试中编写一个通用基类，或者为测试使用“设置”例程。

于 2012-04-20T20:56:14.903 回答
0

考虑使用测试数据生成器模式。

以最简单的形式，它可以作为静态测试辅助方法来完成

public ProductsController BuildControllerWIthMockDependencies ()
{
    var controller = new ProductsController(new MockProductService(), new MockBackOrderService());
return controller;
}

您可以将AutoFixture 用作通用测试数据生成器。

可以在测试数据构建器中找到使用测试数据构建器的更多技术：对象母模式的替代方案

于 2013-07-30T10:29:05.760 回答