小编典典

我应该为流对象调用 Close() 或 Dispose() 吗?

all

Stream,StreamReader等类StreamWriter实现IDisposable接口。这意味着,我们可以Dispose()在这些类的对象上调用方法。他们还定义了一个public名为Close().
现在这让我感到困惑,一旦我完成了对象,我应该调用什么?如果我同时打电话怎么办?

我目前的代码是这样的:

using (Stream responseStream = response.GetResponseStream())
{
   using (StreamReader reader = new StreamReader(responseStream))
   {
      using (StreamWriter writer = new StreamWriter(filename))
      {
         int chunkSize = 1024;
         while (!reader.EndOfStream)
         {
            char[] buffer = new char[chunkSize];
            int count = reader.Read(buffer, 0, chunkSize);
            if (count != 0)
            {
               writer.Write(buffer, 0, count);
            }
         }
         writer.Close();
      }
      reader.Close();
   }
}

如您所见,我编写using()了自动调用Dispose()每个对象的方法的构造。但我也称Close()方法。这样对吗?

请建议我使用流对象时的最佳实践。:-)

MSDN 示例不使用using()构造和调用Close()方法:

好吗?


阅读 75

收藏
2022-08-05

共1个答案

小编典典

快速跳转到 Reflector.NET 显示Close()on 方法StreamWriter是:

public override void Close()
{
    this.Dispose(true);
    GC.SuppressFinalize(this);
}

并且StreamReader是:

public override void Close()
{
    this.Dispose(true);
}

中的Dispose(bool disposing)覆盖StreamReader是:

protected override void Dispose(bool disposing)
{
    try
    {
        if ((this.Closable && disposing) && (this.stream != null))
        {
            this.stream.Close();
        }
    }
    finally
    {
        if (this.Closable && (this.stream != null))
        {
            this.stream = null;
            /* deleted for brevity */
            base.Dispose(disposing);
        }
    }
}

StreamWriter方法类似。

因此,阅读代码很明显,您可以根据需要以任何顺序在流上调用Close()& 。Dispose()它不会以任何方式改变行为。

因此,归结为使用Dispose(),Close()和/或using ( ... ) { ... }.

我个人的偏好是using ( ... ) { ... }应该尽可能使用它,因为它可以帮助你“不要用剪刀跑”。

但是,虽然这有助于正确性,但它确实降低了可读性。在 C# 中,我们已经有过多的右花括号,那么我们如何知道哪一个实际上在流上执行了关闭?

所以我认为最好这样做:

using (var stream = ...)
{
    /* code */

    stream.Close();
}

它不会影响代码的行为,但确实有助于提高可读性。

2022-08-05