如何正确地处理同时循环中的连接、结果集和语句对象

Question

如果我有以下代码，这是关闭Connection、ResultSet和语句对象的正确方式吗？我觉得所有对关闭（）的调用都应该在最终块中。

Connection con = null;
ResultSet rs = null;
Statement stmt = null;

try{
    //Code before the while loop 
    con = DriveManager.getConnection("Stuff");

    while(someBoolean){          

        stmt = con.createStatement();
        rs = stmt.executeQuery("SQL query");

        // do stuff with query results.

        if( rs != null){
               rs.close();
        }

        if( stmt != null){
               stmt.close();
        }

} //end while

    if( con != null ){
        con.close();
    }

catch (Exception e){
    //handle exception
}

Answer 1

在Java 7中，可以使用try with resource语句：

try(Connection con = getConnection(url, username, password, "drivername");
Statement stmt = con.createStatement();
ResultSet rs = stmt.executeQuery(sql);
) {

  //statements
}catch(....){}

在Java 6中，您可以最后关闭块中的资源：

} finally {
   try { rs.close(); } catch (Exception e) {  }
   try { ps.close(); } catch (Exception e) {  }
   try { conn.close(); } catch (Exception e) {  }
}

您甚至可以使用帮助类来关闭连接。Apache Commons DbUtils有一个DbUtils类。

} finally {
    DbUtil.closeQuietly(rs);
    DbUtil.closeQuietly(ps);
    DbUtil.closeQuietly(conn);
}

Answer 2

您不需要在循环中创建语句：您可以重用它。此外，使用Java7尝试使用资源（教程），您不需要处理以正确顺序关闭的单调乏味等。

你可以这样做：

try (
    Connection con = DriverManager.getConnection("Stuff");
    Statement stmt = con.createStatement();
){
    while(someBoolean){          
        try (ResultSet rs = stmt.executeQuery("SQL query")) {
            // do stuff with query results.
        }
    } //end while
} catch (Exception e){
    //handle exception
}

如您所见，与自己管理它相比，它需要的检查和语句要少得多，同时它确保以正确的顺序关闭资源，即使发生异常（即使在关闭其他资源之一时发生该异常）。例如，如果发生任何异常，您的旧代码无法关闭资源。

Answer 3

是的，关闭资源应该在一个finally块中，因为无论在哪里抛出异常，都应该关闭所有资源。

标准图案为：

Connection con = null;
ResultSet rs = null;
Statement stmt = null;

try {
    con = DriveManager.getConnection("Stuff");
    stmt = con.createStatement();
    rs = stmt.executeQuery("SQL query");
    // do stuff with query results
} catch (SQLException e) { // Don't catch Exception, catch what you expect
    // handle exception
} finally {
    // each close can itself explode, so wrap each in try catch
    try {
       if (rs != null)
           rs.close();
    } catch (SQLException ignore) {} // no point handling

    try {
       if (stmt != null)
           stmt.close();
    } catch (SQLException ignore) {} // no point handling

    try {
       if (con != null)
           con.close();
    } catch (SQLException ignore) {} // no point handling
}

虽然未能关闭其中一个资源可能意味着其他资源也将爆炸，但它尝试关闭每一个仍然是很好的做法，因此每个人都会在最后一个区块内尝试捕捉。