java：我需要关闭所有流吗？

Question

我有一个从文件中读取文本的方法；可能需要解压，具体取决于输入参数：

public static String readText(File inFile, boolean compressed) {
    InputStream in = null;
    InputStreamReader isr = null;
    StringBuilder sb = new StringBuilder();//constant resizing is costly, so set the STRING_SIZE
    try {
        in = new FileInputStream(inFile);
        if (compressed) {
            in = new GZIPInputStream(in);
        }
        isr = new InputStreamReader(in);
        int length = 0;
        char[] cbuf = new char[8 * 1024];
        while ((length = isr.read(cbuf)) != -1) {
            sb.append(cbuf, 0, length);
        }
    } catch (Exception e) {
        e.printStackTrace();
    } finally {
        try {
            in.close();
        } catch (Exception e1) {
            e1.printStackTrace();
        }
    }
    return sb.toString();
}

有人建议我像这样使用InputStream，这样更容易编写，所以最后我只需要关闭一件事。我还是有点担心这可能会导致内存泄漏。所以我的问题是：有谁知道上面的代码是否可以？或者我是否必须返回一打流并在finally块中将它们一一关闭？

多谢。

Answer 1

是的，关闭最外面的流/读取器就足够了。

但是，您的代码还有另一个潜在的错误：new InputStreamReader(in) 将使用平台默认编码，这取决于操作系统区域/语言设置。您应该指定文本文件的编码并在构造函数中显式使用它。

Answer 2

这里有一点需要补充：在调用 'in.close()' 之前查看 'in' 是否为 null，因为在第一次分配成功的情况下可能会发生异常。

另外，仅捕获可能的异常（例如 IOException）也是一种很好的形式。这样，如果您添加更多代码并且 IDE 告诉您未处理新的异常类型，您可以添加适当的特定代码，而不是从未听说过它，因为最初用于 IOException 的 catch (Exception ) 也是（错误处理？ ）所有其他类型。

Answer 3

这是干净的 Java 7 方法，适用于任何实现 AutoCloseable/Closeable 的方法：

try (InputStream in = compressed ?
                        new GZIPInputStream(new FileInputStream(inFile))
                      : new FileInputStream(inFile);
     InputStreamReader isr = new InputStreamReader(in))
{
    int length = 0;
    char[] cbuf = new char[8 * 1024];
    while ((length = isr.read(cbuf)) != -1) {
        sb.append(cbuf, 0, length);
    }
}
catch (Exception e) {
    e.printStackTrace();
}

如果您想知道关闭资源时出现异常会发生什么，请阅读还添加的 getSuppressedExceptions() 。