如何正确关闭资源

Question

在清理一些代码时，FindBugs向我介绍了一些使用Connection，CallableStatement和ResultSet对象的JDBC代码。这是该代码的一个片段：

CallableStatement cStmt = getConnection().prepareCall("...");
...
ResultSet rs = cStmt.executeQuery();

while ( rs.next() )
{
    ...
}

cStmt.close();
rs.close();
con.close();

FindBugs指出这些应该在finally块内。我开始重构我的代码来做到这一点，我开始想知道如何在finally块中处理代码。

Connection对象的CallableStatement的创建可能会引发异常，而我的ResultSet对象保留为null。当我尝试关闭ResultSet时，我将收到NullPointerException，而我的Connection将永远不会关闭。确实，该线程提出了相同的概念，并表明将您的close（）调用包装在空检查中是一个好主意。

但是其他可能的例外呢？根据JavaAPI规范，Statement.close（）可能在“如果发生数据库错误”时抛出SQLException。因此，即使我的CallableStatement不为null且可以成功调用close（），我仍然可能会遇到异常，并且没有机会关闭其他资源。

我能想到的唯一“故障安全”解决方案是将每个close（）调用包装在其自己的try / catch块中，如下所示：

finally {

    try {
        cStmt.close();
    } catch (Exception e) { /* Intentionally Swallow  Exception */ }

    try {
        rs.close();
    } catch (Exception e) { /* Intentionally Swallow  Exception */ }

    try {
        con.close();
    } catch (Exception e) { /* Intentionally Swallow  Exception */ }

}

男孩，如果那看起来并不可怕。有更好的方法来解决这个问题吗？

Answer 1

我认为最好的答案已经被提及，但我想提到您可以考虑自动关闭资源的新JDK 7功能可能会很有趣。

try{
    try(Connection conn = DriverManager.getConnection("jdbc:mysql://localhost/hrdb", "obiwan", "kenobi"); 
        Statement stm = conn.createStatement(); 
        ResultSet rs = stm.executeQuery("select name from department")) {

        while(rs.next()){
            System.out.println(rs.getString("name"));
        }

    } 
}catch(SQLException e){
    //you might wanna check e.getSuppressed() as well
    //log, wrap, rethrow as desired.
}

并不是我们所有人现在都可以迁移到JDK7，但是对于那些可以开始使用开发人员预览版的人来说，这提供了一种有趣的方式，并且肯定会在不久的将来弃用其他方法。