Где закрыть Java PreparedStatements и ResultSets?

Для файлового ввода-вывода я обычно добавляю try / catch к блоку finally. Однако вы должны быть осторожны, чтобы не генерировать какие-либо исключения из блока finally, поскольку они приведут к потере исходного исключения (если таковое имеется).

См. Более конкретный пример закрытия соединения с базой данных в эта статья.

Не забывайте звонить близко. Это может вызвать проблемы.

Я предпочитаю добавлять блок try / catch в файл finally.

Если вы действительно вручную скручиваете свой собственный jdbc, он определенно становится беспорядочным. Close () в finally нужно обернуть собственным try catch, что, по крайней мере, некрасиво. Вы не можете пропустить закрытие, хотя ресурсы будут очищены при закрытии соединения (что может произойти не сразу, если вы используете пул). Фактически, одним из основных преимуществ использования фреймворка (например, спящего режима) для управления доступом к базе данных является управление подключением и обработкой набора результатов, чтобы вы не забыли закрыть.

Вы можете сделать что-нибудь такое простое, что хотя бы скроет беспорядок и гарантирует, что вы что-то не забудете.

public static void close(ResultSet rs, Statement ps, Connection conn)
{
    if (rs!=null)
    {
        try
        {
            rs.close();

        }
        catch(SQLException e)
        {
            logger.error("The result set cannot be closed.", e);
        }
    }
    if (ps != null)
    {
        try
        {
            ps.close();
        } catch (SQLException e)
        {
            logger.error("The statement cannot be closed.", e);
        }
    }
    if (conn != null)
    {
        try
        {
            conn.close();
        } catch (SQLException e)
        {
            logger.error("The data source connection cannot be closed.", e);
        }
    }

}

а потом,

finally {
    close(rs, ps, null); 
}

Пользуюсь этим ..

finally
{
    if (ps != null) ps.close();
    if (rs != null) rs.close();
}

Обычно у меня есть служебный метод, который может закрывать такие вещи, в том числе стараюсь не делать ничего с нулевой ссылкой.

Обычно, если close() выдает исключение, мне все равно, поэтому я просто регистрирую исключение и проглатываю его, но другой альтернативой было бы преобразовать его в RuntimeException. В любом случае, я рекомендую делать это с помощью служебного метода, который легко вызвать, поскольку вам может понадобиться сделать это во многих местах.

Обратите внимание, что ваше текущее решение не закроет ResultSet, если закрыть PreparedStatement не удается - лучше использовать вложенные блоки finally.

Не тратьте зря время на кодирование низкоуровневого управления исключениями, используйте высокоуровневый API, такой как Spring-JDBC, или настраиваемую оболочку вокруг объектов connection / statement / rs, чтобы скрыть беспорядочный код, связанный с try-catch.

В Java 7 вы не должны закрывать их явно, но используйте автоматическое управление ресурсами, чтобы гарантировать, что Ресурсы закрыты, а исключения обрабатываются соответствующим образом. Обработка исключений работает так:

Exception in try | Exception in close | Result
-----------------+--------------------+----------------------------------------
      No         |        No          | Continue normally
      No         |        Yes         | Throw the close() exception
      Yes        |        No          | Throw the exception from try block
      Yes        |        Yes         | Add close() exception to main exception
                 |                    |  as "suppressed", throw main exception

Надеюсь, это имеет смысл. В позволяет красивый код, например:

private void doEverythingInOneSillyMethod(String key)
  throws MyAppException
{
  try (Connection db = ds.getConnection()) {
    db.setReadOnly(true);
    ...
    try (PreparedStatement ps = db.prepareStatement(...)) {
      ps.setString(1, key);
      ...
      try (ResultSet rs = ps.executeQuery()) {
        ...
      }
    }
  } catch (SQLException ex) {
    throw new MyAppException("Query failed.", ex);
  }
}

До Java 7 лучше использовать вложенные блоки finally, а не проверять ссылки на null.

Пример, который я покажу, может выглядеть некрасиво с глубоким вложением, но на практике хорошо спроектированный код, вероятно, не будет создавать соединение, оператор и результаты одним и тем же методом; часто каждый уровень вложенности включает передачу ресурса другому методу, который использует его как фабрику для другого ресурса. При таком подходе исключения из close() будут маскировать исключение изнутри блока try. Это можно преодолеть, но это приводит к еще более запутанному коду и требует специального класса исключений, который обеспечивает «подавленную» цепочку исключений, присутствующую в Java 7.

Connection db = ds.getConnection();
try {
  PreparedStatement ps = ...;
  try {
    ResultSet rs = ...
    try {
      ...
    }
    finally {
      rs.close();
    }
  } 
  finally {
    ps.close();
  }
} 
finally {
  db.close();
}

Также обратите внимание:

«Когда объект Statement закрывается, его текущий объект ResultSet, если он существует, также закрывается».

http://java.sun.com/j2se/1.5.0/docs/api/java/sql/Statement.html#close ()

Достаточно закрыть только PreparedStatement в finally, и только если он еще не закрыт. Если вы хотите быть действительно конкретным, закройте ResultSet ПЕРВЫМ, а не после закрытия PreparedStatement (закрытие его после, как некоторые из приведенных здесь примеров, должно фактически гарантировать исключение, поскольку оно уже закрыто).

предложение focus finally,

finally {
   try {
      rs.close();
      ps.close();
   } catch (Exception e) {
      // Do something
   }
}

Думаю, надо 2 пункта доработать.

Во-первых, снова используйте команду try & catch в предложении fainlly.

Во-вторых, выполните rs.close () перед выполнением ps.close ().

[email protected]

Если вы используете Java 7, вы можете использовать улучшения в механизмах обработки исключений в тех классах, которые реализуют Автоматическое закрытие (например, PreparedStatement, Resultset).

Вас также может заинтересовать этот вопрос: Закрытие ResultSet в Java 7

Я знаю, что это старый вопрос, но на всякий случай, если кто-то ищет ответ, у java теперь есть решение try-with-resouce.

static String readFirstLineFromFile(String path) throws IOException {
      try (BufferedReader br =
                   new BufferedReader(new FileReader(path))) {
        return br.readLine();
    }
}

Вероятно, старый (хотя и простой) способ делать что-то, но он все еще работает:

public class DatabaseTest {

    private Connection conn;    
    private Statement st;   
    private ResultSet rs;
    private PreparedStatement ps;

    public DatabaseTest() {
        // if needed
    }

    public String getSomethingFromDatabase(...) {
        String something = null;

        // code here

        try {
            // code here

        } catch(SQLException se) {
            se.printStackTrace();

        } finally { // will always execute even after a return statement
            closeDatabaseResources();
        }

        return something;
    }

    private void closeDatabaseResources() {
        try {
            if (conn != null) {
                System.out.println("conn closed");
                conn.close();
            }

            if (st != null) {
                System.out.println("st closed");
                st.close();
            }

            if (rs != null) {
                System.out.println("rs closed");
                rs.close();
            }

            if (ps != null) {
                System.out.println("ps closed");
                ps.close();
            }

        } catch(SQLException se) {
            se.printStackTrace();
        }               
    }
}

Основываясь на ответе @erickson, почему бы просто не сделать это в одном блоке try, как это?

private void doEverythingInOneSillyMethod(String key) throws MyAppException
{
  try (Connection db = ds.getConnection();
       PreparedStatement ps = db.prepareStatement(...)) {

    db.setReadOnly(true);
    ps.setString(1, key);
    ResultSet rs = ps.executeQuery()
    ...
  } catch (SQLException ex) {
    throw new MyAppException("Query failed.", ex);
  }
}

Обратите внимание, что вам не нужно создавать объект ResultSet внутри блока try, поскольку объекты ResultSet автоматически закрываются при закрытии объекта PreparedStatement.

A ResultSet object is automatically closed when the Statement object that generated it is closed, re-executed, or used to retrieve the next result from a sequence of multiple results.

Ссылка: https://docs.oracle.com/javase/7/docs/api/java/sql/ResultSet.html