Discussion :

[Yoorwan]

Bonjour,

Je développe en java une base de données mysql qui stocke des personnes. J'ai une méthode pour rechercher toutes les personnes de la table par rapport à leur nom.
Mais le problème c'est que je n'arrive pas à récupérer les valeurs du ResultSet alors qu'il n'est pas vide.

Code Java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
public static ArrayList<Personne> findByName(String n) throws SQLException {
 
		String SQLPrep = "SELECT nom, prenom FROM personne WHERE nom = ?;";
		PreparedStatement prep1 = DBconnection.getConnect().prepareStatement(SQLPrep);
		prep1.setString(1, n);
		prep1.execute();
		ResultSet rs = prep1.getResultSet();
		ArrayList<Personne> list = new ArrayList<Personne>();
		// s'il y a un resultat
		if (rs.next()) {
			while (rs.next()) {
				String nom = rs.getString("nom");
				String prenom = rs.getString("prenom");
				Personne p = new Personne(nom, prenom);
				list.add(p);
			}
		}
		return list;
	}

Ici le programme passe bien le vérification du if(rs.next()) mais ne rentre pas dans la boucle et je ne vois pas d'où ça peut venir.

Merci de votre aide.

[joel.drigo]

Salut,

La méthode next() avance vers le prochain enregistrement s'il existe et retourne true dans ce cas, ou false s'il n'y a plus d'enregistrement dans le ResultSet.

Quand tu fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
if (rs.next()) {
			while (rs.next()) {

On avance une première fois avec le rs.next() du if, puis une deuxième fois avec le rs.next() du while : s'il n'y a qu'un enregistrement dans le ResultSet, on n'entre pas dans le while forcément.

Soit tu écris :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
if (rs.next()) {
        do {
                // lecture des données de l'enregistrement
	}	while (rs.next());

soit tout simplement, sans if :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
while (rs.next()) {
                // lecture des données de l'enregistrement
}

D'ailleurs plutôt que :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
prep1.execute();
		ResultSet rs = prep1.getResultSet();

autant faire directement :
[CODE

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
ResultSet rs  = prep1.executeQuery();
   /* tout le reste */
 
}

Par ailleurs, il faut que tu penses à libérer tes ressources (Statement, ResultSet et Connection éventuellement). Avec un try-with-resources ça se fait facilement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
ArrayList<Personne> list = new ArrayList<>();
 
// ici on ne ferme que statement et resultset (pas la connexion)
try(PreparedStatement prep1 = DBconnection.getConnect().prepareStatement(SQLPrep)) {
    prep1.setString(1, n);
    ResultSet rs = prep1.executeQuery();
    while( rs.hasNext() )  {
         /* ... */
   }
}
return list;

[Yoorwan]

Ok j'ai retiré le if et ça fonctionne.
J'avais déjà essayé de le retirer mais j'ai dû m'embrouiller dans mes affichages console car j'avais cru que le problème était toujours là.

J'avais aussi essayé de changer le if(rs.next()) en if(!rs.first()) mais je suppose que le problème reste le même vu que le rs.first() me met sur la première ligne du résultat.

En tout cas merci de ton aide je passe en résolu.