ResultSet non fermé lorsque la connexion est fermée ?

Question

J'ai fait une revue de code (en utilisant principalement des outils comme FindBugs) d'un de nos projets favoris et FindBugs a marqué le code suivant comme erroné (pseudocode) :

Connection conn = dataSource.getConnection();

try{
    PreparedStatement stmt = conn.prepareStatement();
    //initialize the statement
    stmt.execute();
    ResultSet rs =  stmt.getResultSet();
    //get data
}finally{
    conn.close();
}

L'erreur était que ce code pouvait ne pas libérer de ressources. Je me suis rendu compte que le ResultSet et le Statement n'étaient pas fermés, alors je les ai fermés dans finally :

finally{
    try{
        rs.close()
    }catch(SqlException se){
        //log it
    }
    try{
        stmt.close();
    }catch(SqlException se){
        //log it
    }
    conn.close();
}

Mais j'ai rencontré le modèle ci-dessus dans de nombreux projets (de plusieurs entreprises), et personne ne fermait les ResultSets ou les Statements.

Avez-vous eu des problèmes avec les ResultSets et les Statements qui ne sont pas fermés lorsque la connexion est fermée ?

J'ai trouvé seulement ce et il fait référence au fait qu'Oracle a des problèmes avec la fermeture des ResultSets lors de la fermeture des Connexions (nous utilisons une base de données Oracle, d'où mes corrections). java.sql.api ne dit rien dans la javadoc Connection.close().

Answer 1

Un problème lié au fait de fermer UNIQUEMENT la connexion et non le jeu de résultats est que si votre code de gestion des connexions utilise la mise en commun des connexions, l'opération de fermeture de l'ensemble de résultats ne peut pas être effectuée. connection.close() ne ferait que remettre la connexion dans le pool. De plus, certaines bases de données ont une ressource curseur sur le serveur qui ne sera pas libérée correctement à moins qu'elle ne soit explicitement fermée.

Answer 2

J'ai eu des problèmes avec des ResultSets non fermés dans Oracle, même si la connexion était fermée. L'erreur que j'ai obtenue est la suivante

"ORA-01000: maximum open cursors exceeded"

Donc : Fermez toujours votre ResultSet !

Answer 3

Vous devez toujours fermer explicitement toutes les ressources JDBC. Comme Aaron et John l'ont déjà dit, la fermeture d'une connexion ne fera souvent que la remettre dans un pool et tous les pilotes JDBC ne sont pas implémentés exactement de la même manière.

Voici une méthode utilitaire qui peut être utilisée à partir d'un bloc final :

public static void closeEverything(ResultSet rs, Statement stmt,
        Connection con) {
    if (rs != null) {
        try {
            rs.close();
        } catch (SQLException e) {
        }
    }
    if (stmt != null) {
        try {
            stmt.close();
        } catch (SQLException e) {
        }
    }
    if (con != null) {
        try {
            con.close();
        } catch (SQLException e) {
        }
    }
}

Answer 4

Dans ce cas, Oracle vous donnera des erreurs concernant les curseurs ouverts.

Selon : http://java.sun.com/javase/6/docs/api/java/sql/Statement.html

il semble que la réutilisation d'une instruction ferme tous les jeux de résultats ouverts, et que la fermeture d'une instruction ferme tous les jeux de résultats, mais je ne vois rien qui indique que la fermeture d'une connexion ferme toutes les ressources qu'elle a créées.

Tous ces détails sont laissés au fournisseur du pilote JDBC.

Il est toujours plus sûr de tout fermer explicitement. Nous avons écrit une classe util qui enveloppe tout avec try{ xxx } catch (Throwable {} afin que vous puissiez simplement appeler Utils.close(rs) et Utils.close(stmt), etc. sans avoir à vous soucier des exceptions que le scan de fermeture est censé lancer.

Answer 5

L'ODBC Bridge peut produire une fuite de mémoire avec certains pilotes ODBC.

Si vous utilisez un bon pilote JDBC, vous ne devriez pas avoir de problèmes pour fermer la connexion. Mais il y a deux problèmes :

• Savez-vous si vous avez un bon conducteur ?
• Utiliserez-vous d'autres pilotes JDBC à l'avenir ?

Que la meilleure pratique est de tout fermer.