Erreur de requête

Bonjour tousss
J'avais un probleme dans ma fonction aurhentif, qui prends 2 parametres (login, password), je connecte à la base de données chateurs, et vérifie si ces parametres passés sont vrais ou pas. La requete ne s'exécute pas :cry:
Sachant que si je change ma requete, (select * from chateurs) et je faits l'affichage dans Main, ça se connecte et fonctionne, dans le probleme au niveau de la requte
Voici mon code:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39

import java.sql.*;   public class Authentif { public static int Authentif(String login1, String passwd1) { Connection c = null; int a = 10; PreparedStatement ps = null; ResultSet res = null; try {   String query = ("select * from chateurs where login="+login1+" and password="+passwd1+";"); Class.forName("sun.jdbc.odbc.JdbcOdbcDriver"); c = DriverManager.getConnection("jdbc:Odbc:EtudiantDB");   ps = c.prepareStatement(query); res = ps.executeQuery();     } catch (ClassNotFoundException e) { e.getStackTrace(); } catch (SQLException e) { e.getStackTrace(); }   if (res == null) a = 1; else a = 0; return a; }   public static void main(String[] args) { String login = "user1"; String password = "user1"; System.out.println(Authentif(login, password));   }   }

---

Il faut donner plus de détails pour que quelqu'un puisse t'aider .

Comment tu as su que ta requête ne s'éxécute pas : il ya une exception ou un comportement qui ne correspond pas à ton besoin ??

Merci frere Nizar
Bon,comme je vais ai dits, quands je change la requete en: String query= ("select * from chateurs"); ça fonctionne.
Mais quand je la rends:String query = ("select * from chateurs where login="+login1+" and password="+passwd1+";"); ça fonctionne pas.
Aussi j'ai ajoutés des println, devant quaque ligne. le println qui est juste aprés: res = ps.executeQuery(); ne s'affiche pas
j'espere que c clair.

Ta requête finale ne contient-elle pas une erreur de syntaxe ? Les ';' multiples dans la requêtes me paraissent suspects (bien que je ne sois pas un spécialiste du SQL). De plus, j'aurais rajouté des "quotes" autour de tes champs login et password :

Code:

---

"SELECT * FROM chateurs WHERE login='"+login1+"' and password='"+passwd1+"';")

---

Mieux encore, avec la classe PreparedStatement, utilise les requêtes paramétrées : c'est plus propre, et java gérera pour toi les problèmes de quotes (et de caractères spéciaux) :

Code:

---

"SELECT * FROM chateurs WHERE login= ? and password= ?"

---

Merci Ferere Modérateur
Je vais essayer avec les PreparedStatement, et si vous permetter svp de me donner la squelette généralle en bref.

voila, je l'ai faits comme vous avez dits, avec les PreparedStatement
et c la meme chose:
Il me rends tjrs 0 comme retour !!!:cry:
package Authentif;

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44

import java.sql.*;   public class Authentif2 { public static int Authentif(String login1, String passwd1) { Connection c = null; int a = 10; PreparedStatement ps = null; ResultSet res = null; String query = "select * from chateurs where login=? and password=?"; try {   Class.forName("sun.jdbc.odbc.JdbcOdbcDriver"); c = DriverManager.getConnection("jdbc:Odbc:EtudiantDB"); PreparedStatement pr = c.prepareStatement(query);     ps = c.prepareStatement(query); ps.setString(1, login1); ps.setString(2, passwd1); res = ps.executeQuery();     } catch (ClassNotFoundException e) { e.getStackTrace(); } catch (SQLException e) { e.getStackTrace(); }   if (res == null) a = 1; else a = 0; return a; }   public static void main(String[] args) { String login = "user1"; String password = "user1"; System.out.println(Authentif(login, password));   }   }

---

Voilà ce que tu as mis dans ton dernier post

Citation:

---

if (res == null)
a = 1;
else
a = 0;
return a;
}

---

C'est normal qu'il te retourne 0 puisque res n'est pas null .

En plus, si ce que tu veux faire c'est qu'il te retourne 0 si le resultset est vide et 1 sinon , je crois que ce qu'il faut faire :

Code:

---

1 2 3

if(res.next()) return 1 else return 0;

---

oui, je suis d'accords avec toi
Mais ou mettre cet instruction la?
- Si je la met dans le try{}, la fonction ne voit pas le "a" ,le compilateur me dit qu'il faut un retour pour cette fonction,
et si je la mets hors try{}, res.next() va générer une exception, donc je suis obligé de la mettre dans le try{}.
........... je suis vrmt bloqué :aie:

il suffit de mettre des return 0 dans chaque bloc catch{} après les e.printstacktrace()...

Merci ami pour ta suivie
mais je pense que m'avais pas compris
Je veux pas retourner 0 en fin de compte, je veux retourner ou bien 0( si la requete à du retour) ou bien1 (si ya pas de retour).
Car cette fonction je vais l'appeler dans le main, avec deux parametres
et tester son retour.
j'espere que tu m'a compris.
Ou bien c moi qui t'a pas compris?:roll:
Si c le cas ecrit moi sur le code stpp

tu as dit :

Citation:

---

Si je la met dans le try{}, la fonction ne voit pas le "a" ,le compilateur me dit qu'il faut un retour pour cette fonction,

---

je t'ai dit mets le dans le try{} et pour résoudre le problème

Citation:

---

le compilateur me dit qu'il faut un retour pour cette fonction

---

il faut mettre des return 0 ou return -1 par exemple dans tous les catch comme ça le compilateur ne te sortira plus cette erreur. ET dans ce cas s'il ya une exception il te retournera 0 (ou -1 si tu veux )

Donc pour résumer , le comportement de ta classe sera comme ceci :

Si pas d'exception et le resultset est vide il te retourne 0

si pas d'exception et le resultset n'est pas vide , il retourne 1.

JE crois que tu dois te documenter plus sur le langage java et tout ce qui concerne les connexions aux bases de données comme ça tu seras plus à l'aise pour faire tes programmes

Citation:

---

Envoyé par NizarK

JE crois que tu dois te documenter plus sur le langage java et tout ce qui concerne les connexions aux bases de données comme ça tu seras plus à l'aise pour faire tes programmes

---

Pareil pour moi... et si tu pouvais utiliser la balise [ code ] (bouton # dans l'éditeur) pour mettre tes portions de code, ce serait bien également...

A+

re salutttt
je vous remerciiiii
c bien résoluuuuu
Aussi j'avais une petite erreur sur ce code au niveau de la requete et je l'ai corrigé comme suit:
String query = "select * from chateurs where login='"+login1+"' and password='"+passwd1+"'";

Oui ça me rapelle quelque chose :

Regarde ICI : Plus haut dans cette discussion

Enfin, l'important est que tu aies résolu ton problème

N'oublies pas d'appuyer sur le bouton :resolu:

Bonjour

d'abord il faut que tu verifes si "login" et "password" sont ecrites ainsi sur ta base de données
puis enleve les parentheses tu met String query ="directemnt ta requete";

si ca ne marche pas tu verifie l'emplacement de tes cotes

essayes ca

Code:

---

1 2	String query="select * from chateurs where login="+login1+" and password="+passwd1;

---

bonne chance

ah d'accord
felicitation aors :)

Code:

---

String query = "select * from chateurs where login='"+login1+"' and password='"+passwd1+"'";

---

Cette syntaxe ne fonctionne que si login1 et passwd1 ne comporte pas d'apostrophe.

Exemples :

1) login1 = "Arthur", passwd1 = "secret" ....... ça marche :D

2) login1 = "Hubert d'Aubagne", passwd1 = "m'enfin" ....... ça plante :evilred:

Solutions :
- soit interdire la création d'utilisateur avec apostrophe
- soit doubler les apostrophes avec une instruction du type : login1 = login1.replaceAll("'","''") avant d'exécuter la requête (à vérifier en fonction du type de base de données).

Merci mesbzineb
(biensur que les 2 champs login et password se trouvet dans ma base de donnée)
Merci Népomucene pour ce truc d'appostrophe

Merci à toussssssssss
Que dieu vous bénisse
:king:

Citation:

---

Envoyé par Népomucène

Solutions :
- soit interdire la création d'utilisateur avec apostrophe
- soit doubler les apostrophes avec une instruction du type : login1 = login1.replaceAll("'","''") avant d'exécuter la requête (à vérifier en fonction du type de base de données).

---

Ou encore (ce que je conseillais dans un précédent post), utiliser les requêtes paramétrées de la classe PreparedStatement. Elles ont le bon goût de gérer les apostrophes et autres caractères spéciaux dans les chaînes de caractère, et ce indépendemment de la base de données !