ASP.Net MVC 异常日志记录与错误处理相结合

Question

我正在寻找一个简单的解决方案来在我的 ASP.Net MVC 1.0 应用程序中进行异常日志记录和错误处理。

我读过很多文章，包括 StackOverflow 上发布的问题，它们都针对不同情况提供了不同的解决方案。 我仍然无法想出适合我需求的解决方案。

以下是我的要求：

1. 能够在我的控制器上使用 [HandleError] 属性（或等效的属性）来处理可能从任何操作或视图引发的所有异常。 这应该处理未在任何操作上专门处理的所有异常（如第 2 点所述）。 我希望能够为控制器中的所有操作指定在错误情况下用户必须重定向到哪个视图。

2. 我希望能够在特定操作的顶部指定 [HandleError] 属性（或等效属性）以捕获特定异常并将用户重定向到适合该异常的视图。 所有其他异常仍必须由控制器上的 [HandleError] 属性处理。

3. 在上述两种情况下，我希望使用 log4net（或任何其他日志库）记录异常。

   在

我该如何实现上述目标？ 我读过有关使我的所有控制器继承自重写 OnException 方法的基本控制器的内容，并在其中进行日志记录。 然而，这会扰乱将用户重定向到适当的视图，或者使其变得混乱。

我读过有关编写自己的过滤器操作的信息，该操作实现了 IExceptionFilter 来处理此问题，但这将与 [HandleError] 属性发生冲突。

到目前为止，我的想法是最好的解决方案是编写我自己的继承自 HandleErrorAttribute 的属性。 这样我就可以获得 [HandleError] 的所有功能，并且可以添加我自己的 log4net 日志记录。 解决方案如下：

    public class HandleErrorsAttribute: HandleErrorAttribute {

      private log4net.ILog log = log4net.LogManager.GetLogger(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType);

      public override void OnException(ExceptionContext filterContext)
      {
          if (filterContext.Exception != null)
          {
            log.Error("Error in Controller", filterContext.Exception);
          }

          base.OnException(filterContext);
      }
   }

上面的代码能满足我的要求吗？ 如果没有，什么解决方案可以满足我的要求？

Answer 1

我仍然对所有不同的解决方案以及属性如何相互干扰感到有点困惑，但我采用了这个解决方案：

public class LogErrorsAttribute: FilterAttribute, IExceptionFilter
{
    #region IExceptionFilter Members

    void IExceptionFilter.OnException(ExceptionContext filterContext)
    {
        if (filterContext != null && filterContext.Exception != null)
        {
            string controller = filterContext.RouteData.Values["controller"].ToString();
            string action = filterContext.RouteData.Values["action"].ToString();
            string loggerName = string.Format("{0}Controller.{1}", controller, action);

            log4net.LogManager.GetLogger(loggerName).Error(string.Empty, filterContext.Exception);
        }

    }

    #endregion
}

我仍然使用原始问题中所解释的 [HandleError] 属性，并且我只是装饰每个控制器都有一个 [LogErrors] 属性。

这对我有用，因为它将错误记录保留在一个位置，并且不会导致多次记录重复的异常（如果我扩展 [HandleError] 并在多个位置使用该属性，则会发生这种情况）。

我认为不可能将异常日志记录和错误处理合并到一个属性或类中，而不会使它变得非常乏味和复杂，或影响 [HandleError] 的使用

但这对我有用，因为我装饰了每个控制器仅使用 [LogErrors] 属性一次，并按照我想要的方式使用 [HandleError] 装饰控制器和操作，而不会相互干扰。

更新：

这是我如何使用它的示例：

[LogErrors(Order = 0)]
[HandleError(Order = 99)]
public class ContactController : Controller
{
    public ActionResult Index()
    {
        return View(Views.Index);
    }

    public ActionResult Directions()
    {
        return View(Views.Directions);
    }


    public ActionResult ContactForm()
    {
        FormContactMessage formContactMessage = new FormContactMessage();

        return View(Views.ContactForm,formContactMessage);
    }

    [HandleError(ExceptionType = typeof(SmtpException), View = "MessageFailed", Order = 1)]
    [AcceptVerbs(HttpVerbs.Post)]
    public ActionResult ContactForm(FormContactMessage formContactMessage)
    {
        if (ModelState.IsValid)
        {
            if (formContactMessage.IsValid)
            {
                SmtpClient client = new SmtpClient();

                MailAddress recipientAddress = new MailAddress(Properties.Settings.Default.ContactFormRecipientEmailAddress);
                MailAddress senderAddress = new MailAddress(Properties.Settings.Default.ContactFormSenderEmailAddress);
                MailMessage mailMessage = formContactMessage.ToMailMessage(recipientAddress, senderAddress);

                client.Send(mailMessage);

                return View("MessageSent");
            }
            else
            {
                ModelState.AddRuleViolations(formContactMessage.GetRuleViolations());
            }
        }
        return View(Views.ContactForm, formContactMessage);
    }

    private static class Views
    {
        public static string Index { get { return "Index"; } }
        public static string Directions { get { return "Directions"; } }
        public static string ContactForm { get { return "ContactForm"; } }

    }
}

在上面的代码中，ContactForm操作重载中的SmtpExceptions以非常具体的方式处理 - 向用户呈现特定于发送失败的消息的 ViewPage ，在本例中称为“MessageFailed”。 所有其他异常均由 [HandleError] 的默认行为处理。 另请注意，首先记录错误，然后处理错误。 这由以下内容表示：

[LogErrors(Order = 0)]
[HandleError(Order = 99)]

更新：

对此有一个替代解决方案，并且有很好的解释。 我建议通读它以更好地理解所涉及的问题。

ASP.NET MVC HandleError 属性、自定义错误页面和日志记录异常
（感谢下面的 Scott Shepherd，他在下面的答案中提供了链接）。