Stream
、StreamReader
、StreamWriter
等类实现了IDisposable
接口。这意味着,我们可以在这些类的对象上调用Dispose()
方法。他们还定义了一个名为Close()
的public
方法。现在这让我很困惑,一旦我处理完对象,我应该调用什么呢?如果我把两个都叫来呢?
我当前的代码是:
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()
方法:
是不是很好?
发布于 2011-09-23 14:32:08
快速跳转到Reflector.NET,可以看到StreamWriter
上的Close()
方法是:
public override void Close()
{
this.Dispose(true);
GC.SuppressFinalize(this);
}
StreamReader
是:
public override void Close()
{
this.Dispose(true);
}
StreamReader
中的Dispose(bool disposing)
覆盖是:
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();
}
它不会影响代码的行为,但确实有助于提高可读性。
发布于 2011-09-23 14:06:37
不,您不应该手动调用这些方法。在using
块的末尾,将自动调用Dispose()
方法,该方法将负责释放非托管资源(至少对于标准的.NET BCL类,如流、读取器/写入器等)。所以你也可以像这样写你的代码:
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()
。
发布于 2011-09-23 14:32:11
文档中说这两种方法是等价的:
StreamReader.Close:此Close实现调用传递真值的Dispose方法。
StreamWriter.Close:Close的这个实现调用Dispose方法,传递一个真值。
Stream.Close:此方法调用Dispose,指定为true以释放所有资源。
因此,这两种方法都是有效的:
/* 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();
}
就我个人而言,我会坚持第一种选择,因为它包含的“噪音”较少。
https://stackoverflow.com/questions/7524903
复制相似问题