c# - Streamwriter 锁不工作

Question

我正在接管一个 C# 项目，在对其进行测试时出现错误。错误是无法写入日志文件，因为它正在被另一个进程使用。这是代码:

    public void WriteToLog(string msg)
    {
        if (!_LogExists) 
        {
            this.VerifyOrCreateLogFile();  // Creates log file if it does not already exist.
        }

        // do the actual writing on its own thread so execution control can immediately return to the calling routine.
        Thread t = new Thread(new ParameterizedThreadStart(WriteToLog));
        t.Start((object)msg);
    }

    private void WriteToLog(object msg)
    {
        lock (_LogLock)
        {
            string message = msg as string;
            using (StreamWriter sw = File.AppendText(LogFile))
            {
                sw.Write(message);
                sw.Close();
            }
        }
    }

_LogLock 定义为类变量:

    private object _LogLock = 0;

根据我的研究以及几年来这在生产系统中运行良好的事实，我不知道问题出在哪里。该锁应防止另一个线程尝试写入日志文件。

我所做的需要测试的更改是更多的日志使用。我们基本上是在添加 Debug模式，以便在日志中保存比以往更多的信息。

感谢您的帮助!

编辑:

感谢您的快速解答! VerifyOrCreateLogFile() 的代码确实使用了 _LogLock，因此这应该不是问题。它确实会在出错之前对日志进行一些写入，因此它可以顺利创建文件。

问题似乎是以前只有一个类创建了日志类的实例，现在我已经将实例添加到其他类。这会产生问题是有道理的。将 _LogLock 字段更改为静态可解决此问题。

再次感谢!

Answer 1

The lock should prevent another thread from attempting to write to the log file.

仅当您使用此类的单个实例时才适用。

如果每个(甚至某些)日志请求使用一个单独的实例，那么锁就不会保护你。

您可以通过将 _LogLock 字段设置为 static 轻松“纠正”此问题:

private static object _LogLock = 0;

这样，所有实例将共享相同锁。