带有多个存储库和服务的.NET MVC控制器？

Question

看看我的Controller(我正在使用依赖注入来管理依赖项):

public RoleController(IRoleRepository roleRepository, ISiteRepository siteRepository, IUserRepository userRepository, IDbContext dbContext)
{
    _roleRepository = roleRepository;
    _siteRepository = siteRepository;
    _userRepository = userRepository;
    _dbContext = dbContext;
}

拥有许多依赖项的类是代码味道？对？

但是,在我的示例中,我需要关联Users并Sites在a中Role,然后我需要这些依赖关系来执行此关联.

在邮件列表上的一些人我被告知有太多的依赖关系表明某些事情可能是错误的.但我没有别的办法.我分开了我的责任,在那种情况下有一些我不知道如何对待？有什么不对吗？

更新:

我需要Repositories和DbContext,因为DbContext是我的UnitOfWork,存储库不保存.

此示例是一个简单的CRUD,其中包含一些其他功能,例如视图中的关联和GRID.

更新2:

我正在使用我的UI层是MVC的架构.

Answer 1

鉴于您使用良好的DI框架(即不使用穷人的DI)管理依赖关系,我不认为这是一件坏事.这样,你明确地说控制器将需要所有这些东西,因为它会.(请注意,在应用程序的许多其他部分中,这可能不是一个有效的参数 - 控制器在控制和指导程序流的方式上是特殊的,因此有一个自然的解释为什么它需要看到很多应用程序的一部分......)

但是,如果你真的想限制这种特殊情况下依赖关系的数量,它可能是有意义的创造MembershipService,它完成所有涉及的工作Users,Sites和Roles.那将依赖于这三个存储库,并且您的控制器将仅依赖于成员资格服务.

为了响应您的更新:您可以将工作单元(即数据库上下文)注册为"每个Web请求"单例 - 这可以使用Castle Windsor和许多其他DI框架.然后你可以让你的存储库依赖它并做所有的更改,并让控制器依赖它来保存,并且它们都将获得由DI框架传递给它们的相同实例.