comparer les résultats d'une requete sql

Version imprimable

02/03/2008, 01h08
oceane751

comparer les résultats d'une requete sql

bonjour à tous!!

j'aimerai comparer les resultats d'une requete SQL avec des elements provenant de plusieurs JTextField

par exemple
j'ai un JTextField "nomm" et "prenomm"
et ma requete est : SELECT nom, prenom FROM membres";

donc j'aimerai comparer nomm avec nom et prenomm avec prenom

si quelqu'un pouvait m'aider, se serait vraiment tres sympa car là j'ai aucune idée, aucune piste..

merci
02/03/2008, 09h24
guigui5931

Il y a apparement plusieurs problème. Pour ce qui est d'exécuter ta requête SQL je t'invite à faire un tour sur la FAQ JDBC et tu trouvera comment faire un requete et en récupérer les résultats.
Pour ce qui est du JTextField la méthode getText permet de récupérer le texte qu'il contient.
Pour ce qui est de la comparaison entre les 2 chaines tu dois faire attention à utiliser la méthode equals (et non pas ==).

pour tout ce qui concerne la requete et le getText() je connais et j'ai fait
j'ai tout simplement pas voulu mettre tout le code, car je n'ai pas trouver cela necessaire

en regardant la FAQ, j'ai trouvé ce que je cherchais
Code:

1 2 3 4 5 6 7 8 ResultSet resultat = statement.executeQuery(sql); while(resultat.next()){ ==> c'estcela que je cherchais int id = resultat.getInt(1); String nom = resultat.getString(2); double prix = resultat.getDouble(3);

03/03/2008, 00h29
oceane751
bon finalement je crois qu'il y a un soucis

normalement lorsque l'on boucle sur els résultats de la requete et qu'on veux les recuperer, il devrait apparaitre l'ensemble des resultats et non pas le 1er enregistrements
j'ai fait, comme dans la FAQ :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 while(res.next()) { String heureReq = res.getString(1); System.out.println(heureReq); String dateReq = res.getString(2); System.out.println(dateReq);
suis peut être à coté de la plaque :aie:
03/03/2008, 14h46
Duc Lebowski

Effectivement le while(rs.next()) boucle sur tous les enregistrements.

Si tu ne rentres qu'une seule fois dans la boucle, c'est que ta requête SQL ne retourne qu'un seul enregistrement.
03/03/2008, 18h08
oceane751

donc il faut que je boucle avec un for dans la boucle while ?
03/03/2008, 18h12
DevTeam_

Citation:

Envoyé par oceane751

donc il faut que je boucle avec un for dans la boucle while ?

:koi:
03/03/2008, 18h19
jhaythem

poste ton code,
juste en fesant un while(rst.next()) tu parcours ligne par lignes tes enregistrements.
03/03/2008, 19h18
oceane751

ba oui je sais c'est ridicule mais franchement je comprends paspourquoi avec une boucle ça ne boucle pas

Citation:

Envoyé par DevTeam_

:koi:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
public void actionPerformed(ActionEvent e) 
	  {
		if (e.getSource() == but)
		{
		Fonction fonc = new Fonction();
		String requetee = "SELECT heure, date_rdv FROM membres ";
		ResultSet res = fonc.Conec(requetee);
 
			try {
				while(res.next())
				{
 
 
				String heureReq = res.getString(1);
				System.out.println(heureReq);
				String dateReq = res.getString(2);
				System.out.println(dateReq);
 
 
.........

03/03/2008, 22h11
guigui5931

pour moi si tu n'as qu'une ligne de résultat c'est que ta requete ne renvoi qu'une seule ligne. Essaye de la jouer directement sur ta base pour vérifier.
Si elle te renvoi plus d'une ligne donne nous le code de ta classe Fonction pour voir ce que fait la méthode Conec.

oui oui il y a plein d'enregistrements dans ma bdd

alors ma classe Fonction

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
 
public class Fonction   {
 
 
	public Fonction()
	{
 
	}
 
 
public ResultSet Conec (String requet)
 
	{
 
		String pilote = "com.mysql.jdbc.Driver";
 
		try{
 
		Class.forName(pilote);
		Connection connexion = DriverManager.getConnection("jdbc:mysql://localhost/membre","jo","oui");
		Statement instruction = connexion.createStatement();
		ResultSet resultat = instruction.executeQuery(requet);
		return resultat;
 
		}
		catch (Exception e){
 
			System.out.println("echec pilote : "+e);
		}
		return null;
 
	}
}

04/03/2008, 08h45
DevTeam_

Citation:

Envoyé par oceane751

ba oui je sais c'est ridicule mais franchement je comprends paspourquoi avec une boucle ça ne boucle pas

J'abonde dans le sens de guigui5931, essaie d'exécuter ta requête directement sur ta base de données...

Poste également la suite de ta boucle while, des fois que tu aies mis une instruction qui vrille le resultset

Vu que tu crées ta connection dans une autre méthode et que tu ne la donne pas en retour, j'ai bien peur qu'elle soit automatiquement fermée derrière :aie:.

A chaque fois c'est le même problème, donc je répète que le code permettant d'exécuter une requête SQL doit TOUJOURS être de la forme suivante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
Connection conn = null;
Statement stmt = null;
ResultSet rs = null;
try
{
    conn = DriverManager.getConnection(..., ..., ...);
    stmt = conn.createStatement();
    rs = stmt.executeQuery(...);
    while (rs.next()) { ... }
}
catch (SQLException e)
{
    e.printStackTrace();
}
finally
{
    if (rs != null)
        try{rs.close();} catch(SQLException e){}
    if (stmt != null)
        try{stmt.close();} catch(SQLException e){}
    if (conn != null)
        try{conn.close();} catch(SQLException e){}
}

Un des moyens que j'ai trouvé pour "capitaliser" du code est de faire un truc comme ça :

Interface pour remplir les résultats :

Code:

1
2
3
interface ResultMap<T> {
    public T fill(ResultSet rs) throws SQLException;
}

Bean représentant le résultat :

Code:

1
2
3
4
5
6
7
class BeanRes
{
    private String heure;
    private String date;
    public BeanRes() {}
    // GET/SET
}

Gestion de la connection/requêtes :

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
class DAOManager
{
    void executeQuery(String sql, ResultMap rm)
    {
        Connection conn = null;
        Statement stmt = null;
        ResultSet rs = null;
        try
        {
            conn = DriverManager.getConnection(..., ..., ...);
            stmt = conn.createStatement();
            rs = stmt.executeQuery(...);
            while (rs.next()) {
                rm.fill(rs);
            }
        }
        catch (SQLException e)
        {
            e.printStackTrace();
        }
        finally
        {
            if (rs != null)
                try{rs.close();} catch(SQLException e){}
            if (stmt != null)
                try{stmt.close();} catch(SQLException e){}
            if (conn != null)
                try{conn.close();} catch(SQLException e){}
        }
    }
}

Exemple main/exécution d'une requête :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
Collection<BeanRes> results = 
    DAOManager.getInstance().executeQuery(
        "SELECT heure, date_rdv FROM membres",
        new ResultMap<BeanRes>(){
    public BeanRes fill(ResultSet rs)
        throws SQLException
    {
        BeanRes br = new BeanRes();
        br.setDate(rs.getString(1)); 
        br.setHeure(rs.getString(2));
        return br;
    }
});

L'étape suivante consiste à faire un truc du genre :

Code:

1
2
3
Collection<BeanRes> result =
    DAOManager.getInstance().executeQuery(
        "SELECT heure, date_rdv FROM membres", BeanRes.class);

Mais après celà veut dire qu'il faut passer à hibernate dans ce cas ;)

04/03/2008, 11h28
DevTeam_

Totalement d'accord sur le probleme de structuration du code, mais pour son souci, je ne pense pas que la connexion soit la cause puisqu'elle parvient à récupérer 1 enreg ! Si la connexion était fermée en sortie de méthode, le 1er res.next() lèverait une exception non ?
04/03/2008, 11h34
Duc Lebowski

Le connexion n'est peut-être pas fermée immédiatement.

Il me semble qu'elle peut se fermer au passage du garbage collector sur le finalize(), ou avec un timeout si elle n'est pas appelée genre pendant 10 sec...

Celà est du au fait que les 3/4 des développeurs ne ferment jamais leurs connexions explicitement avec conn.close()...

Ca dépend aussi certainement des implémentations fournies par les SGBD...

Mais tu as raison, une exception serait certainement levée dans ce cas.

En réalité, je pense tout simplement que sa requête renvoi un seul record ;)

Mais je voulais juste au passage revenir sur la syntaxe parce que je m'apperçoit que beaucoup cherchent à capitaliser du code à fond sur l'exécution d'une requête SQL et que généralement ça donne du grand nimporte quoi :mouarf:
04/03/2008, 13h56
DevTeam_

Dans le cas d'oceane, je ne pense pas que la connexion puisse être garbagée puisque le resutlset référence son statement, qui lui référence cette connexion... Mais dans l'absolu, tu as surement raison.

Citation:

Envoyé par Duc Lebowski

En réalité, je pense tout simplement que sa requête renvoi un seul record ;)

Mais je voulais juste au passage revenir sur la syntaxe parce que je m'apperçoit que beaucoup cherchent à capitaliser du code à fond sur l'exécution d'une requête SQL et que généralement ça donne du grand nimporte quoi :mouarf:

Je plussoie avec vigueur !

oui mais comment se fait il qu'avec ce code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
Fonction fl = new Fonction();
		String re = "SELECT nom, heure FROM membres WHERE date_rdv="+"\""+datee+"\"";
		ResultSet y = fl.Conec(re);
 
		try {
			while( y.next())
			{
				String nom =  y.getString("nom");
				String heure = y.getString("heure");
 
 
		listeItems.addElement("RDV avec "+nom+  " " + "à" +heure);
 
			}
		} catch (SQLException e)
		{
		    e.printStackTrace();
		}

ça me renvoit tous les resultats ? alors que c'est pratiquement le meme code
:roll:

je vous mets le bout de code en entier

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
public void actionPerformed(ActionEvent e) 
	  {
		if (e.getSource() == but)
		{
			Fonction fonc = new Fonction();
			String requetee = "SELECT heure, date_rdv FROM membres ";
			ResultSet res = fonc.Conec(requetee);
 
			try {
				while(res.next())
				{
					String heureReq = res.getString("heure");
					System.out.println(heureReq);
					String dateReq = res.getString("date_rdv");
					System.out.println(dateReq);
 
 
 
 
 
 
					if ( (heureReq == textheure.getText()) && (dateReq == textdate.getText()) )
					{	
 
						JOptionPane.showMessageDialog(null, "dejà une RDV pour cette heure","deja une RDV pour cette heure", JOptionPane.NO_OPTION );
						break;
 
 
 
 
					}
					else
					{
						String requete = "INSERT INTO membres (nom, heure, date_rdv) VALUES('"+textnom.getText()+"', '"+textheure.getText()+"', '"+textdate.getText()+"')";   
						//System.out.print(requete);
 
						Fonction foncc = new Fonction();
						int y = foncc.maj(requete);
 
						if (y != 0)
						{
							Disposition dispo = new Disposition();
							dispo.setVisible(true);
							setVisible(false);
							JOptionPane.showMessageDialog(null, "insertion bien effectuéé","insertion bien effectuéé", JOptionPane.OK_OPTION ); 
						break;
						}
					}
				}
			}
 
 
 
 
			catch (SQLException e1) 
			{
				// TODO Auto-generated catch block
				e1.printStackTrace();
			}
 
 
 
 
		} 
}

en faite en fonction de l'heure et de la date entrées, je verifie si pour une date donnée, dans la bdd, il n'y a pas dejà un rdv pour l'heure voulue
dans ce cas, la requete d'insertion ne se fera pas.

c'est sur la 1ere requete

Code:

1
2
 
String requetee = "SELECT heure, date_rdv FROM membres ";

que pose le problème, cad que je n'arrive pas à "sortir" tous les resultats afin de faire les comparaison

Code:

1
2
 
if ( (heureReq == textheure.getText()) && (dateReq == textdate.getText()) )

05/03/2008, 08h38
DevTeam_

Le break que tu as mis dans ton code te fera sortir de ta boucle while, donc si, dans ton exemple, tu arrives à faire ta mise à jour (y != 0), alors tu va passer dans le break, et c'est pour ça que tu dis n'avoir qu'1 seul enregistrement retourné...

D'où l'importance de bien mettre tout son code avant de mettre des phrases du genre "ça me renvoit tous les resultats ? alors que c'est pratiquement le meme code" et d'utiliser le smiley :roll:, ça pourrait exaspérer les personnes de ce forum qui passent du temps à t'aider.

Enfin,

Code:

heureReq == textheure.getText()

ça ne marchera jamais....... Va faire un tour sur le tutoriel "Développons en Java (La syntaxe et les éléments de bases de Java)" :
http://jmdoudoux.developpez.com/java...e_java#L3.10.3

ok
j'ai enlevé les break; et j'ai mis le .equals() et ça marche pas plus
j'avais mis les break; car quand j'insere, ça me mets un message en boucle, "insertion bien effectuee" X fois

j'ai pas envie d'enerver qui que se soit j'ai juste envie de comprendre pourquoi ça ne me compare rien du tout

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
 
public void actionPerformed(ActionEvent e) 
	  {
		if (e.getSource().equals( but) )
		{
			Fonction fonc = new Fonction();
			String requetee = "SELECT heure, date_rdv FROM membres ";
			ResultSet res = fonc.Conec(requetee);
 
			try {
				while(res.next())
				{
 
 
 
					String heureReq = res.getString("heure");
					System.out.println(heureReq);
					String dateReq = res.getString("date_rdv");
					System.out.println(dateReq);
 
 
					if ( (heureReq.equals(textheure.getText())) && (dateReq.equals(textdate.getText())) )
					{	
 
						JOptionPane.showMessageDialog(null, "dejà une RDV pour cette heure","deja une RDV pour cette heure", JOptionPane.NO_OPTION );
						//break;
 
 
					}
					else
					{
						String requete = "INSERT INTO membres (nom, heure, date_rdv) VALUES('"+textnom.getText()+"', '"+textheure.getText()+"', '"+textdate.getText()+"')";   
						//System.out.print(requete);
 
						Fonction foncc = new Fonction();
						int y = foncc.maj(requete);
 
						if (y != 0)
						{
							Disposition dispo = new Disposition();
							dispo.setVisible(true);
							setVisible(false);
							JOptionPane.showMessageDialog(null, "insertion bien effectuéé","insertion bien effectuéé", JOptionPane.OK_OPTION ); 
						//break;
						}
					}
				}
			}
 
 
 
 
			catch (SQLException e1) 
			{
 
				e1.printStackTrace();
			}
 
 
 
 
		} 
}

05/03/2008, 17h12
DevTeam_

Le fait que ça te mette X fois "insertion réussie" montre que désormais ton res.next() parcourt bien toutes les lignes de ta table. Encourageant !

Ensuite, le fait que ta comparaison plante peut venir des formats de ton heure et de ta date... Peux-tu afficher
1) les dates & heures issues de la base (heureReq et datereq)
2) les dates & heures issues de je-ne-sais-pas-où (textheure.getText() et textdate.getText())
et poster le résultat sur le forum ?
05/03/2008, 17h14
DevTeam_

Citation:

Envoyé par oceane751

j'ai pas envie d'enerver qui que se soit

T'inquiète, j'ai pris mes petites pilules roses ;)

Citation:

Envoyé par oceane751

j'ai juste envie de comprendre pourquoi ça ne me compare rien du tout

Très bonne intention ! Mais j'ai l'impression que tu t'attaques à difficile sans avoir compris les bases de Java, c'est pas toujours la bonne politique.

Par ailleurs, si tu utilises un IDE tel qu'Eclipse, le debugger peut t'être d'un grand secours (points d'arrêt pour voir ce que valent tes variables, exécution pas à pas, ...).

Je pense que des tutoriaux bien faits sont dispos sur le net.
05/03/2008, 17h37
Trencavel

Je pense que le probleme n'est pas forcement technique mais plus algorythmique.

Si j'ai bien compris ton code tu cherche a savoir si il y a deja un rendez vous a l'heure et a la date indique et s'il n'y en a pas d'ajouter un enregistrement a ta table.

Or avec ce code pour chaque ligne de ta table qui n'as pas la bonne heures tu va rajouter une ligne a ta table ( en gros tu va quasi doubler le nombre de ligne a chaque execution 8O)

en fait lieu de "select heure,date_rdv from menbre" je suppose que la requete
"select heure,date_rdv from menbre where heure='"+textheure.getText()+"' and date_rdv='"+textdate.getText()+"'" fonctionneras mieux.
En effet dans ce cas s'il n'a pas de rendez vous tu ne rentres pas dans le while. En fait tu peut juste utiliser (if (!rs.next()) then insert ....).

Ps : avant tout test checke ta table car tes donnee sont peut etre legerement corrompu :roll:
05/03/2008, 17h48
oceane751

Citation:

Envoyé par DevTeam_

Le fait que ça te mette X fois "insertion réussie" montre que désormais ton res.next() parcourt bien toutes les lignes de ta table. Encourageant !

Ensuite, le fait que ta comparaison plante peut venir des formats de ton heure et de ta date... Peux-tu afficher
1) les dates & heures issues de la base (heureReq et datereq)
2) les dates & heures issues de je-ne-sais-pas-où (textheure.getText() et textdate.getText())
et poster le résultat sur le forum ?

je me suis gravement facilité la vie a ne pas utiliser de type DATETIME ou de type DATE (je sais c'est pas tres bien et pas rès logique non plus mais bon)
car là ça aurait été beaucoup plus difficile , déjà que je galère avec des choses simples...
05/03/2008, 17h56
DevTeam_

Citation:

Envoyé par oceane751

je me suis gravement facilité la vie a ne pas utiliser de type DATETIME ou de type DATE (je sais c'est pas tres bien et pas rès logique non plus mais bon)
car là ça aurait été beaucoup plus difficile , déjà que je galère avec des choses simples...

Es-tu sure de la corrélation entre les valeurs (textes) des dates & heures que tu saisies et celles récupérées de ta base ??? Car c'est la seule et unique raison pour laquelle ton test d'égalité ne fonctionne pas !

Comme je te l'ai proposé, fais un "System.out.println" de tes valeurs, ainsi tu verras bien si elles sont égales ou pas.

Citation:

Envoyé par Trencavel

en fait lieu de "select heure,date_rdv from menbre" je suppose que la requete
"select heure,date_rdv from menbre where heure='"+textheure.getText()+"' and date_rdv='"+textdate.getText()+"'" fonctionneras mieux.

bon ta requete est bonne et ça fonctionne! j'ai bien le message qui me dit qu'un rdv a dejà été enregistré pour cette date et cette heure!

mais l'insertion se fait quand meme... normal

donc j'ai fait ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 int flag =0; try { if (flag == 0) { while(res.next()) { JOptionPane.showMessageDialog(null, "dejà une RDV pour cette heure","deja une RDV pour cette heure", JOptionPane.NO_OPTION ); break; } } else { String requete = "INSERT INTO membres (nom, heure, date_rdv) VALUES('"+textnom.getText()+"', '"+textheure.getText()+"', '"+textdate.getText()+"')";
voilou

merci à tous de m'avoir aidé!!!
:yaisse2: