小编典典

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

c#

类如StreamStreamReaderStreamWriter等工具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()方法:

好吗?


阅读 329

收藏
2020-05-19

共1个答案

小编典典

快速跳转到Reflector.NET可以看到打开的Close()方法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();
}

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

2020-05-19