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

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

所以你不能总是指望connection.close()来清理。

来自javadocs:

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

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

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

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

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

使用Java 6表单，我认为最好在关闭之前检查它是否关闭(例如，如果一些连接池在其他线程中驱逐了连接)-例如一些网络问题-语句和结果集状态可以关闭。(这是不经常发生的，但我在Oracle和DBCP有这个问题)。我的模式是(在旧的Java语法中):

try {
    //...   
    return resp;
} finally {
    if (rs != null && !rs.isClosed()) {
        try {
            rs.close();
        } catch (Exception e2) { 
            log.warn("Cannot close resultset: " + e2.getMessage());
        }
    }
    if (stmt != null && !stmt.isClosed()) {
        try {
            stmt.close();
        } catch (Exception e2) {
            log.warn("Cannot close statement " + e2.getMessage()); 
        }
    }
    if (con != null && !conn.isClosed()) {
        try {
            con.close();
        } catch (Exception e2) {
            log.warn("Cannot close connection: " + e2.getMessage());
        }
    }
}

理论上，它不是100%完美的，因为在检查关闭状态和关闭本身之间，有一点空间来改变状态。在最坏的情况下，你很快就会得到警告。-但在长期运行的查询中，它小于状态变化的可能性。我们在“平均”负载(150个同时用户)的生产环境中使用了这个模式，我们没有遇到任何问题——所以从来没有看到警告消息。

不，您不需要关闭连接以外的任何东西。根据JDBC规范，关闭任何较高的对象都会自动关闭较低的对象。关闭连接将关闭连接所创建的任何语句。关闭任何语句将关闭由该语句创建的所有结果集。连接是否可池并不重要。即使是可入池的连接也必须在返回池之前进行清洁。

当然，您可能在Connection上有很长的嵌套循环，创建了许多语句，然后关闭它们是合适的。虽然我几乎从来没有关闭ResultSet，但关闭语句或连接将关闭它们时似乎过度了。

我创建了以下方法来创建可重用的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);
}

我现在使用Oracle和Java。以下是我的观点:

您应该显式地关闭ResultSet和Statement，因为Oracle以前在关闭连接后仍然保持游标打开的问题。如果您不关闭ResultSet(游标)，它将抛出一个错误，如超过最大打开游标。

我认为您在使用其他数据库时可能会遇到同样的问题。

完成后关闭ResultSet:

完成时关闭ResultSet 完成后立即关闭ResultSet对象 使用ResultSet对象甚至 对象关闭 ResultSet对象 close，显式关闭ResultSet 给垃圾收集器机会 尽早回忆 因为ResultSet对象可能占用 大量内存取决于查询。 ResultSet.close ();