在 catch 块中设置时 TempData 为空，我们重新抛出

Question

我正在使用自定义错误 (web.config) 将未处理的异常发送到我的 ErrorController。

在我的主控制器中，我抛出一个异常，在 catch 块中捕获它，并将 TempData 设置为有一些用户友好的消息。然后我将throw 发送到ErrorController。

当我在这里查看TempData 时，它是空的（Session 也是如此，但它会是，不是吗？）。

我确定它不应该是空的，这是在控制器之间发送数据的正确方法......但它不起作用！我的理解有误吗？

HomeController:

catch (Exception ex)
{
    TempData["MsgForUser"] = "useful message to show";
    // Do some logging...

    throw;
}

ErrorController:

public ActionResult DisplayError(int id)
{
    ViewBag.MsgForUser = TempData["MsgForUser"];
    return View();
}

Web.config:

<system.web>
  <customErrors mode="On" defaultRedirect="~/ErrorPage/DisplayError">
    <error redirect="~/Error/DisplayError/403" statusCode="403" />
    <error redirect="~/Error/DisplayError/404" statusCode="404" />
    <error redirect="~/Error/DisplayError/500" statusCode="500" />
  </customErrors>

Answer 1

我发现在顶级catch 块中执行throw 时会话被破坏，因为对错误控制器的请求。 但是我确实发现返回的RedirectToAction() 保留了它。

所以，我创建了自己的异常类型：

public class MyApplicationException : Exception
{
    /// <summary>
    /// A message that can be shown to the user i.e. not technical
    /// </summary>
    public string MessageForUser { get; set; }

    public MyApplicationException()
    {
    }

    public MyApplicationException(string message, string messageForUser="")
        : base(message)
    {
        MessageForUser = messageForUser;
    }

    public MyApplicationException(string message, Exception inner, string messageForUser = "")
        : base(message, inner)
    {
        MessageForUser = messageForUser;
    }
}

然后，当我遇到预期的问题并将最终将用户发送到通用错误页面时，我会这样使用它：

if (MyList.Count > 14)
{
      throw new MyApplicationException("More than 14 records received: " + MyList.Count.ToString(), "There is a maximum limit of 14 records that can be processed at a time. Please call us to discuss your requirement to process in greater quantities.");
}

在捕获它后，我为用户挑选消息，记录错误，然后return RedirectToAction() 转到错误页面而不会丢失TempData。

catch (MyApplicationException myAppEx)
{
    // If there is a user-friendly message then store it for DisplayError to show
    if (String.IsNullOrEmpty(myAppEx.MessageForUser) == false)
    {
        TempData["MsgForUser"] = myAppEx.MessageForUser;
    }

    // Do some logging to elmah or a custom logger
    util.LogError(myAppEx, "MyFunc() threw an exception", args);

    //
    // We cannot throw here because we lose the Session & TempData. we have to do a manual redirect.
    //
    return RedirectToAction("DisplayError", "ErrorPage", new { id = 500 });
}

最后在我看来，我寻找它并显示它是否存在：

<p>
    An error occurred. Please check & try again later.
</p>
@if (String.IsNullOrEmpty(ViewBag.MsgForUser) == false)
{
    <p class="more-info">Details: @ViewBag.MsgForUser</p>
}

Answer 2

使用 RedirectToAction() 方法时，TempData 是稳定的，在异常处理中，ASP.net 是否使用此方法？也许不吧 ！！ 为什么不使用 Exception.Message 来设置用户友好的消息？在您的 catch 块中设置 ex.Message ，然后在您的错误视图中您可以访问它。您的错误视图模型必须是 @model HandleErrorInfo ，并且 @Model.Exception.Message 已设置您的消息！