1

我的团队正在开发一个 C# ASP.NET Web 应用程序。我决定通过写入错误日志文件来处理错误。

我编写的 log.Error 方法将记录到错误目录中的物理日志文件中。

我认为在执行 log.Error 方法时使用锁来控制线程是合适的,因为我想在每次发生错误时记录错误。

因此,这是我的大多数异常处理的格式:

protected void blahblahCsharpMethodBlahBlah()
{
try
{
    blah blah C# code blah blah
}
catch(Exception ex  ){
    lock (_objectblahblahCsharpMethodBlahBlah)
    {
        // The following log.Error method that I wrote would log to a physical log file in an error directory.
        log.Error(ex.ToString(),
                  PerlsPivotErrorDirectory,
                  System.Reflection.MethodBase.GetCurrentMethod().Name,
                  this.GetType().Name,    System.IO.Path.GetFileName(System.Reflection.Assembly.GetExecutingAssembly().Location));
}
}
} // protected void blahblahCsharpMethodBlahBlah()

使用锁来控制线程以便它们以正确的序列化方式执行 log.Error 方法是否正确?

4

1 回答 1

4

如果要确保线程安全,则需要使用锁。但为了让它更简单、更防弹,我会锁定在 Error 方法中。通常,您在应用程序的不同类之间共享相同的日志对象(或至少相同的日志文件),因此直接登录您的日志记录(此处为错误)函数更有意义。

但你不想或不需要重新发明轮子。使用默认线程安全的NLog 。

于 2012-09-24T21:50:36.203 回答