java - 不关闭我的 JDBC PreparedStatements 会导致内存泄漏吗？

Question

我正在使用 java.sql PreparedStatements，我想知道以下内容:

在Java is Pass-by-Value, Dammit! ，以下是 Java 传递约定的示例:

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，所以我知道在 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 block 中。