Discussion :

[ysahel]

Bonjour,

Je possede une table formée de 2 colonnes: "Login" et "mot de passe". Alors dans mon programme java je vais exécuter la requête SQL suivante:

Code :

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

1
2
3
 
String req="SELECT * FROM user WHERE ("login ="+log+"and password ="+pw+");";
rs = stmt.executeQuery(req);

Comment je peux déterminer qu'il y a une valeur ou pas dans ma table (c'est a dire est ce qu'il y a une fonction en java qui permet de retourner un entier par exemple pour indiquer si cette "login et password" existe ou pas dans ma table?

J'espère que c'est claire et j'espère que vous pouvez m'aider.
Merci d'avance.

[Ivelios]

Code :

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

1
2
3
4
boolean exist = false;
while(rs.next){
    exist = true;
}

Mais il y a plus simple que ça je pense

[ysahel]

Mais il y a plus simple que ça je pense

Oui c'est ce que je cherche car j'ai pensé en ce que vous avez me dit:

Code :

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

1
2
3
4
boolean exist = false;
while(rs.next){
    exist = true;
}

Merci.

[Ivelios]

Code :

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

1
2
3
4
boolean exist = true;
if(rs.isAfterLast()){
    exist = false;
}

Mais ça revient un peu près au même au niveau du nombre de lignes

[OButterlin]

La solution la plus simple est (si tu n'as pas besoin des données)

Code :

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

1
2
3
4
5
6
 
PreparedStatement pstmt = connection.prepareStatement("select login from user where login=? and password=?");
pstmt.setString(1, unLogin);
pstmt.setString(2, unPassword);
ResultSet rs = pstmt.executeQuery();
boolean exist = rs.next();

J'ai mis la solution complète avec un PreparedStatement parce qu'il faut éviter au maximum les requêtes du type de celle que tu montres (surtout pour une authentification !), il y a un gros risque d'injection .

Je ne vois pas l'intérêt de faire une boucle while dans ce cas puisqu'il serait très surprenant d'avoir plusieurs utilisateurs avec le même login (bug fonctionnel)

A+

[ysahel]

Voila j'ai essayer d'utiliser votre code, mais j'ai pas obtenu de résultat:

Code :

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

1
2
3
4
5
6
7
 
String req="SELECT * FROM user WHERE ( login ="+log+"and password ="+pw+");";
PreparedStatement pstmt = connection.prepareStatement(req);
pstmt.setString(1, log);
pstmt.setString(2, pw);
ResultSet rs = pstmt.executeQuery();
boolean exist = rs.next();return exist;

S'il vous plait expliquez mieux le code que vous avez mis.

Merci.

[Ivelios]

Quand tu utilise un ResulSet :
imaginons qu'il y ai 3 éléments dedans

|1|2|3| ça faira comme un tableau, et tu as un pointeur qui bouge dans ce tableau

Quand tu fait .next(); tu vas à la case suivante...
Au départ tu te situe dans la case "0", celle juste avant la premiere. (beforeFirst). quand tu fais rs.next() il te renvoie un boolean true si il y a un suivant. L'élément qui suis le beforeFirst est le First (le premier). En bon français, rs.next()= Y a t il un premier élément??

[in]

S'il vous plait expliquez mieux le code que vous avez mis.

Je pense qu'avec ça tu devrais avoir les explications nécessaires :
Les instructions paramétrées - PreparedStatement

[Yomhgui]

Il y a un autre point qui ne va pas dans ton code, c'est le passage des paramètres au PreparedStatment. Tu écris:

Code :

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

1
2
3
4
1    String req="SELECT * FROM user WHERE ( login ="+log+"and password ="+pw+");";
2    PreparedStatement pstmt = connection.prepareStatement(req);
3    pstmt.setString(1, log);
4    pstmt.setString(2, pw);

Lors des instructions 3 et 4, tu essayes de donner à des variables de ton PreparedStament les valeurs des données log et pw respectivement. Or ces variables n'existent pas puisque ta requête ne les défini pas. En fait, ta requête devrait être formée de la manière suivante:

"SELECT * FROM user WHERE login = :1 AND password = :2"

(je ne suis plus 100% certain de la syntaxe de déclaration des variables, c'est peut être des ? au lieu des :index mais la Javadoc te permettra de vérifier ça sans problème). Sans ça, l'utilisation du PreparedStatment est quasiment inutile et risque même de pas mal surcharger le serveur base de données car celui-ci va conserver des requêtes précompilées pour chaque combinaison de (login, password) qui lui est envoyée.

Ensuite, il y a de manière latente un problème dans ce que tu envois car, comme toutes les ressources, celles qui permettent de manipuler une base de données doivent être libérées.

Si tu fais:

Code :

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

1
2
3
4
5
6
7
String req="SELECT * FROM user WHERE ( login ="+log+"and password ="+pw+");";
PreparedStatement pstmt = connection.prepareStatement(req);
pstmt.setString(1, log);
pstmt.setString(2, pw);
ResultSet rs = pstmt.executeQuery();
boolean exist = rs.next();
return exist;

Ton ResultSet et ton PreparedStatment ne sont pas fermés. Pour le ResultSet, c'est simple, il suffit d'ajouter un rs.close() avant de quitter la méthode de validation de la connexion. Pour le Statment, c'est autre chose parce que tout l'intérêt est justement de le conserver temps qu'il peut être utilisé. Il faut peut être alors le créer ailleurs que dans la méthode de validation et il faut ensuite déterminer le bon moment pour le libérer.