Ne réutilisation d'une Déclaration et Resultset de libérer les ressources de son utilisation antérieure? Ou dois-je fermer explicitement eux avant de les réutiliser?

Exemple de code:

        aStmt = aConn.prepareStatement(aQuery);
        aRset = aStmt.executeQuery(cQuery);

        while (cRset.next()) {
            //stuff to determine value of parm1

            aStmt.setString(1, parm1);                
            aRset = aStmt.executeQuery(); 

            //more stuff
        }

Dois-je fermer aStmt et aRset après chaque boucle à l'intérieur de l'instruction while? Ou va les réutiliser dans la réussir boucles de libérer de la mémoire/ressources utilisées à partir de la précédente boucles?

InformationsquelleAutor heisenbergman | 2013-07-12
  1. 25

    Le comportement de jeux de résultats et (préparé) états est explicitement décrite dans l'API Java. Je vous suggère de lire la documentation disponible (et JDBC spec) pour obtenir les détails.

    La Déclaration API dit:

    Par défaut, un seul ResultSet objet par Statement objet peuvent être ouvertes en même temps. Par conséquent, si la lecture d'un ResultSet objet est entrelacé avec la lecture de l'autre, chacun doit avoir été généré par les différents Statement objets. Toutes les méthodes d'exécution dans le Statement interface implicitement à proximité d'une déclaration de l'actuel ResultSet objet si l'un ouvert existe.

    (l'emphase est mienne).

    Dans votre code, lorsque vous appelez aStmt.executeQuery(), l'ancien ResultSet attribué à aRset est fermé implicitement par le pilote. Cela dit, il serait préférable de fermer explicitement-le vous-même (ou l'utilisation de Java 7 try-with-resources), pour vous éviter d'oublier de fermer la ResultSet dans la dernière itération de la boucle.

    Maintenant à la PreparedStatement: Lorsque vous préparez une déclaration (en général, la mise en œuvre peut varier), la requête est envoyée au serveur pour la compilation. Sur l'exécution des paramètres pour l'exécution est envoyé au serveur. L'appel de close() sur aStmt entraînerait l'instruction préparée être libéré sur le serveur, c'est clairement PAS ce que vous voulez comme vous le voulez ré-utiliser l'instruction avec les différentes valeurs de ce paramètre.

    Donc en bref

    1. De clôture ResultSet n'est pas techniquement nécessaire ici (sauf pour la dernière ResultSet créé), mais il est préférable de le faire explicitement
    2. Vous ne devez fermer la PreparedStatement lorsque vous avez terminé avec elle.

    À l'aide de try-with-resources est une façon pour enlever une partie de la confusion sur ces questions, comme votre code sera automatiquement libérer des ressources lorsque cela est fait avec elle (à la fin de la portée de l'utilisation):

    try (
    ResultSet cRset = cStmt.executeQuery(cQuery);
    PreparedStatement aStmt = aConn.prepareStatement(aQuery);
) {
    while (cRset.next()) {
        //stuff to determine value of parm1

        aStmt.setString(1, parm1);                
        try (ResultSet aRset = aStmt.executeQuery()) {
            //more stuff
        }
    }
}

    À la fin de ce morceau de code toutes les ressources JDBC sont correctement fermés (dans le bon ordre, même si des exceptions s'est produite etc)

    • En fait, pour le dernier jeu de résultats créés (tous les tous les autres ne sont pas fermées ressources), j'ai une méthode qui se ferme tout à la fin du programme. Considérant que, est bien de laisser ma logique?
    • Vous devez fermer ressources dès que vous n'en avez plus besoin. En les laissant ouvertes pourrait augmenter l'utilisation de la mémoire à la fois dans votre application et la base de données.
    InformationsquelleAutor Mark Rotteveel
  2. 2

    Non, vous ne pouvez pas fermer la ResultSet et Statement à l'intérieur de la while boucle.

    Vous avez à les refermer après la boucle.

    Aussi, si vous voulez réutiliser le PreparedStatement alors vous ne pouvez pas le fermer jusqu'à ce que vous êtes prêt avec votre traitement.

    Meilleure règle est de fermer ces ressources dans le même bloc que ils sont créés. Dans votre cas, la meilleure chose à faire est de fermer les ressources dans un finally bloc après la capture de la SQLException.

    E. g.

    try {
    aStmt = aConn.prepareStatement(aQuery);
    cRset = cStmt.executeQuery(cQuery);

    while (cRset.next()) {
        //stuff to determine value of parm1

        aStmt.setString(1, parm1);
        try {
            aRset = aStmt.executeQuery();
        } finally {
            aRset.close();
        }

        //more stuff
    }
} catch (SQLException ex) {
    //Do error handling
} finally {
    //Close Resultset
}

    Dans Java 7, vous pouvez utiliser l'essayer avec des ressources.

    InformationsquelleAutor Uwe Plonus
  3. 0

    PreparedStatement API: UNE instruction SQL est précompilé et stockées dans un objet PreparedStatement. Cet objet peut ensuite être utilisé pour exécuter efficacement cette déclaration à plusieurs reprises.

    Mais vous ne pouvez pas réutiliser un objet ResultSet. Lorsque vous appelez executeQuery sur un objet PreparedStatement la deuxième fois qu'un nouveau jeu de résultats est créé, si vous ne fermez pas le précédent jeu de résultats, vous risquez d'obtenir une fuite de ressources.

    • JDBC spécification exige que les conducteurs à proximité de la vieille ResultSet si un nouveau est créé à partir de la même Statement objet.
    InformationsquelleAutor Evgeniy Dorofeev