在我正在使用的MVC3 Web应用程序中
public static void RegisterGlobalFilters(GlobalFilterCollection filters) { filters.Add(new HandleErrorAttribute()); }
应用全局错误处理,如果发生未处理的异常,用户将显示“错误”视图.
对于一个特定的视图,如果通过使用[HandleError(View =“SpecialError”)]装饰方法发生未处理的异常,我还想要显示一个不同的错误视图.这工作正常
然后我想添加全局日志记录未处理的异常.我创建了一个具有日志记录代码的自定义HandleError属性:
public class MyHandleErrorAttribute : HandleErrorAttribute { public override void OnException(ExceptionContext context) { // Write to log code base.OnException(context); } }
并更新了RegisterGlobalFilters和方法装饰,以改用此属性名称.这通常工作,但是当使用MyHandleError(View =“SpecialError”)装饰的方法中发生异常时,OnException方法被调用两次.我原来假设用这个属性来装饰方法取代了全局处理程序,但是它似乎只是被添加到(这更有意义,但不是我想要的).通过调用OnException两次,同样的异常记录两次,这不会发生.我不认为OnException被调用两次,因为它是一个自定义属性 – 我相信这也发生在标准的HandleError属性,它现在可以看到,因为我正在创建一个记录.
最终,我想记录所有未处理的异常(一次),同时保留[HandleError]提供的功能,特别是为特定的方法异常设置不同的视图.有没有干净的方法呢?
解决方法
我相信我自己找到了一个干净的解决方案.扩展HandleError似乎是一个好主意,但现在我认为这是错误方向的一步.我不想处理任何错误,只是在HandleError接收到之前写入异常记录一次.因此,默认的HandleError可以原样保留.虽然OnException可以被多次调用,但它似乎在HandleErrorAttribute的标准实现中是完全良性的.
而是创建了一个异常记录过滤器:
public class LoggedExceptionFilter : IExceptionFilter { public void OnException(ExceptionContext filterContext) { // logging code } }
它不需要太过继承FilterAttribute,因为它刚刚在RegisterGlobalFilters中与HandleErrorAttribute一起注册.
public static void RegisterGlobalFilters(GlobalFilterCollection filters) { filters.Add(new LoggedExceptionFilter()); filters.Add(new HandleErrorAttribute()); }
这允许在不改变标准[HandleError]功能的情况下整理记录异常