在静态方法中管理一次性对象

本文关键字：一次性 对象 管理 静态方法 | 更新日期: 2023-09-27 18:22:03

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。

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

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

在静态方法中管理一次性对象

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

  • 同步Write(以及对文件的任何其他访问(，因此只有一个调用方可以尝试一次打开该文件
  • 使用已经可以处理此场景的预制日志记录框架(此处的常见方法包括同步，但也包括：在本地缓冲数据，然后定期推送数据 - 避免一遍又一遍地打开文件(

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

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

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

private static object _objectLock = new object();

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

public static void Write(string msg)      
{        
    lock(_objectLock)
    {  
        using (var sw = GetStream())          
        {              
            sw.Write(msg);          
        }
    }      
}
相关文章：