首页
学习
活动
专区
工具
TVP
发布
社区首页 >问答首页 >我应该为流对象调用Close()还是Dispose()?

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

Stack Overflow用户
提问于 2011-09-23 14:05:26
回答 6查看 154.1K关注 0票数 180

StreamStreamReaderStreamWriter等类实现了IDisposable接口。这意味着,我们可以在这些类的对象上调用Dispose()方法。他们还定义了一个名为Close()public方法。现在这让我很困惑,一旦我处理完对象,我应该调用什么呢?如果我把两个都叫来呢?

我当前的代码是:

代码语言:javascript
复制
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()方法:

是不是很好?

EN

回答 6

Stack Overflow用户

回答已采纳

发布于 2011-09-23 14:32:08

快速跳转到Reflector.NET,可以看到StreamWriter上的Close()方法是:

代码语言:javascript
复制
public override void Close()
{
    this.Dispose(true);
    GC.SuppressFinalize(this);
}

StreamReader是:

代码语言:javascript
复制
public override void Close()
{
    this.Dispose(true);
}

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

代码语言:javascript
复制
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#中,我们已经有了过多的右大括号,那么我们如何知道哪一个是在流上执行关闭的呢?

所以我认为最好这样做:

代码语言:javascript
复制
using (var stream = ...)
{
    /* code */

    stream.Close();
}

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

票数 125
EN

Stack Overflow用户

发布于 2011-09-23 14:06:37

不,您不应该手动调用这些方法。在using块的末尾,将自动调用Dispose()方法,该方法将负责释放非托管资源(至少对于标准的.NET BCL类,如流、读取器/写入器等)。所以你也可以像这样写你的代码:

代码语言:javascript
复制
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);
                 }
            }
         }

Close()方法调用Dispose()

票数 56
EN

Stack Overflow用户

发布于 2011-09-23 14:32:11

文档中说这两种方法是等价的:

StreamReader.Close:此Close实现调用传递真值的Dispose方法。

StreamWriter.Close:Close的这个实现调用Dispose方法,传递一个真值。

Stream.Close:此方法调用Dispose,指定为true以释放所有资源。

因此,这两种方法都是有效的:

代码语言:javascript
复制
/* Option 1, implicitly calling Dispose */
using (StreamWriter writer = new StreamWriter(filename)) { 
   // do something
} 

/* Option 2, explicitly calling Close */
StreamWriter writer = new StreamWriter(filename)
try {
    // do something
}
finally {
    writer.Close();
}

就我个人而言,我会坚持第一种选择,因为它包含的“噪音”较少。

票数 18
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/7524903

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档