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;

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

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

感谢您的帮助！

编辑：

感谢您的快速解答！ verifyOrCreateLogFile() 的代码确实使用了 _LogLock，因此这不应该成为问题。它确实在出错之前对日志进行了一些写入，因此它可以很好地完成创建文件的任务。

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

再次感谢！

Answer 1

锁应该防止另一个线程尝试写入日志文件。

仅当您使用此类的单个实例时，这才是正确的。

如果每个（甚至某些）日志请求使用单独的实例，那么锁将无法保护您。

您可以通过将 _LogLock 字段设为静态来轻松“纠正”此问题：

private static object _LogLock = 0;

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

Answer 2

我发现代码有两个问题：

• Log 类的所有“用户”之间的锁必须相同，最简单的解决方案是使 _LogLock 或完整的类 static
• 如果 2 个或更多并行线程在 _LogExists 为 false 时调用 WriteToLog，VerifyOrCreateLogFile 可能会出现问题...

Answer 3

一种可能性是，在退出 WriteToLog 中的锁之前，操作系统没有足够快地释放文件锁，并且另一个被阻塞等待锁的线程试图打开它在操作系统完成释放文件锁之前。是的，这有可能发生。您要么需要在尝试打开文件之前睡一会儿，将日志写入集中到专用对象（这样他并且只有他可以访问该文件，并且您不必担心文件锁争用） 。

另一种可能性是您需要锁定

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

第三种可能性是您拥有容纳这些方法的任何类的多个实例。锁对象不会在实例之间共享（将其设置为静态以解决此问题）。

归根结底，除非您是编写安全多线程代码的专家，否则就让其他人为您担心这些事情吧。使用一个可以为您处理这些问题的框架（log4net？）。

Answer 4

即可使代码可执行

您只需删除 sw.Close() ；从你的代码......

做它......
它会工作得很好......