ASP.NET MVC自定义错误处理Application_Error Global.asax?


108

我有一些基本代码来确定我的MVC应用程序中的错误。目前在我的项目,我有一个名为控制器Error与操作方法HTTPError404()HTTPError500()General()。它们都接受字符串参数error。使用或修改下面的代码。将数据传递给错误控制器进行处理的最佳/正确方法是什么?我想有一个尽可能强大的解决方案。

protected void Application_Error(object sender, EventArgs e)
{
    Exception exception = Server.GetLastError();
    Response.Clear();

    HttpException httpException = exception as HttpException;
    if (httpException != null)
    {
        RouteData routeData = new RouteData();
        routeData.Values.Add("controller", "Error");
        switch (httpException.GetHttpCode())
        {
            case 404:
                // page not found
                routeData.Values.Add("action", "HttpError404");
                break;
            case 500:
                // server error
                routeData.Values.Add("action", "HttpError500");
                break;
            default:
                routeData.Values.Add("action", "General");
                break;
        }
        routeData.Values.Add("error", exception);
        // clear error on server
        Server.ClearError();

        // at this point how to properly pass route data to error controller?
    }
}

Answers:


104

无需为此创建新路由,您只需重定向到控制器/操作并通过querystring传递信息即可。例如:

protected void Application_Error(object sender, EventArgs e) {
  Exception exception = Server.GetLastError();
  Response.Clear();

  HttpException httpException = exception as HttpException;

  if (httpException != null) {
    string action;

    switch (httpException.GetHttpCode()) {
      case 404:
        // page not found
        action = "HttpError404";
        break;
      case 500:
        // server error
        action = "HttpError500";
        break;
      default:
        action = "General";
        break;
      }

      // clear error on server
      Server.ClearError();

      Response.Redirect(String.Format("~/Error/{0}/?message={1}", action, exception.Message));
    }

然后,您的控制器将收到您想要的任何东西:

// GET: /Error/HttpError404
public ActionResult HttpError404(string message) {
   return View("SomeView", message);
}

您的方法需要权衡。在这种错误处理中循环时要非常小心。另一件事是,由于要通过asp.net管道来处理404,因此将为所有这些匹配创建会话对象。对于频繁使用的系统,这可能是一个问题(性能)。


当您说“小心循环”时,您指的是什么?有没有更好的方法来处理这种类型的错误重定向(假设它曾经是一个使用率很高的系统)?
09年

4
循环是指当您在错误页面中出现错误时,您会一次又一次地重定向到错误页面...(例如,您想将错误记录在数据库中并且已关闭)。
andrecarlucci

125
重定向错误与Web体系结构背道而驰。当服务器响应正确的HTTP状态代码时,URI应保持不变,以便客户端知道失败的确切上下文。实现HandleErrorAttribute.OnException或Controller.OnException是更好的解决方案。如果失败,请在Global.asax中执行Server.Transfer(“〜/ Error”)。
阿斯比约恩Ulsberg

1
@Chris,可以接受,但不是最佳实践。尤其是由于它经常被重定向到带有HTTP 200状态代码的资源文件,因此客户端可以认为一切正常。
阿斯比约恩Ulsberg

1
我必须将<httpErrors errorMode =“ Detailed” />添加到web.config才能在服务器上工作。
Jeroen K

28

要回答最初的问题“如何正确地将routedata传递给错误控制器?”:

IController errorController = new ErrorController();
errorController.Execute(new RequestContext(new HttpContextWrapper(Context), routeData));

然后在您的ErrorController类中,实现如下函数:

[AcceptVerbs(HttpVerbs.Get)]
public ViewResult Error(Exception exception)
{
    return View("Error", exception);
}

这会将异常推送到视图中。视图页面应声明如下:

<%@ Page Language="C#" Inherits="System.Web.Mvc.ViewPage<System.Exception>" %>

以及显示错误的代码:

<% if(Model != null) { %>  <p><b>Detailed error:</b><br />  <span class="error"><%= Helpers.General.GetErrorMessage((Exception)Model, false) %></span></p> <% } %>

这是从异常树中收集所有异常消息的函数:

    public static string GetErrorMessage(Exception ex, bool includeStackTrace)
    {
        StringBuilder msg = new StringBuilder();
        BuildErrorMessage(ex, ref msg);
        if (includeStackTrace)
        {
            msg.Append("\n");
            msg.Append(ex.StackTrace);
        }
        return msg.ToString();
    }

    private static void BuildErrorMessage(Exception ex, ref StringBuilder msg)
    {
        if (ex != null)
        {
            msg.Append(ex.Message);
            msg.Append("\n");
            if (ex.InnerException != null)
            {
                BuildErrorMessage(ex.InnerException, ref msg);
            }
        }
    }

9

我找到了Lion_cl指出的ajax问题的解决方案。

global.asax:

protected void Application_Error()
    {           
        if (HttpContext.Current.Request.IsAjaxRequest())
        {
            HttpContext ctx = HttpContext.Current;
            ctx.Response.Clear();
            RequestContext rc = ((MvcHandler)ctx.CurrentHandler).RequestContext;
            rc.RouteData.Values["action"] = "AjaxGlobalError";

            // TODO: distinguish between 404 and other errors if needed
            rc.RouteData.Values["newActionName"] = "WrongRequest";

            rc.RouteData.Values["controller"] = "ErrorPages";
            IControllerFactory factory = ControllerBuilder.Current.GetControllerFactory();
            IController controller = factory.CreateController(rc, "ErrorPages");
            controller.Execute(rc);
            ctx.Server.ClearError();
        }
    }

ErrorPagesController

public ActionResult AjaxGlobalError(string newActionName)
    {
        return new AjaxRedirectResult(Url.Action(newActionName), this.ControllerContext);
    }

AjaxRedirectResult

public class AjaxRedirectResult : RedirectResult
{
    public AjaxRedirectResult(string url, ControllerContext controllerContext)
        : base(url)
    {
        ExecuteResult(controllerContext);
    }

    public override void ExecuteResult(ControllerContext context)
    {
        if (context.RequestContext.HttpContext.Request.IsAjaxRequest())
        {
            JavaScriptResult result = new JavaScriptResult()
            {
                Script = "try{history.pushState(null,null,window.location.href);}catch(err){}window.location.replace('" + UrlHelper.GenerateContentUrl(this.Url, context.HttpContext) + "');"
            };

            result.ExecuteResult(context);
        }
        else
        {
            base.ExecuteResult(context);
        }
    }
}

AjaxRequestExtension

public static class AjaxRequestExtension
{
    public static bool IsAjaxRequest(this HttpRequest request)
    {
        return (request.Headers["X-Requested-With"] != null && request.Headers["X-Requested-With"] == "XMLHttpRequest");
    }
}

在执行此操作时,出现以下错误:'System.Web.HttpRequest'不包含'IsAjaxRequest'的定义。本文提供了一种解决方案:stackoverflow.com/questions/14629304/…–
朱利安·多蒙

8

之前,我在集中将全局错误处理例程集中在MVC应用程序中的想法困扰。我在ASP.NET论坛上有一个帖子

它基本上可以处理global.asax中的所有应用程序错误,而无需错误控制器,使用[HandlerError]属性进行修饰或摆弄customErrorsweb.config中的节点。


6

在MVC中处理错误的更好方法可能是将HandleError属性应用于控制器或操作,并更新Shared / Error.aspx文件以执行所需的操作。该页面上的Model对象包括Exception属性以及ControllerName和ActionName。


1
那么您将如何处理404错误?因为没有为此指定控制器/动作?
Dementic 2015年

接受的答案包括404。此方法仅对500个错误有用。
布赖恩

也许您应该将其编辑为答案。Perhaps a better way of handling errors听起来很像所有错误,而不仅仅是500。
Dementic 2015年


4

这可能不是MVC的最佳方法(https://stackoverflow.com/a/9461386/5869805

下面是如何在Application_Error中呈现视图并将其写入http响应。您不需要使用重定向。这将阻止对服务器的第二次请求,因此浏览器地址栏中的链接将保持不变。这可能是好是坏,这取决于您想要什么。

Global.asax.cs

protected void Application_Error()
{
    var exception = Server.GetLastError();
    // TODO do whatever you want with exception, such as logging, set errorMessage, etc.
    var errorMessage = "SOME FRIENDLY MESSAGE";

    // TODO: UPDATE BELOW FOUR PARAMETERS ACCORDING TO YOUR ERROR HANDLING ACTION
    var errorArea = "AREA";
    var errorController = "CONTROLLER";
    var errorAction = "ACTION";
    var pathToViewFile = $"~/Areas/{errorArea}/Views/{errorController}/{errorAction}.cshtml"; // THIS SHOULD BE THE PATH IN FILESYSTEM RELATIVE TO WHERE YOUR CSPROJ FILE IS!

    var requestControllerName = Convert.ToString(HttpContext.Current.Request.RequestContext?.RouteData?.Values["controller"]);
    var requestActionName = Convert.ToString(HttpContext.Current.Request.RequestContext?.RouteData?.Values["action"]);

    var controller = new BaseController(); // REPLACE THIS WITH YOUR BASE CONTROLLER CLASS
    var routeData = new RouteData { DataTokens = { { "area", errorArea } }, Values = { { "controller", errorController }, {"action", errorAction} } };
    var controllerContext = new ControllerContext(new HttpContextWrapper(HttpContext.Current), routeData, controller);
    controller.ControllerContext = controllerContext;

    var sw = new StringWriter();
    var razorView = new RazorView(controller.ControllerContext, pathToViewFile, "", false, null);
    var model = new ViewDataDictionary(new HandleErrorInfo(exception, requestControllerName, requestActionName));
    var viewContext = new ViewContext(controller.ControllerContext, razorView, model, new TempDataDictionary(), sw);
    viewContext.ViewBag.ErrorMessage = errorMessage;
    //TODO: add to ViewBag what you need
    razorView.Render(viewContext, sw);
    HttpContext.Current.Response.Write(sw);
    Server.ClearError();
    HttpContext.Current.Response.End(); // No more processing needed (ex: by default controller/action routing), flush the response out and raise EndRequest event.
}

视图

@model HandleErrorInfo
@{
    ViewBag.Title = "Error";
    // TODO: SET YOUR LAYOUT
}
<div class="">
    ViewBag.ErrorMessage
</div>
@if(Model != null && HttpContext.Current.IsDebuggingEnabled)
{
    <div class="" style="background:khaki">
        <p>
            <b>Exception:</b> @Model.Exception.Message <br/>
            <b>Controller:</b> @Model.ControllerName <br/>
            <b>Action:</b> @Model.ActionName <br/>
        </p>
        <div>
            <pre>
                @Model.Exception.StackTrace
            </pre>
        </div>
    </div>
}

这是IMO的最佳方式。正是我想要的。
史蒂夫·哈里斯

@SteveHarris很高兴它有所帮助!:)
burkay

3

Brian,这种方法非常适合非Ajax请求,但正如Lion_cl所述,如果在Ajax调用过程中出错,则您的Share / Error.aspx视图(或自定义错误页面视图)将返回给Ajax调用者, -用户将不会被重定向到错误页面。


0

使用以下代码在路线页面上进行重定向。使用exception.Message实例化异常。如果Coz异常查询字符串扩展了查询字符串的长度,则会给出错误。

routeData.Values.Add("error", exception.Message);
// clear error on server
Server.ClearError();
Response.RedirectToRoute(routeData.Values);

-1

我对这种错误处理方法有疑问:对于web.config:

<customErrors mode="On"/>

错误处理程序正在搜索View Error.shtml,并且仅在异常发生后,控制流才进入Application_Error global.asax。

System.InvalidOperationException:找不到视图“错误”或其主视图,或者没有视图引擎支持搜索到的位置。搜索以下位置:〜/ Views / home / Error.aspx〜/ Views / home / Error.ascx〜/ Views / Shared / Error.aspx〜/ Views / Shared / Error.ascx〜/ Views / home / Error。 cshtml〜/ Views / home / Error.vbhtml〜/ Views / Shared / Error.cshtml〜/ Views / Shared / Error.vbhtml,位于System.Web.Mvc.ViewResult.FindView(ControllerContext上下文)................。 ............

所以

 Exception exception = Server.GetLastError();
  Response.Clear();
  HttpException httpException = exception as HttpException;

httpException始终为null,然后customErrors mode =“ On” :(这具有误导性,<customErrors mode="Off"/>否则<customErrors mode="RemoteOnly"/>用户将看到customErrors html,然后customErrors mode =“ On”,此代码也将出错


该代码的另一个问题是

Response.Redirect(String.Format("~/Error/{0}/?message={1}", action, exception.Message));

返回页面,代码为302,而不是实际错误代码(402,403等)

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.