在 MVC 中，对于具有多个构造函数的控制器，我如何指示要调用哪个构造函数？

Question

通常情况下，我希望在控制器中拥有多个构造函数，因为其中一个构造函数最适合在单元测试期间手动注入依赖项，而另一个构造函数最适合使用 IOC 容器注入依赖项。

在 MVC 3 中使用标准服务定位器或 DependencyResolver 时，是否有任何方法可以向控制器工厂指示在激活控制器时使用哪个构造函数？

我不想使用 IOC 框架特有的属性之一，因为我希望代码保持独立于 IOC 容器。

更新： 似乎大多数答案都表明拥有多个构造函数是不好的做法。我并不完全不同意这一点，但是，这种情况是由于试图找到映射/转换层的良好解决方案而出现的，我在 这个问题但从未得到很好的答案。

我的代码如下所示：

public class SomeController : Controller
{
    private ISomeService _SomeService;
    private IEntityConverter _EntityConverter;

    // this would be used during unit tests when mocking the entity converter 
    public SomeController(ISomeService someService, IEntityConverter entityConverter)
    {
        _SomeService = someService;
        _EntityConverter = entityConverter;
    }

    // this would be used when injecting via an IOC container, where it would be tedious
    // to always specify the implementations for the converters
    public SomeController(ISomeService someService)
        : this(someService, new EntityConverter())
    {
    }

    public ActionResult Index(SomeViewModel someViewModel)
    {
        if (!ModelState.IsValid)
            return View(someViewModel);
        else
        {
            ServiceInput serviceInput = _EntityConverter.ConvertToServiceInput(someViewModel);
            _SomeService.DoSomething(serviceInput);
            return View("Success");
        }
    }
}

在这个非常简单的示例中，我使用一个接口在视图模型和作为服务层输入的实体之间进行转换。

我更喜欢使用接口，因为可以模拟此转换，而不是使用静态方法。然而，我觉得总是必须在 IOC 容器中指定这些转换实现的实现是很乏味的，因为这些转换接口的实现就位于接口旁边。

这可能不是最好的解决方案。我对更好的选择持开放态度。

Answer 1

不要使用多个构造函数。您的类应该对它所需的依赖项有一个单一的定义。该定义位于构造函数中，因此它应该只有 1 个公共构造函数。

我必须说我不喜欢使用模拟框架。我发现我的应用程序和单元测试的良好设计使我不必一起使用模拟框架（当然里程可能会有所不同）。

对我有帮助的一个技巧是允许通过不验证构造函数参数来将 null 引用注入到构造函数中。如果被测类不使用依赖项，这对于单元测试来说非常方便，而在生产中，您仍然可以保证不注入 null，因为 IOC 容器永远不会将 null 注入到构造函数中（或者至少不会注入 null）。我所知道的容器）。

Answer 2

似乎有一种代码味道，您需要一个显式构造函数来进行测试。如果测试人员可以工作但生产人员不能工作怎么办？看起来就像是错误进入的一扇门。对于测试来说，尽可能使用与真实客户端相同的路径是理想的。

两者有何不同？

另外，如果您使用某种基于属性的 DI（例如 MEF），我不会太担心解耦。拥有额外的属性（以及相应的参考）应该不会有那么大的影响......除非您预见到未来 IOC 技术会发生变化。

Answer 3

也许我误读了你的问题，但我不确定你为什么要测试不同的构造函数。测试应该测试实际将要使用的内容。

对于您的测试，您应该模拟您的依赖项，以便它们不需要 IoC 容器。

使用 Moq：

控制器：

public SomeController(IService1 service1, IService2 service2)
{
    // Do some cool stuff
}

测试方法：

[Test]
public void Some_Action_In_SomeController_Should_Do_Something()
{
    var service1Mock = new Mock<IService1>();
    var service2Mock = new Mock<IService2>();
    var controller = new SomeController(service1Mock.Object, service2Mock.Object);

    controller.Action();

    // Assert stuff
}

Answer 4

这一切都与您的服务定位器有关，您需要在注册类型的位置对其进行配置。

下面是使用 Unity 时使用默认无参数构造函数的示例。

请注意，默认情况下服务定位器将使用最具体的构造函数（参数最多的构造函数）

ServiceLocator.Current.RegisterType<ICache, HttpContextCache>(new InjectionConstructor()); //use default constructor

那么为什么我需要这个呢？确实，我可以创建一个受保护的虚拟方法来解析 HttpContextBase 并在测试中使用提取和重写，或者使用另一个接口，该接口具有创建可以注入到此类中的 HttpContextBase 的方法......或者我可以只是告诉 Unity 使用默认构造函数。

    private readonly HttpContextBase _httpContextBase;

    public HttpContextCache()
    {
        _httpContextBase = new HttpContextWrapper(HttpContext.Current);
    }

    public HttpContextCache(HttpContextBase httpContextBase)
    {
        _httpContextBase = httpContextBase;
    }

这是一种罕见的情况，项目中几乎所有其他类都只有一个构造函数