在 MVC 中使用 DI 时的大量控制器构造函数参数列表

我正在开发 ASP.NET MVC3 解决方案，该解决方案使用 autofac 的依赖注入。 我们的控制器由 autofac 正确创建，所有必需的对象都正确传入。这些对象通常包括服务、存储库和将域对象转换为 MVC（视图）模型的映射器。所以控制器构造函数看起来有点像：

public abcController(
        ILogger logger,
        IabcRepository abcRepository,
        IabcService abcService,
        IMapper<AbcDomain, AbcViewModel> abcMapper,
        ...
        )

不幸的是，随着时间的推移，这些构造函数参数列表往往会增长得很快。我们的一些控制器现在需要 60 个或更多参数。

我们在这里创建了一些反模式吗？

EDIT

我应该提到我们尝试遵循瘦控制器模式。此外，大多数参数往往是映射器 - 大约 66%。控制方法通常非常简单，遵循以下模式：

• 根据参数调用适当的服务或存储库
• 使用映射器将结果转换为适当的视图模型
• 将视图模型传递给视图

或者这个模式：

• 从后期操作接收模型
• 使用映射器将其转换为适当的域对象
• 使用域对象调用适当的服务或存储库

尽管我同意大多数其他答案，但我无法真正谈论您应该如何重新构建控制器 - 60 个传入参数已经很多了。

可能有助于减少参数数量但不会减少依赖项数量的东西是聚合服务支持 https://code.google.com/p/autofac/wiki/AggregateServiceAutofac 有。

您可以采用包含 60 个属性的单个聚合参数，而不是直接采用 60 个参数。

您使用依赖项创建一个接口（只是接口，实际上不必实现它）：

public interface IMyAggregateService
{
  IFirstService FirstService { get; }
  ISecondService SecondService { get; }
  IThirdService ThirdService { get; }
  IFourthService FourthService { get; }
}

然后修改您的控制器以采用该聚合接口：

public class SomeController
{
  private readonly IMyAggregateService _aggregateService;

  public SomeController(
    IMyAggregateService aggregateService)
  {
    _aggregateService = aggregateService;
  }
}

您可以注册聚合服务接口、依赖项和控制器，当您解析控制器时，聚合服务接口将自动为您实现和解析。

var builder = new ContainerBuilder();
builder.RegisterAggregateService<IMyAggregateService>();
builder.Register(/*...*/).As<IFirstService>();
builder.Register(/*...*/).As<ISecondService>();
builder.Register(/*...*/).As<IThirdService>();
builder.Register(/*...*/).As<IFourthService>();
builder.RegisterType<SomeController>();
var container = builder.Build();

再说一遍，它并没有涉及需要那么多依赖项的更大问题，但如果您只是想简化构造函数和控制器上的属性数量，使其更易于管理，这是 Autofac 提供的一种策略可以帮助您解决这一问题。 。

查看 wiki 页面了解更多详细信息。 https://code.google.com/p/autofac/wiki/AggregateService