Discussion :

[Toogda]

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 :

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
 
 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:
c'est là que j'ai besoin d'aide.
cordialement!!!

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

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

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.

[Toogda]

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

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

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

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

Comment puis-je corriger?

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

[joel.drigo]

Code :

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

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.

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 ?

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 ?

[Toogda]

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

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.

..mais pourquoi faire ?

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

[joel.drigo]

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.

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 ?

[Toogda]

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 :

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

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

[joel.drigo]

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 !

[Toogda]

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

Code :

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

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

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

[cobos]

Tu peu afficher le message d'erreur?