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

Question

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

在Java中是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块中关闭resultSet。 因此，如果抛出异常，则rs.close()将不会被执行。

正如Andrei评论的那样，关闭结果集也将关闭基础声明。 我不确定你在哪里关闭连接，但这也应该发生在finally块中。