关闭 FileStream 会关闭 StreamReader 吗？

Question

如果我使用 FileStream 创建 StreamReader，当我关闭 FileStream 时 StreamReader 会关闭还是我也需要关闭 StreamReader？

public void ReadFile()
{
    var file = new FileStream("c:\file.txt", FileMode.Open, FileAccess.Read);
    var reader = new StreamReader(file);

    try
    {
        txtFile.Text = reader.ReadToEnd();
    }
    catch (Exception)
    {
        throw;
    }
    finally
    {
        file.Close();
    }
}

Answer 1

基本上是的。您实际上不必关闭 StreamReader。如果这样做，它所做的只是关闭底层流。

@Bruno 关于关闭最外层包装器提出了一个很好的观点。最好关闭最外层流并让它关闭底层流，以确保正确释放所有资源。

从反光镜...

public class StreamReader : TextReader
{
    public override void Close()
    {
        this.Dispose(true);
    }

    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;
                this.encoding = null;
                this.decoder = null;
                this.byteBuffer = null;
                this.charBuffer = null;
                this.charPos = 0;
                this.charLen = 0;
                base.Dispose(disposing);
            }
        }
    }
}

Answer 2

不可以。您应该关闭阅读器。实际上，这可能不会出现任何问题，但是 StreamReader 可能会增加一些可能需要清理的开销。所以你应该总是关闭最上面的包装。

Answer 3

您也可以只使用 File.ReadAllText 方法：

txtFile.Text = File.ReadAllText(@"c:\file.txt");

Answer 4

您不需要关闭 StreamReader，因为它不拥有任何非托管资源。关闭 FileStream 就足够了。您可以使用 using 重写您的代码，如下所示：

public void ReadFile()
{
    using (var file = new FileStream("c:\file.txt", FileMode.Open, FileAccess.Read))
    {
        txtFile.Text = new StreamReader(file).ReadToEnd();
    }
}

一般来说，如果您有疑问，最好确保安全，并在使用完所有 IDisposable 对象后释放它们。

public void ReadFile()
{
    using (FileStream file = new FileStream("c:\file.txt", FileMode.Open, FileAccess.Read))
    {
        using (StreamReader streamReader = new StreamReader(file))
        {
            txtFile.Text = streamReader.ReadToEnd();
        }
    }
}

Answer 5

不。最好的办法是按照打开它们的相反顺序关闭它们。

Answer 6

在我看来，总体上做到这一点的最佳方法是让 FileStream 仅关闭其自身。它并不隐式地了解其自身之上的层中存在的任何内容，因此它执行任何会影响这些更高层的操作实际上都是错误的。

话虽如此，更高级别的构造也不应该公理地假设任何有关任何提供的底层的任何内容，或者如果他们这样做，他们应该明确地这样做：

1）如果它是从现有流创建的，则更高级别的构造构造应该能够独立关闭底层流（实际上只是处理它分配给自己使用的任何资源），或者关闭包括底层流。这些应该是两个不同的函数调用，例如 Close() 和 CloseSelf() （如果要以向后兼容现有代码的方式实现）。

2）如果它不是从现有流创建的（也就是说，构造函数必须创建底层流），那么关闭更高级别的构造也应该强制底层流关闭，因为在这种情况下底层流是高层结构的隐含部分。在这种情况下，CloseSelf() 将简单地调用 Close()。

以这种方式实现这些类似乎很浪费。如果您计划使用相同的文件（例如）串行输入和串行输出，并且您希望获得对后代类的更高级别功能的访问权限，系统实际上会强制您将其视为两个不同的实体。您的替代方案是坚持较低级别的构造并自己实现较高级别的功能 - 有效地重新实现您自己的已存在的后代类的特殊版本。

如果按照上述方式完成，典型的功能将像现在一样简单地实现，但对于更复杂的应用程序，人们将保留在文件上放置单个锁并在需要时根据需要重新调整其用途的能力，如下所示：与必须放弃锁和所有相关资源然后立即重新分配它们相反 - 在没有任何正当理由的情况下增加系统开销和内存碎片。

但在现有条件下，正确的做法是显而易见的。不能假设 FileStream 了解有关它所属的任何对象的任何信息，因此您必须关闭最外层的封闭构造。正如布鲁诺等人所指出的，无论它是否以任何方式工作，这都适用，并且出于他们给出的原因 - 兼容性。假设是最丑陋的 bug 的曾祖父。

Answer 7

有趣的是，关闭 StreamReader 或 writer 会影响所属 FileStream 的读/写状态。这似乎意味着您不能使用 StreamReader，然后使用相同的文件流使用 StreamWriter。