JDBC结果集和语句必须分开关闭，尽管连接随后关闭?

据说在使用完JDBC资源后关闭所有资源是一个好习惯。但是如果我有下面的代码，是否有必要关闭Resultset和Statement?

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    try { if (rs != null) rs.close(); } catch (Exception e) {};
    try { if (stmt != null) stmt.close(); } catch (Exception e) {};
    try { if (conn != null) conn.close(); } catch (Exception e) {};
}

问题是关闭连接是否有效，或者是否会留下一些可用的资源。

当前回答

如果你想要更紧凑的代码，我建议使用Apache Commons DbUtils。在这种情况下:

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    DbUtils.closeQuietly(rs);
    DbUtils.closeQuietly(stmt);
    DbUtils.closeQuietly(conn);
}

2016-01-13 07:30:50

其他回答

我创建了以下方法来创建可重用的One Liner:

public void oneMethodToCloseThemAll(ResultSet resultSet, Statement statement, Connection connection) {
    if (resultSet != null) {
        try {
            if (!resultSet.isClosed()) {
                resultSet.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
    if (statement != null) {
        try {
            if (!statement.isClosed()) {
                statement.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

    if (connection != null) {
        try {
            if (!connection.isClosed()) {
                connection.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
}

我在父类中使用此代码，它继承给所有发送DB查询的类。我可以在所有查询上使用联机程序，即使我没有resultSet。方法负责按照正确的顺序关闭ResultSet、Statement、Connection。这是我的最终块的样子。

finally {
    oneMethodToCloseThemAll(resultSet, preStatement, sqlConnection);
}

2019-12-09 10:56:27

如果你想要更紧凑的代码，我建议使用Apache Commons DbUtils。在这种情况下:

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    DbUtils.closeQuietly(rs);
    DbUtils.closeQuietly(stmt);
    DbUtils.closeQuietly(conn);
}

2016-01-13 07:30:50

据我所知，在当前的JDBC中，resultset和语句实现了AutoCloseable接口。这意味着它们会在被摧毁或超出范围时自动关闭。

2019-03-25 19:30:09

来自javadocs:

当一个Statement对象被关闭时，它的当前ResultSet对象，如果有存在，也是封闭的。

然而，当你关闭底层连接时，javadocs并不清楚Statement和ResultSet是否关闭。它们简单地声明关闭一个连接:

释放这个Connection对象的数据库和JDBC资源立即而不是等待它们将被自动释放。

在我看来，当你用完ResultSets, Statements和Connections时，总是显式地关闭它们，因为close的实现在不同的数据库驱动程序中是不同的。

通过在Apache的DBUtils中使用closequiet等方法，可以节省大量的模板代码。

2010-12-22 09:05:14

你所做的是完美的，非常好的练习。

我之所以说这是一种很好的实践……例如，如果由于某种原因，您正在使用“原始”类型的数据库池，并且调用connection.close()，连接将返回到池中，ResultSet/Statement将永远不会关闭，然后您将遇到许多不同的新问题!