Повинно чи закрити набори результатів JDBC та заяви окремо, хоча З'єднання після цього закрите?

Кажуть, що це добра звичка закривати всі ресурси JDBC після використання. Але якщо у мене є наступний код, чи потрібно закрити набір результатів та заяву?

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    try { if (rs != null) rs.close(); } catch (Exception e) {};
    try { if (stmt != null) stmt.close(); } catch (Exception e) {};
    try { if (conn != null) conn.close(); } catch (Exception e) {};
}

Питання полягає в тому, чи закриття з’єднання виконує цю роботу, чи це залишає деякі ресурси у використанні.

Те, що ви зробили, - це ідеальна та дуже хороша практика.

Причиною я вважаю свою добру практику ... Наприклад, якщо ви з якоїсь причини використовуєте "примітивний" тип об'єднання баз даних і ви зателефонували connection.close(), з'єднання буде повернуто до пулу і ResultSet/ Statementніколи не буде закрито, і тоді ви зіткнеться з багатьма різними новими проблемами!

Тож не завжди можна розраховувати на connection.close()прибирання.

Сподіваюся, це допомагає :)

Java 1.7 значно полегшує наше життя завдяки випробуванню ресурсів .

try (Connection connection = dataSource.getConnection();
    Statement statement = connection.createStatement()) {
    try (ResultSet resultSet = statement.executeQuery("some query")) {
        // Do stuff with the result set.
    }
    try (ResultSet resultSet = statement.executeQuery("some query")) {
        // Do more stuff with the second result set.
    }
}

Цей синтаксис досить короткий і елегантний. І connectionсправді буде закрито, навіть коли statementнеможливо створити.

З javadocs :

Коли Statementоб’єкт закритий, його поточний ResultSetоб'єкт, якщо такий існує, також закривається.

Однак Javadocs не надто ясно чи Statementй ResultSetзакриті при закритті основний Connection. Вони просто заявляють, що закриваючи з'єднання:

Випускає Connectionбазу даних цього об’єкта та ресурси JDBC негайно, замість того, щоб чекати їх автоматичного звільнення.

На мій погляд, завжди явно близько ResultSets, Statementsі Connectionsколи ви закінчите з ними, так як реалізація closeможе відрізнятися між драйверами баз даних.

Ви можете заощадити багато коду плит котла, використовуючи такі методи, як closeQuietlyу DBUtils від Apache.

Зараз я використовую Oracle з Java. Ось моя точка зору:

Вам слід закрити ResultSetі Statementявно, оскільки раніше у Oracle виникли проблеми з утримуванням курсорів відкритими навіть після закриття з'єднання. Якщо ви не закриєте ResultSet(курсор), це призведе до помилки, на кшталт перевищення Максимально відкритих курсорів .

Я думаю, що ви можете зіткнутися з тією ж проблемою і в інших базах даних, якими ви користуєтесь.

Ось підручник Close ResultSet, коли закінчите :

Закрийте ResultSet, коли закінчите

Закрийте ResultSetоб'єкт, як тільки ви закінчите роботу з ResultSetоб'єктом, навіть якщо Statementоб'єкт закриває ResultSetоб'єкт неявно, коли він закривається, закриття ResultSetявно дає можливість сміттєзбірнику якнайшвидше згадати пам'ять, оскільки ResultSetоб'єкт може зайняти багато пам'яті залежно від запиту.

ResultSet.close();

Якщо ви хочете більш компактний код, я пропоную використовувати Apache Commons DbUtils . В цьому випадку:

Connection conn = null;
PreparedStatement stmt = null;
ResultSet rs = null;
try {
    conn = // Retrieve connection
    stmt = conn.prepareStatement(// Some SQL);
    rs = stmt.executeQuery();
} catch(Exception e) {
    // Error Handling
} finally {
    DbUtils.closeQuietly(rs);
    DbUtils.closeQuietly(stmt);
    DbUtils.closeQuietly(conn);
}

Правильний і безпечний метод закрити ресурси, пов'язані з JDBC, це (взято з розділу Як правильно закривати ресурси JDBC - кожен раз ):

Connection connection = dataSource.getConnection();
try {
    Statement statement = connection.createStatement();

    try {
        ResultSet resultSet = statement.executeQuery("some query");

        try {
            // Do stuff with the result set.
        } finally {
            resultSet.close();
        }
    } finally {
        statement.close();
    }
} finally {
    connection.close();
}

Не має значення, чи Connectionє об'єднаним чи ні. Навіть об'єднане з'єднання має очиститися перед поверненням у басейн.

"Очистити" зазвичай означає закриття наборів результатів і відкат будь-яких очікуваних транзакцій, але не закриття з'єднання. В іншому випадку об’єднання втрачає сенс.

Ні, вам не потрібно нічого закривати, АЛЕ з'єднання. За специфікації JDBC, які закривають будь-який вищий об'єкт, автоматично закриються нижчі об'єкти. Закриття Connectionзакриє всі Statementстворені з’єднання. Закриття будь-якого Statementзакриє всі ResultSet, створені цим Statement. Не має значення, чи Connectionє об'єднаним чи ні. Навіть об'єднане з'єднання має очиститися перед поверненням у басейн.

Звичайно, у вас можуть бути довгі вкладені цикли на Connectionстворенні безлічі висловлювань, тоді їх закриття доречно. Я майже ніколи не закриваюсь ResultSet, здається надмірним при закритті Statementабо ConnectionБУДЕ їх закривати.

Я створив наступний метод для створення одноразового вкладиша:

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();
        }
    }
}

Я використовую цей код у батьківському класі, який успадковується всім моїм класам, які надсилають запити БД. Я можу використовувати Oneliner для всіх запитів, навіть якщо у мене немає результатуSet.The Method забезпечує закриття ResultSet, Statement, Connection у правильному порядку. Ось так виглядає мій остаточний блок.

finally {
    oneMethodToCloseThemAll(resultSet, preStatement, sqlConnection);
}

Наскільки я пам’ятаю, у поточному JDBC, Resultsets та оператори реалізують інтерфейс AutoCloseable. Це означає, що вони автоматично закриваються після знищення або виходу з поля зору.

Деякі функції зручності:

public static void silentCloseResultSets(Statement st) {
    try {
        while (!(!st.getMoreResults() && (st.getUpdateCount() == -1))) {}
    } catch (SQLException ignore) {}
}
public static void silentCloseResultSets(Statement ...statements) {
    for (Statement st: statements) silentCloseResultSets(st);
}

З формою 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 одночасних користувачів), і ми не мали з цим проблем - тому ніколи не бачимо цього попереджувального повідомлення.