为什么当它的 writer 被释放时，流也会被释放？

Question

考虑以下代码：

using (var ms = new MemoryStream())
{
    using(var writer = BinaryWriter(ms))
    {
        writer.Write(/*something*/);
        writer.Flush();
    }

    Assert.That(ms.Length > 0); // Throws ObjectDisposedException
}

一方面，一次性对象应该处置其资源；另一方面，我明白了，但另一方面，该对象没有创建并且不拥有该资源，它是提供的 ->调用代码应该对此负责...不是吗？

我想不出任何其他类似的情况，但是对于任何接收一次性对象并自行处理它们的类来说，这是框架中一致的模式吗？

Answer 1

有一个隐含的假设，即每个流只有一个写入器，因此为了方便起见，写入器假设该流的所有权 - 然后您显然需要清理一件事。

但我同意；但这并不总是正确的，而且常常带来不便。某些实现（例如 DeflateStream、GZipStream）允许您进行选择。否则，唯一真正的选择是在编写器和底层流之间注入一个虚拟流； IIRC Jon Skeet 的“MiscUtil”库中有一个 NonClosingStreamWrapper 可以实现此目的：http://www.yoda.arachsys。 com/csharp/miscutil/

用法如下：

using (var ms = new MemoryStream())
{
    using(var noClose = new NonClosingStreamWrapper(ms))
    using(var writer = BinaryWriter(noClose))
    {
        writer.Write(/*something*/);
        writer.Flush();
    }

    Assert.That(ms.Length > 0);
}

Answer 2

我完全同意你的观点。这不是一致的行为，但它是如何实现的。文档末尾有关于此的注释行为不是很直观。所有流编写者都只是获得底层流的所有权并处置它。就我个人而言，我总是像这样嵌套我的 using 语句：

using (var ms = new MemoryStream())
using(var writer = BinaryWriter(ms))
{
    writer.Write(/*something*/);
}

这样就不应该编写像您放入 Assert 中的代码。

Answer 3

正确的做法是让流编写器有一个构造函数参数，指示在构造函数释放时是否应释放流。鉴于 Microsoft 没有这样做，最好定义一个 NonDisposeStream(Of T as Stream) 类，该类包装流，但不将 Dispose 调用传递给包装的流。然后，可以将新的 NonDisposeStream 传递给 StreamWriter 的构造函数，并且底层流将不会被处置（当然，您自己处置该流是必要的）。

拥有一个可以处理传入对象的对象是很有用的。虽然这种行为与对象创建者处理其处置的通常模式并不相符，但在某些情况下，对象的创建者通常不知道该对象实际需要多长时间。例如，一个方法可能会创建一个使用新 Stream 的新 StreamWriter。 StreamWriter 的所有者将知道何时应该处置它，但可能不知道内部流的存在。内部流的创建者将不知道外部 StreamWriter 将使用多长时间。将流的所有权“移交给”StreamWriter 可以解决该特定（常见）情况下的处置问题。

Answer 4

我建议这个包装类：

public class BetterStreamWriter : StreamWriter
{
    private readonly bool _itShouldDisposeStream;

    public BetterStreamWriter(string filepath)
        :base(filepath)
    {
        _itShouldDisposeStream = true;
    }

    public BetterStreamWriter(Stream stream)
        : base(stream)
    {
        _itShouldDisposeStream = false;
    }

    protected override void Dispose(bool disposing)
    {
        base.Dispose(disposing && _itShouldDisposeStream);
    }
}

对象不应该处置它们没有实例化的东西。如果它是文件流编写器，则应该进行处理。如果它是外部流，则不应该。

一开始就不应该实现打开文件路径。这违反了单一职责原则，因为对象同时管理文件的写入和生命周期。