将TempData和/或ViewData传递给服务是否是一种好的做法？

Question

在试图降低我的MVC控制器的尺寸，我重构了很多逻辑伸到服务（不过，如果它被纳入模型同样的问题也适用）。我经常发现我是直接设置ViewData和/或TempData的信息我想被显示给用户，如：

var repoUser = new UserRepository(); 
var foundUser = repoUser.GetUser(userId); 
if (foundUser == null) { 
    ViewData["ErrorMessage"] = "Could not find user with ID {0}.".FormatWith(userId); 
    return View("UsersRegister"); 
} 

当然，只要你进入一个服务类，你输了直达ViewData，TempData，像View()或RedirectToAction()方法，所以我试图找出对付它的最好实践。两种解决方案浮现在脑海中：

1. 创建一个包含各条关于控制器应该做什么信息类，从服务传回，如：

   public class ResponseInfo { 
       public string Name { get; set; } 
       public bool DoRedirect { get; set; } 
       public object RedirectRouteValues { get; set; } 
       public string InfoMessage { get; set; } 
       public string ErrorMessage { get; set; } 
   } 
   

2. 尝试，并允许服务方法直接获得的东西像ViewData和TempData，例如：

   public ActionResult ToggleUserAttended(int eventId, int userId, HttpRequestBase request, TempDataDictionary tempData, ViewDataDictionary viewData, Func<string, object, ActionResult> redirectAction) { 
       //... 
       var repoUser = new UserRepository(); 
       var foundUser = repoUser.GetUser(userId); 
       if (foundUser == null) { 
        tempData["ErrorMessage"] = "Could not find user with ID {0}.".FormatWith(userId); 
        return redirectAction("UsersRegister", new { id = eventId, returnUrl = request.QueryString["returnUrl"] }); 
       } 
       //... 
   } 
   

   ...然后在控制器：

   return _svcEvent.ToggleUserAttended(123, 234, Request, TempData, ViewData, (name, routeVals) => RedirectToAction(name, routeVals)); 
   

对于编号1中，控制器将有更多的逻辑看ResponseInfo对象，并确定是否重定向做，显示视图，堵塞错误或信息消息分成TempData或ViewData等Number 2几乎可以让一个单线控制器，但是你让服务非常清楚控制器的具体内容。但是，服务总是与控制器紧密相连，所以这是一个问题吗？会1或2是最佳做法，还是我没有列出的其他东西？

Answer 1

也许我误解的东西，但我觉得很奇怪，你要提供相关演示关注到服务的信息。国际海事组织，这是一个非常糟糕的主意，因为服务是业务逻辑，不应该依赖表示层。

我真的不喜欢你们俩都不建议作为在我看来，他们模糊了介绍和服务之间的界限的做法。

举个例子，一个服务应该抛出，如果它不能找到用户，然后控制器应该处理此错误使用适当的UI机制：错误消息，HTTP错误或任何适合您的应用程序。同样，该服务如何知道在哪里重定向？它知道GUI？你如何计划在其他上下文中使用它（API，非Web GUI，不管）？

我最常做的是建立一个基于表单/参数的命令/ DTO，并提供服务。然后任何服务客户端都可以使用相同的命令。如果我需要显示数据，我可以根据需要提供服务/仓库，并将其映射到表示形式（如果需要），然后将其放入ViewData/TempData/Model/Whatever。

以下是一些例子（把它当作伪代码）：

[HttpPost] 
public ActionResult ChangeUserInfo(UserInfoModel model) 
{ 
    var cmd = new ChangeUserInfoCommand(CurrentUserId, model.FirstName, model.LastName); 
    try { 
     userSvc.UpdateUser(cmd); 
     return RedirectToAction(...); 
    } 
    catch(XxxxException e) { // e.g. something wrong (business logic) 
     TempData["Error"] = "an error occurred: " + e.FriendlyError(); 
     return RedirectToAction(...); // error action 
    } 
    catch(SecurityException e) { 
     throw new HttpException(404, "NotFound"); 
    } 
} 

public ActionResult ShowUsers() 
{ 
    var userInfo = svc.GetUsers().Select(u => { Id = u.id, FullName = u.First + " " + u.Last); 
    // or var userInfo = svc.GetUsers().Select(u => Mapper.To<UserDTO>(u)); 
    return View(userInfo); 

    // or without model 
    // ViewData["users"] = userInfo; 
    // return View(); 
} 

请注意，这是一个例证 - 我通常做的完全不同，但我已经制定了很多其他的基础设施（例如我没有明确地处理这样的例外，我的服务有时只有一个方法Execute(IEnumerable<Command> cmds)，我对匿名类作为查看模型等有很好的支持，等等）