Eclipse警告潜在的资源泄漏尽管我有一个finally块关闭了最外面的流，但我缺少什么？

Question

Eclipse给我以下的资源泄漏警告有什么原因吗：resource leak：'br'is never close“？我所说的代码在本文的底部。

我以为我的最后一个区块已经把它都覆盖了，我的推理是：

• 只有当FileInputStream构造函数抛出时，res才会为null，因此不需要关闭任何东西
• 如果InputStreamReader构造函数抛出（例如，格式错误的编码字符串），则Res将是inputstream，然后只有inputstream必须关闭，所以OK
• 等..

那我错过了什么？或者这可能是一个eclipse bug？

亲切的问候！

S.

public static String fileToString(String fileName, String encoding) throws IOException {
    InputStream is;
    InputStreamReader isr;
    BufferedReader br;
    Closeable res = null;
    try {
        is = new FileInputStream(fileName);
        res = is;
        isr = new InputStreamReader(is, encoding);
        res = isr;
        br = new BufferedReader(isr);
        res = br;
        StringBuilder builder = new StringBuilder();
        String line = null;
        while ((line = br.readLine()) != null) {
            builder.append(line);
            builder.append(LS);
        }
        return builder.toString();
    } finally {
        if (res != null) {
            res.close();
        }
    }
}

Answer 1

Eclipse可能只是不理解您对res变量所做的调整。

我推荐使用try-with-resources语句（在Java ；7及更高版本中可用，现在已经有三年半了），它极大地简化了这些类型的链：

public static String fileToString(String fileName, String encoding) throws IOException {

    try (
        InputStream is = new FileInputStream(fileName);
        InputStreamReader isr = new InputStreamReader(is, encoding);
        BufferedReader br = new BufferedReader(isr)
    ) {
        StringBuilder builder = new StringBuilder();
        String line = null;
        while ((line = br.readLine()) != null) {
            builder.append(line);
            builder.append(LS);
        }
        return builder.toString();
    }
}

如果您不能使用try-with-resources，那么您可能需要类似于Apache Commons的ioutils类的closequiete方法（可以是该方法，也可以是您自己的方法），而不是将res到处乱放，这样读起来会很尴尬，而且我敢说很容易出现维护问题。

使用IOUTILS可能如下所示：

public static String fileToString(String fileName, String encoding) throws IOException {
    InputStream is = null;
    InputStreamReader isr = null;
    BufferedReader br = null;

    try {
        is = new FileInputStream(fileName);
        isr = new InputStreamReader(is, encoding);
        br = new BufferedReader(isr)
        StringBuilder builder = new StringBuilder();
        String line = null;
        while ((line = br.readLine()) != null) {
            builder.append(line);
            builder.append(LS);
        }
        br.close();
        return builder.toString();
    }
    finally {
        IOUtils.closeQuietly(br, isr, is);
    }
}

请注意我是如何在try中使用普通的close的，但然后确保在finally中进行清理的。

但是尝试使用资源是更好的答案，因为它更简洁，并且与新的（ish）“被抑制的异常”相关联。

附带说明：行的=null初始化没有任何原因，您可以在下一行指定它。

边注2：如果文件很可能是任意大小的，请考虑提前找出它有多大，并在构造函数中设置StringBuilder的容量。StringBuilder的默认容量是16，因此即使是几百html" target="_blank">字节的文件也需要重新分配StringBuilder的内部缓冲区。