如何正确关闭资源

Question

当我清理一些代码时，FindBugs 向我指出了一些使用 Connection、CallableStatement 和 ResultSet 对象的 JDBC 代码。这是该代码中的一个 sn-p：

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

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

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

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

创建 Connection 对象的 CallableStatement 可能会引发异常，使我的 ResultSet 对象为空。当我尝试关闭 ResultSet 时，我会收到 NullPointerException，而我的 Connection 将永远不会关闭。事实上，this thread 提出了相同的概念，并表明将您的 close() 调用包装在一个空检查中是一个好主意。

但是其他可能的例外情况呢？根据 Java API 规范，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.
}

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

Answer 2

如果可以，请使用Lombok's cleanup：

@Cleanup
Connection c = ...
@Cleanup
statement = c.prepareStatement(...);
@Cleanup
rs = statement.execute(...);

这可以转换为三个嵌套的 try-finally 块，并且可以正常工作，但有异常。 永远不要在没有充分理由的情况下吞下异常！

另一种选择：

像这样编写一个自己的实用程序方法：

public static void close(ResultSet rs, Statement stmt, Connection con) throws SQLException {
    try {
        try {
            if (rs!=null) rs.close();
        } finally {
            if (stmt!=null) stmt.close();
        }
    } finally {
        if (con!=null) con.close();
    }
}

并在

中使用它

try {
    Connection con = ...
    Statement stmt = ...
    ResultSet rs = ...
} finally {
    close(rs, stmt, con);
}

让异常冒泡或随心所欲地处理它。

Answer 3

嗯，基本上这就是你所做的，除了你首先不一定吞下异常（你可以空检查并至少记录异常）。其次，您可以设置一个不错的实用程序类，例如

public static void close(ResultSet rs) {
   try { if (rs != null) rs.close();
   } catch (SQLException (e) {
      log.error("",e);
   } 

}

然后你只需静态导入该类。

你终于变成了这样的东西

finally {
     close(resultset);
     close(statement);
     close(connection);
}

这真的没有那么可怕。

Answer 4

您只需关闭连接。

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

来自 docs.oracle.com：

Statement 对象在被垃圾回收时会自动关闭。当 Statement 对象关闭时，其当前的 ResultSet 对象（如果存在）也将关闭。

在 Connection 上调用 close() 会释放其数据库和 JDBC 资源。

Answer 5

我知道隐藏所有难看的 try-catch 样板代码的唯一方法是使用 Spring's JBDC Template 之类的东西。

Answer 6

您可以将一个块包装到另一个块中：

try{
  Connection c = ...
    try{
      statement = c.prepareStatement(...);
      try{
        rs = statement.execute(...);
      }finally{
        rs.close();
      }
    }finally{
      statement.close()
    }
  }finally{
    c.close();
  }
}catch(SQLException e){}

对可能出现的所有内容都使用最低的捕获块