Discussion :

[aniss77]

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
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 :

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
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));
 
	}
 
}

[NizarK]

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 ??

[aniss77]

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.

[CyberChouan]

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 :

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

"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 :

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

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

[aniss77]

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.

[aniss77]

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

Code :

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
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));
 
    }
 
}

[NizarK]

Voilà ce que tu as mis dans ton dernier post

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 :

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

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

[aniss77]

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é

[NizarK]

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

[aniss77]

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?
Si c le cas ecrit moi sur le code stpp

[NizarK]

tu as dit :

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

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

[OButterlin]

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+

[aniss77]

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+"'";

[NizarK]

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

[mesbzineb]

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 :

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

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

bonne chance

[mesbzineb]

ah d'accord
felicitation aors

[Népomucène]

Code :

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

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

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

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).

[aniss77]

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

[CyberChouan]

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 !