我正在寻找一种简单的解决方案来在我的ASP.Net MVC 1.0应用程序中进行异常记录与错误处理相结合。
我已经阅读了很多文章,包括在此处发布在StackOverflow上的问题,它们都为不同情况提供了不同的解决方案。我仍然无法提出适合我需要的解决方案。
这是我的要求:
为了能够在我的Controller上使用[HandleError]属性(或等效的东西),处理所有可能从任何Action或View抛出的异常。这应该处理所有未在任何操作上明确处理的异常(如第2点所述)。对于控制器中的所有操作,我希望能够指定在错误情况下必须将用户重定向到哪个视图。
我希望能够在特定操作的顶部指定[HandleError]属性(或等效项),以捕获特定异常并将用户重定向到适合该异常的View。所有其他异常仍必须由Controller上的[HandleError]属性处理。
在以上两种情况下,我都希望使用log4net(或任何其他日志记录库)记录异常。
我如何实现以上目标?我已经读过有关使我的所有Controller都继承自覆盖OnException方法的基本控制器的信息,并且在其中进行日志记录。但是,这将使用户重定向到适当的视图,或使其变得混乱。
我已经读过编写自己的实现IExceptionFilter来处理此操作的Filter Action,但这将与[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);
}
}
以上代码是否可以满足我的要求?如果没有,哪种解决方案可以满足我的要求?
我仍然对所有不同的解决方案以及属性如何相互干扰感到困惑,但是我采用了这种解决方案:
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 Attribute, Custom Error Pages and Logging Exceptions(感谢下面的Scott Shepherd,他在下面的答案中提供了链接)。