连接关闭时结果集未关闭？

Question

我一直在对我们的一个宠物项目进行代码审查（主要使用 FindBugs 等工具），并且 FindBugs 将以下代码标记为错误（伪代码）：

Connection conn = dataSource.getConnection();

try{
    PreparedStatement stmt = conn.prepareStatement();
    //initialize the statement
    stmt.execute();
    ResultSet rs =  stmt.getResultSet();
    //get data
}finally{
    conn.close();
}

错误是此代码可能不会释放资源。我发现 ResultSet 和 Statement 没有关闭，所以我最终关闭了它们：

finally{
    try{
        rs.close()
    }catch(SqlException se){
        //log it
    }
    try{
        stmt.close();
    }catch(SqlException se){
        //log it
    }
    conn.close();
}

但我在很多项目（来自不少公司）中都遇到了上述模式，并且没有人关闭 ResultSets 或 Statements。

当连接关闭时，您是否遇到过结果集和语句未关闭的问题？

我只找到了this，它指的是 Oracle 在关闭连接时遇到关闭 ResultSets 的问题（我们使用 Oracle db，因此我进行了更正）。 java.sql.api 在 Connection.close() javadoc 中什么也没说。

Answer 1

我在 Oracle 中遇到了未关闭的 ResultSets 的问题，即使连接已关闭。我得到的错误是

"ORA-01000: maximum open cursors exceeded"

所以：总是关闭你的 ResultSet！

Answer 2

只关闭连接而不关闭结果集的一个问题是，如果您的连接管理代码使用连接池，connection.close() 只会将连接放回池中。此外，某些数据库在服务器上有游标资源，除非显式关闭，否则无法正确释放。

Answer 3

我在一个大型 J2EE Web 环境中工作。我们有几个数据库可以在一个请求中连接。我们的一些应用程序开始出现逻辑死锁。问题如下：

1. 用户会请求页面
2. 服务器连接到 DB 1
3. DB 1 上的服务器选择
4. 服务器“关闭”与 DB 1 的连接
5. 服务器连接到 DB 2
6. 陷入僵局！

发生这种情况有两个原因，我们遇到的流量远高于正常情况，并且 J2EE 规范默认情况下不会真正关闭您的连接，直到线程完成执行。因此，在上面的示例中，步骤 4 从未真正关闭连接，即使它们在 finally 中正确关闭。

要解决此问题，您必须在 web.xml 中为数据库连接使用资源引用，并且必须将 res-sharing-scope 设置为不可共享。

例子：

<resource-ref>
    <description>My Database</description>
    <res-ref-name>jdbc/jndi/pathtodatasource</res-ref-name>
    <res-type>javax.sql.DataSource</res-type>
    <res-auth>Container</res-auth>
    <res-sharing-scope>Unshareable</res-sharing-scope>
</resource-ref>

Answer 4

您应该始终明确关闭所有 JDBC 资源。正如 Aaron 和 John 已经说过的，关闭连接通常只会将其返回到池中，并且并非所有 JDBC 驱动程序都以完全相同的方式实现。

这是一个可以在 finally 块中使用的实用方法：

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

Answer 5

ODBC 桥可能会在某些 ODBC 驱动程序中产生内存泄漏。

如果您使用良好的 JDBC 驱动程序，那么关闭连接应该不会有任何问题。但是有两个问题：

• 你知道你有没有好的司机吗？
• 您将来会使用其他 JDBC 驱动程序吗？

最好的做法是全部关闭。

Answer 6

在 Java 中，语句（不是结果集）与 Oracle 中的游标相关联。最好关闭您打开的资源，因为 JVM 和系统资源可能会发生意外行为。

此外，一些 JDBC 池框架将语句和连接池化，因此不关闭它们可能不会将这些对象标记为池中的空闲对象，并导致框架出现性能问题。

一般来说，如果一个对象上有一个 close() 或 destroy() 方法，那么调用它是有原因的，如果忽略它，后果自负。

Answer 7

我确实看到了未关闭的 ResultSets 的问题，一直关闭它们有什么坏处，对吧？需要记住这样做的不可靠性是迁移到为您管理这些细节的框架的最佳理由之一。这在您的开发环境中可能不可行，但我很幸运使用 Spring 来管理 JPA 事务。打开连接、语句、结果集以及编写过于复杂的 try/catch/finally 块（在 finally 块中使用 try/catch 块！）以再次关闭它们的混乱细节就消失了，只剩下你实际上完成了一些工作。我强烈建议迁移到这种解决方案。

Answer 8

在这种情况下，Oracle 会给你关于打开游标的错误。

根据：http://java.sun.com/javase/6/docs/api/java/sql/Statement.html

看起来重用语句会关闭所有打开的结果集，关闭语句会关闭所有结果集，但我没有看到任何关于关闭连接会关闭它创建的任何资源的信息。

所有这些细节都留给 JDBC 驱动程序提供者。

明确关闭所有内容总是最安全的。我们编写了一个 util 类，它使用 try{ xxx } catch (Throwable {} 包装所有内容，这样您就可以调用 Utils.close(rs) 和 Utils.close(stmt) 等，而不必担心关闭扫描可能会抛出的异常.