使用 Elmah 进行异常处理

Question

我用 Elmah 记录异常，想知道我使用的技术是否是好的设计？

现在，我捕获并重新抛出各种类和方法中发生的异常，并将它们记录到程序的主 try-catch 块中的 Elmah。

// 主程序

try
{
   // Some code that fires off other classes, etc...
   MyTestClass myTestClass = new MyTestClass();
   myTestClass.Execute();   

}
catch(Exception ex)
{
   ErrorSignal.FromCurrentContext().Raise(ex);
}

// MyTestClass

    public class MyTestClass
    {
        public object ApiResult { get; set; }


        public string Execute()
        {            
            try
            {
                // execute some code
                // ....
                // set xml message
                ApiResult = "User information xml response";
            }
            catch (Exception ex)
            {
                // set xml message
                ApiResult = "something went wrong xml error response...";

                throw;
            }
        }

    }

将异常记录在发生的地方会更好吗？另一个问题，我应该记录可以在不捕获异常的情况下处理的错误吗？例如，如果某些内容为空，我应该对此进行测试（如果为空...）并在 Elmah 中记录一条消息吗？

Answer 1

您应该努力让 ELMAH 自动为您记录错误，而不是使用 Elmah 的 ErrorSignal 类手动记录错误，这会在引发应用程序的 Error 事件时发生。

在您的示例中，主程序存在严重问题。也就是说，它吞噬了异常，至少对于最终用户而言是如此。是的，例外情况是登录 ELMAH，但您向用户隐藏了错误。最终用户会认为她的表单提交（或其他任何内容）没有错误，但实际上存在严重问题。

简而言之，应该谨慎使用 try...catch 块，例如在可以从错误中恢复的情况下，或者错误是“次要”错误并且不应停止工作流程时。但大多数错误都是真正的阻碍，并且没有优雅的解决方法。对于大多数人来说，您希望让错误渗透到 ASP.NET 运行时，ELMAH 将在其中自动记录错误，并且用户将在其中看到错误页面，提醒他们发生了错误。

查看我的这篇文章：ASP 的异常处理建议.NET Web 应用程序。

Answer 2

我认为以下情况会让您了解何时使用 ErrorSignal.FromCurrentContext().Raise(ex);

var bo = new CustomerBO();
bo.Update(customer);
try
{
   Email.SendProfileChangedNotification();
}
catch(Exception ex)
{
   ErrorSignal.FromCurrentContext().Raise(ex);
}
Response.Redirect(Constants.ProfilePage);

Email.SendProfileChangedNotification 方法在这段代码中并不是那么重要，我的意思是，如果它有任何错误，我可以保留它，并且我不想将它们显示给用户。重要的是，他/她的个人资料已更新，并且用户可以通过查看个人资料页面来了解这一点。

所以我相信代码中的某些地方可能会不断失败，我想收到有关它们的通知，但我不想分解整个操作。