不关闭我的 JDBC PreparedStatements 是否会导致内存泄漏？

Question

我正在使用 java.sql PreparedStatements，我想知道以下问题：

在Java is Pass-by-Value, Dammit! 中，给出了以下作为 Java 的 Pass-By 约定的示例：

public void foo(Dog d) {
    d = new Dog("Fifi"); // creating the "Fifi" dog
}

Dog aDog = new Dog("Max"); // creating the "Max" dog
// at this point, aDog points to the "Max" dog
foo(aDog);
// aDog still points to the "Max" dog 

在我的代码中，这出现如下（半 Java 伪代码）：

public void method() {
  PreparedStatement pstmt = null;
  ResultSet rs = null;
  try {
    rs = executeStatement(sql-string, pstmt, conn, vars...);
  } catch (....) { /* error-handling */ }
  /// do stuff with the data
  rs.close();
}

executeStatement 是（类似于）以下内容：

ResultSet executeStatement(String sql, PreparedStatement pstmt, Connection conn, Object[] vars...) {
  pstmt = conn.prepareStatement(sql);
  /// set pstmt variables...
  ResultSet rs = pstmt.execute();
  return rs;
}

根据我对 Java 的传递约定的理解，我在主代码中使用 pstmt 做任何事情都没有用，因为即使在调用 executeStatement 之后它仍然是 null。但是，因为关闭 PreparedStatement 也会关闭 ResultSet，所以我知道在处理 ResultSet 时在 executeStatement 中创建的 PreparedStatement 不会关闭。

这是否意味着这里存在内存泄漏？ （我对内存泄漏以及如何诊断/修复它们的理解充其量是参差不齐的）。有什么方法可以让我以不同的方式构造它以避免泄漏，但继续使用可以执行 SQL 字符串并以抽象方式返回 ResultSet 的方法？

Answer 1

首先，传统意义上的内存泄漏，即分配的数据不能再被引用，在 Java 中不存在，因为只有没有被引用的数据才会被收集。但是，在这种情况下，您无论如何都不是在寻找内存泄漏，而是在寻找资源泄漏（这在 Java 中是一个更大的问题）：PreparedStatement 的内存最终将被收集并释放内存，因为它是在执行您的方法后不再引用，但是，语句持有的资源应该更早地释放，而不仅仅是在垃圾收集器运行时释放。

您可以做的是编写一个包含Statement 和ResultSet 作为成员的类并返回这个类，如下所示：

class ResultSetStatementPair {
  ResultSetStatementPair(ResultSet rs, Statement stmt) {
    this.rs = rs; this.stmt = stmt;
  }

  ResultSet rs;
  Statement stmt;
}

ResultSetStatementPair executeStatement(String sql, Connection conn, Object[] vars...) {
  PreparedStatement pstmt = conn.prepareStatement(sql);
  ResultSet rs = pstmt.execute();
  return new ResultSetStatementPair(rs, pstmt);
}

public void method() {
  Statement pstmt = null;
  ResultSet rs = null;
  try {
    ResultSetStatementPair pair = executeStatement(sql-string, pstmt, conn, vars...);
    rs = pair.rs;
    stmt = pair.stmt;
    // do stuff with the data
  } catch (....) { /* error-handling */ }
  finally { 
    if(rs != null) rs.close();
    if(stmt != null) stmt.close();
  }
}

还注意到我添加了 finally 并将 do stuff 移动到 try 块中。

Answer 2

我在这里没有看到内存泄漏。我发现的唯一问题是您没有在 finally 块中关闭结果集。因此，如果抛出异常，则 rs.close() 将不会被执行。

正如 Andrei 所说，关闭结果集也将关闭基础语句。我不确定你在哪里关闭连接，但这也应该发生在 finally 块中。