Problème de requête SQL

Version imprimable

1 pièce(s) jointe(s)

salut mon but c'est d'arriver à obtenir les données d'une table de ma base de données (Mysql) et de l'afficher dans des JTextField.
pour cela je fais:
code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
 try
                    {
                       Statement st = con.createStatement();
                       String rq = "SELECT * FROM etudiant WHERE idEtudiant='"+Integer.valueOf(text.getText()).intValue()+"'";
                               /*'"+Integer.valueOf(text.getText()).intValue()+"'"; parce que mon  idEtudiant est de type INT */
                       ResultSet rs= st.executeQuery(rq);
 
                        matricule.setText(rs.getString("idEtudiant"));
                        nom.setText(rs.getString("nomEtudiant"));
                        prenom.setText(rs.getString("prenomEtudiant"));
                        sexe.setText(rs.getString("sexeEtudiant"));
                        filiere.setText(rs.getString("Filiere"));
                        con.close();
                    }
                    catch(SQLException c)
                     {
                        String error = "SQLState: " + c.getSQLState() 
                     + "\nError Code: " + c.getErrorCode() 
                     + "\nMessage: " + c.getMessage();
                        JOptionPane.showMessageDialog(this,"Erreur Survenue !\nVeuillez réessayer !!!\n\n"+error, "Erreur", JOptionPane.ERROR_MESSAGE);
                     }

Mais ça débouche toujours sur le catch(SQLException c)
erreur:Pièce jointe 290983
c'est là que j'ai besoin d'aide.
cordialement!!!

03/07/2017, 12h51
joel.drigo
Salut,

Avant de commencer à lire des données dans le ResultSet, il faut placer le curseur, même si tu es sûr qu'il y a un résultat au plus (un seul étudiant au plus avec l'id).
Code:

1 2 3 4 5 6 7 8 9 if ( rs.next() ) { // il y a bien au moins résultat matricule.setText(rs.getString("idEtudiant")); /*...*/ } else { // il n'y a pas de résultat... }
NB: je te conseille d'opter tout de suite pour les PreparedStatement au lieu des Statement classique.

ok Mais j'ai un peu de mal dans l'utilisation du PreparedStatement.
quand je fais

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public void SQLupdate(String sql)
 {
        Connection con = null;
        PreparedStatement pst = null;
        try{
            pst = con.prepareStatement(sql); //  <--- erreur ici!
            pst.executeUpdate();
            JOptionPane.showMessageDialog(this, "Opération effectuée avec succès!!");
            pst.close();
        }
        catch(SQLException c) {
           JOptionPane.showMessageDialog(this, "Impossible d'exécuter la requête" + e.getMessage());
		}
}

ce code signal une erreur lorsque je l'appelle comme ceci

Code:

1
2
3
 
 String rq = "INSERT INTO enseignant VALUES ('"+matricule.getText()+"', '"+nom.getText()+"', '"+prenom.getText()+"')";
  SQLupdate(rq);

Comment puis-je corriger?

05/07/2017, 15h12
Toogda

de plus est ce ke c'est possible de faire une methode generale au lieu de faire
une methode SQLupdate pour les ordre executeUpdate et
SQLquery pour les ordre executeQuery ?
05/07/2017, 15h39
joel.drigo
Citation:
Envoyé par Toogda

Code:

1 2 3 String rq = "INSERT INTO enseignant VALUES ('"+matricule.getText()+"', '"+nom.getText()+"', '"+prenom.getText()+"')"; SQLupdate(rq);
Cela ne sert pas à grand chose d'utiliser un PreparedStatement si c'est pour mettre la requête en dur dedans. Le but c'est, entre autres, d'éviter de traiter les quotes à la main et ce genre de chose.

Citation:

Envoyé par Toogda

ce code signal une erreur lorsque je l'appelle comme ceci
[...]
Comment puis-je corriger?

Si seulement je savais quelle erreur... à moins que cela ne soit tout simplement une NullPointerException parce que con est null ?

Citation:

Envoyé par Toogda

de plus est ce ke c'est possible de faire une methode generale au lieu de faire
une methode SQLupdate pour les ordre executeUpdate et
SQLquery pour les ordre executeQuery ?

Ce n'est pas impossible, mais pourquoi faire ?
05/07/2017, 18h12
Toogda

en fait j'avais oublier d'appeler la methode de connexion comme vous l'avez souligner (erreur corrigé).

Citation:

Cela ne sert pas à grand chose d'utiliser un PreparedStatement si c'est pour mettre la requête en dur dedans. Le but c'est, entre autres, d'éviter de traiter les quotes à la main et ce genre de chose.

Mais Si je place la requête directement ça m'empêcherais d'utiliser cette methode (SQLupdate(...)) pour des insertion dans d'autres tables.

Citation:

..mais pourquoi faire ?

Jaimerais modiffier SQLupdate(..) affin de pouvouir faire des 'executeQuery et executeUpdate ' avec cette même fonction.
05/07/2017, 18h51
joel.drigo

Citation:

Envoyé par Toogda

Mais Si je place la requête directement ça m'empêcherais d'utiliser cette methode (SQLupdate(...)) pour des insertion dans d'autres tables.

Oui, ça c'est sûr qu'il faut modifier la méthode.

Citation:

Envoyé par Toogda

Jaimerais modiffier SQLupdate(..) affin de pouvouir faire des 'executeQuery et executeUpdate ' avec cette même fonction.

Non, mais ça, j'ai bien compris. La question était pourquoi donc vouloir faire qu'une seule méthode pour faire deux choses différentes ?

Mais avant je tiens a formuler correctement une methode SQLquery()
//objectif : faire une recherche a partir d'un paramètre recceulli dans un JTextField
et afficher toute la ligne dans des JTextField.

donc:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
 public ResultSet SQLquery(String sql, int parametre)
    {
         Connection con = null;
         PreparedStatement pst = null;
         ResultSet rs = null;
        try {
           con = connecter();
        } catch (ClassNotFoundException ex) {
            Logger.getLogger(P_inscrire2.class.getName()).log(Level.SEVERE, null, ex);
        }
        try{
            pst = con.prepareStatement(sql);
            pst.setInt(1, parametre);
            rs=pst.executeQuery();
            JOptionPane.showMessageDialog(this, "Opération effectuée avec succès!!");
            pst.close();
        }
        catch(SQLException c) {
             JOptionPane.showMessageDialog(this, "Impossible d'exécuter la requête" + c.getMessage());
        }
        return rs;
    }

et pour l'appeller

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
if(evt.getSource()==rch)
        {
            ResultSet rs = null;
            String rq = "SELECT * FROM ETUDIANT WHERE idEtudiant=?";
            int pm = Integer.valueOf(matricule.getText()).intValue();
            System.out.println("valeur"+pm); // juste pour verifier si la valeur est bien prise. et c'est ok.
            rs = p.SQLquery(rq, pm);
 
                //matricule.setText(rs.getString("idEtudiant"));  // note erreur
// matricule étant un int je le mets en commentaire mais y'a toujours erreur sur la ligne suivante 
                nom.setText(rs.getString("nomEtudiant"));    // note erreur
                prenom.setText(rs.getString("prenomEtudiant"));
                sexe.setText(rs.getString("sexeEtudiant"));
                filiere.setText(rs.getString("Filiere"));
    }

Comment puis-je afficher la ligne de résultat obtenu correctement?

05/07/2017, 19h46
joel.drigo

Citation:

Envoyé par Toogda

Comment puis-je afficher la ligne de résultat obtenu correctement?

J'ai répondu à cette question, avec code en exemple, dans le message #2 !

mettre le if(..) ...else(..) ne resolvait pas mon problème

Code:

1
2
3
4
5
6
7
8
9
 
 if(rs.next()) {    //<------ l'execution echoe à ce niveau maintenant.
                matricule.setText(rs.getString("idEtudiant"));
               ...
            }
            else
            {
                ...
            }

06/07/2017, 08h15
joel.drigo

Tu ne peux pas lire d'information d'un enregistrement dans un ResultSet si tu n'as pas avancé le curseur dessus. C'est tout, c'est comme ça. Et c'est ce que dit le message d'erreur que tu as miontré. Le faire résoud ce problème. Ça ne veut pas dire qu'il n'y en a pas d'autres et que ça les résoud aussi.
02/08/2017, 10h25
cobos

Tu peu afficher le message d'erreur?