2

我开始为我的 MVC 项目实现一个服务层，以减少一些臃肿的控制器（它也有存储库/工作单元模式）。

我的问题是，如果您有一个包含大量子对象等的页面的复杂视图模型，并且在幕后进行了大量逻辑（让您了解原始开发人员编写的控制器有近 4000 行代码！ ！）可以让多个服务停止做他们的事情吗？还是我应该只拥有一个可以完成所有工作的大型 ReportService？

我的控制器开始看起来像这样？如果我继续下去，我最终可能会调用很多不同的服务来构建视图模型。

这看起来不错还是开始朝错误的方向发展？

public ViewResult Index(int? reportId)
    {
        // get the base report object
        var reportService = new ReportService();
        var report = reportService.GetByReportId(reportId);
        var model = Mapper.Map<Report, ReportViewModel>(report);

        // get the current active user
        var userService = new UserService();
        var user = userService.GetCurrentUser();
        model.User = Mapper.Map<User, ReportViewModel.UserViewModel>(user);

        // get the first unread message
        var messageService = new MessageService();
        var message = messageService.GetFirstUnread(user.Id);
        model.Message = Mapper.Map<Message, ReportViewModel.MessageViewModel>(message);

        // get the category navigation
        var categoryService = new CategoryService();
        var categoryNavigation = categoryService.GetCategoryNavigation(report.Id);
        model.CategoryNavigation = Mapper.Map<IEnumerable<Category>, IEnumerable<ReportViewModel.CategoryNavigationViewModel>>(categoryNavigation);

        return View(model);
    }
4

2 回答 2

4

在您的控制器中有多个小型服务很好。但是，这里有一个问题：

您的服务应该在整个控制器中可用，并通过构造函数注入以实现松耦合。

所以是这样的：

private readonly IReportService _reportService;
private readonly IUserService _userService;

public SomeConstructor(IReportService reportService, IUserService userService, etc.) 
{
    _reportService = reportService;
    _userService = userService;
    // etc
}
于 2012-06-05T22:55:17.453 回答
1

这看起来确实是一个不错的方法，另一种方法是通过使用子操作将其中的一部分拆分——不过，最好的解决方案将取决于您的具体用例。

例如，如果视图使用 ViewModel 属性CategoryNavigation来创建一种可能在多个不同视图中有用的导航“小部件”，则最好将其拆分为 ChildAction，例如

[ChildActionOnly]
public ActionResult CategoryNavigationWidget(int reportId)
{
    // get the category navigation
    var categoryService = new CategoryService();
    var categoryNavigation = categoryService.GetCategoryNavigation(report.Id);

    return PartialView(categoryNavigation);
}

然后，任何 View 都可以通过以下方式呈现该 ChildAction：

   @{ Html.RenderAction("CategoryNavigationWidget", "Report", 
           new { reportId = Model.ReportId }); }

这是否是一个好主意可能取决于“小部件”是否可重用。

于 2012-06-06T03:48:38.357 回答