3 
public class SimpleLogger
{
    static readonly string logFile = ConfigurationManager.AppSettings["LogFile"];

    static StreamWriter GetStream()
    {
        return File.Exists(logFile) ?
            File.AppendText(logFile) : File.CreateText(logFile);
    }

    public static void Write(string msg)
    {
        using (var sw = GetStream())
        {
            sw.Write(msg);
        }
    }
}

上面的代码使用失败，因为它似乎没有正确关闭/处理流。随后的写入会给出“正在使用的文件”IOException。

如果将类修改为使用非静态方法，它似乎可以正常工作。

我不明白为什么会有任何行为差异？

4

2 回答 2

7

处置很好；GetStream提供一个开放的作家；Write关闭/处理它 - 排序。但是，如果我不得不猜测，问题是并发使用 - 即多个线程（特别是在 Web 应用程序中）同时访问文件。如果是这种情况，选项：

  • 使Write（以及对文件的任何其他访问）同步，因此只有一个调用者可以尝试一次打开文件
  • 使用已经可以处理这种情况的预制日志框架（这里的常用方法包括同步，但也：在本地缓冲数据，然后定期将数据向下推送 - 避免一遍又一遍地打开文件）

尤其; 您唯一的静态状态是文件路径本身。因此，将其用作静态方法与实例方法之间没有显着差异。

作为旁注，File.AppendAllText在这里可能有用，但不能避免并发问题。

于 2012-02-01T14:36:29.767 回答
3

我认为从静态更改为实例不会解决问题，因为它们最终都在争夺静态资源（文件）。这个答案可能会对你有所帮助。也许如果您将这两种方法都保留为静态并声明一个静态同步对象以调用线程以锁定（因为资源本身是静态的）会有所帮助？例如：

private static object _objectLock = new object();

用于同步从多个线程访问文件，因此：

public static void Write(string msg)      
{        
    lock(_objectLock)
    {  
        using (var sw = GetStream())          
        {              
            sw.Write(msg);          
        }
    }      
}
于 2012-02-01T14:57:36.220 回答