Renvoyer un ResultSet

Vous ne devriez jamais passer un ResultSet à travers les méthodes publiques. C'est sujette à des ressources fuit, car vous êtes obligé de garder la déclaration et la connexion ouverte. La fermeture serait implicitement fermez le jeu de résultats. Mais les laissant ouverts les ferait pendre autour de et la cause de la DB à court de ressources quand il y a trop de leur ouvrir.

Carte à une collection de Javabeans comme et de le retourner à la place:

public List<Biler> list() throws SQLException {
    Connection connection = null;
    PreparedStatement statement = null;
    ResultSet resultSet = null;
    List<Biler> bilers = new ArrayList<Biler>();

    try {
        connection = database.getConnection();
        statement = connection.prepareStatement("SELECT id, name, value FROM Biler");
        resultSet = statement.executeQuery();

        while (resultSet.next()) {
            Biler biler = new Biler();
            biler.setId(resultSet.getLong("id"));
            biler.setName(resultSet.getString("name"));
            biler.setValue(resultSet.getInt("value"));
            bilers.add(biler);
        }
    } finally {
        if (resultSet != null) try { resultSet.close(); } catch (SQLException ignore) {}
        if (statement != null) try { statement.close(); } catch (SQLException ignore) {}
        if (connection != null) try { connection.close(); } catch (SQLException ignore) {}
    }

    return bilers;
}

Ou, si vous êtes sur l'île de Java 7, il suffit de l'utilisation de try-with-resources déclaration fermeture automatique de ces ressources:

public List<Biler> list() throws SQLException {
    List<Biler> bilers = new ArrayList<Biler>();

    try (
        Connection connection = database.getConnection();
        PreparedStatement statement = connection.prepareStatement("SELECT id, name, value FROM Biler");
        ResultSet resultSet = statement.executeQuery();
    ) {
        while (resultSet.next()) {
            Biler biler = new Biler();
            biler.setId(resultSet.getLong("id"));
            biler.setName(resultSet.getString("name"));
            biler.setValue(resultSet.getInt("value"));
            bilers.add(biler);
        }
    }

    return bilers;
}

Par ailleurs, vous ne devez pas être de déclarer les ConnectionStatement et ResultSet que les variables d'instance à tous les (grands threadsafety problème!!!!), ni avaler la SQLException au point que, à tous (l'appelant n'ai aucune idée de qu'un problème est survenu), ni de fermer les ressources dans le même try (si par exemple d'ensemble de résultats à proximité déclenche une exception, l'état de connexion et sont encore ouvertes). Tous ces problèmes sont résolus dans le code ci-dessus des extraits.

Bien, vous ne appel rs.close() dans votre finally-bloc.

C'est fondamentalement une bonne idée, car vous devez fermer toutes vos ressources (connexions, des déclarations, des ensembles de résultats, ...).

Mais vous devez fermer les après vous les utilisez.

Il existe au moins trois solutions possibles:

1. ne ferment pas le jeu de résultats (et de connexion, ...) et nécessitent l'appelant pour appeler un "proche" de la méthode.

   Cela signifie essentiellement que, maintenant, la appelant doit n'oubliez pas d'appeler à proximité et n'a pas vraiment rendre les choses plus faciles.

2. l'appelant de passer dans une classe qui est passée le jeu de résultats et d'appel que dans votre méthode

   Cela fonctionne, mais il peut devenir un peu verbeux, comme vous aurez besoin d'une sous-classe d'une interface (peut-être un anonyme intérieur de la classe) pour chaque bloc de code que vous voulez exécuter sur le jeu de résultats.

   L'interface ressemble à ceci:

   public interface ResultSetConsumer<T> {
     public T consume(ResultSet rs);
   }

   et votre select méthode ressemblait à ceci:

   public <T> List<T> select(String query, ResultSetConsumer<T> consumer) {
     Connection con = null;
     Statement st = null;
     ResultSet rs = null;
   
       try {
         con = DriverManager.getConnection(url, user, password);
         st = con.createStatement();
   
         rs = st.executeQuery(query);
         List<T> result = new ArrayList<T>();
         while (rs.next()) {
             result.add(consumer.consume(rs));
         }
       } catch (SQLException ex) {
         //logging
       } finally {
         try {
           if (rs != null) {
               rs.close();
           }
           if (st != null) {
               st.close();
           }
           if (con != null) {
               con.close();
           }
         } catch (SQLException ex) {
           Logger lgr = Logger.getLogger(MySQL.class.getName());
           lgr.log(Level.WARNING, ex.getMessage(), ex);
         }
       }
     return rs;
   }

3. faire tout le travail à l'intérieur de la select méthode et revenir quelques List.

   C'est probablement le plus largement utilisé: itération sur le jeu de résultats et de les convertir en données personnalisées dans votre propre Otd et retour ceux.

Vous pouvez utiliser le CachedRowSet objet qui est juste de ce que vous voulez:

public CachedRowSetImpl select(String url, String user, String password) {

    CachedRowSetImpl crs = null;

    try (Connection con = DriverManager.getConnection(url, user, password);
         Statement st = con.createStatement();
         ResultSet rs = st.executeQuery("SELECT * FROM biler");) {

        crs = new CachedRowSetImpl();
        crs.populate(rs);

    } catch (SQLException ex) {
        Logger lgr = Logger.getLogger(MySQL.class.getName());
        lgr.log(Level.SEVERE, ex.getMessage(), ex);


    } catch (SQLException ex) {
        Logger lgr = Logger.getLogger(MySQL.class.getName());
        lgr.log(Level.WARNING, ex.getMessage(), ex);
    }

    return crs;
}

Vous pouvez lire la documentation ici:
https://docs.oracle.com/javase/7/docs/api/javax/sql/rowset/CachedRowSet.html