FindBugs：“可能无法关闭流”，如何解决？

Question

以下代码被FindBugs标记为错误。 FindBugs说“这种方法可能无法清理（关闭，处理）流，数据库对象或需要显式清理操作的其他资源。” 错误标记在output = new FileOutputStream (localFile);行output = new FileOutputStream (localFile);

但是我们已经在块中添加了try / finally。

inputStream   input =null;
OutputStream output =null;
try {
    input = zipFile.getInputStream(entry);
    File localFile = new File(unzipFile.getAbsolutePath()
            + File.separator + entryName);
    output = new FileOutputStream (localFile);  // BUG MARKED HERE
    byte[] buffer = new byte[1024 * 8];
    int readLen = 0;
    while ((readLen = input.read(buffer, 0, 1024 * 8)) != -1) {
        output.write(buffer, 0, readLen);
    }
    output.flush();
    output.close();
    input.close();

} finally {
    if(output!=null) {
        output.flush();
        output.close();
    }
    if(input!=null) {
        input.close();
    }
}

Answer 1

删除位于try块中的所有close()调用（以及最后一次flush()调用），因为如果出现错误，将无法访问该代码。 这就是警告的来源。 在那里触发Findbugs相信你试图在try / catch块中处理它们。

另外在finally你最好包装另一个try catch块，以便你可以处理那里的错误，如果output结束失败，你仍然可以继续关闭input 。

                   // your code here
                   output.write(buffer, 0, readLen);
                   // removed these since findbug complains about this
                }
            } finally {
                // TODO some more try catching here so that if output closing 
                // fails you can still try to close second one and log the errors!
                if(output!=null)
                {
                    output.flush();
                    output.close();
                }
                if(input!=null)
                {
                    input.close();
                }
            }

对于java 7及更高版本，还有更好的解决方案。 请参阅http://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html （感谢@ user2864740获取该链接）。