Является ли хорошей практикой передача TempData и/или ViewData службам?

Пытаясь уменьшить размер моих контроллеров 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 и т. д. Номер 2 в значительной степени позволит одно- контроллер лайнера, но вы делаете службу очень осведомленной о вещах, специфичных для контроллера. Тем не менее, служба всегда будет тесно связана с контроллером, так что это проблема? Будет ли 1 или 2 лучшей практикой или что-то еще, чего я не указал?