我应该为流对象调用Close（）还是Dispose（）吗？

一尘不染

我应该为流对象调用Close（）还是Dispose（）吗？

类如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()方法：

如何：使用FTP下载文件

好吗？

阅读 277

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