Problème insertion BDD

**supstoov** · 14/06/2011, 11h07

Bonjour,

Je vous demande votre aide car j'ai une application que j'ai crée avec swing connecté à une base de données Postgresql et j'essaye d'inserer des donner dans cette base avec les JTextfield de mon apllication en utilisant un bouton Enregistrer .Je vous donne un bout de mon code pour cette partie et j'espère avoir votre aide svp.

le code c'est:

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
private void EnregistrerClient() {
 
            boutonenregistrer=new JButton();
            boutonenregistrer.addActionListener(this);
 
            String data ="jdbc:odbc:vraiconnexion"; 
            saisieIdclient  = new JTextField(10);
            saisieNom         = new JTextField(10);
            saisiePrenom    = new JTextField(10);
            saisieAdresse     = new JTextField(10);
            saisieTelephone = new JTextField(10);
 
 
            try { 
            String codeclient,nom,prenom,adresse,telephone; 
 
            codeclient=saisieIdclient.getText();
            nom          = saisieNom.getText(); 
            prenom          = saisiePrenom.getText(); 
            adresse         = saisieAdresse.getText(); 
            telephone      =saisieTelephone.getText(); 
            Connection conn = DriverManager.getConnection(data); 
            Statement state = conn.createStatement(); 
            state.executeUpdate("INSERT INTO tabclients VALUES (saisieNom.getText(),saisiePrenom.getText(),saisieAdresse.getText(),saisieTelephone.getText()')");
            state.close(); 
 
            } 
 
            catch(SQLException s){ 
 
            System.out.println("SQL Error:" + s.toString() +" " 
 
            + s.getErrorCode() + " " + s.getSQLState()); 
            }
 
        }

et pour l'action de bouton c'est:

Code :

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

1
2
3
4
5
6
7
 
 else if(e.getSource()==boutonenregistrer)
         {
 
             EnregistrerClient();
 
         }

et merci.

**guillaumerivière7** · 14/06/2011, 11h58

Bonjour,

Tu exécute ça :

Code :

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

state.executeUpdate("INSERT INTO tabclients VALUES (saisieNom.getText(),saisiePrenom.getText(),saisieAdresse.getText(),saisieTelephone.getText()')");

Essaye de mettre ta requête dans un String et de te l'afficher avant de l'exécuter.

**supstoov** · 14/06/2011, 14h59

j'ai essayé votre méthode avec ce code mais ça marche pas et je vois pas le problème .

le code c'est:

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
private void EnregistrerClient() {
 
            boutonenregistrer=new JButton();
            boutonenregistrer.addActionListener(this);
 
 
            saisieIdclient  = new JTextField(10);
            saisieNom         = new JTextField(10);
            saisiePrenom    = new JTextField(10);
            saisieAdresse     = new JTextField(10);
            saisieTelephone = new JTextField(10);
 
 
            try {
 
 
 
 
 
 
                Connection conn = null;
                Statement requete1= conn.createStatement();
                requete1.executeUpdate("INSERT INTO tebclients ('saisieIDclient.getText()','saisieNom.getText()','saisiePrenom.getText()','saisieAdresse.getText()','saisieTelephone.getText()')");
                String requete = "INSERT INTO tebclients ('saisieIDclient.getText()','saisieNom.getText()','saisiePrenom.getText()','saisieAdresse.getText()','saisieTelephone.getText()'))";
                requete1.executeUpdate(requete);
                requete1.close();
 
            } catch (SQLException e) {
 
                e.printStackTrace();
            }
}

**guillaumerivière7** · 14/06/2011, 15h06

Tu as vu ce que donne la requête avant de s'executer??

essaye avant d'executer ta requête :

System.out.println(requete);

En fait il y a 1 problème de concaténation dans ton String...

Essaye ça :

Code :

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

String requete = "INSERT INTO tebclients VALUES('"+saisieIDclient.getText()+"','"+saisieNom.getText()+"','"+saisiePrenom.getText()+"','"+saisieAdresse.getText()+"','"+saisieTelephone.getText()+"'))";

**supstoov** · 14/06/2011, 15h51

j'ai essayé ta méthode mais ça marche pas je te donne le 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
private void EnregistrerClient() {
 
            boutonenregistrer=new JButton();
            boutonenregistrer.addActionListener(this);
 
 
            saisieIdclient  = new JTextField(10);
            saisieNom         = new JTextField(10);
            saisiePrenom    = new JTextField(10);
            saisieAdresse     = new JTextField(10);
            saisieTelephone = new JTextField(10);
 
 
            try {
 
 
                Connection conn = null;
                Statement requete1= conn.createStatement();
                requete1.executeUpdate("INSERT INTO tebclients ('saisieIDclient.getText()','saisieNom.getText()','saisiePrenom.getText()','saisieAdresse.getText()','saisieTelephone.getText()')");
                String requete = "INSERT INTO tebclients ('"+saisieIdclient.getText()+"','"+saisieNom.getText()+"','"+saisiePrenom.getText()+"','"+saisieAdresse.getText()+"','"+saisieTelephone.getText()+"')";
 
                requete1.executeUpdate(requete);
 
                requete1.close();
 
            } catch (SQLException e) {
 
                e.printStackTrace();
            }
}

mais je pense que peut j'ai un problème avec mon action performed et le code c'est:

Code :

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

1
2
3
4
5
6
else if(e.getSource()==boutonenregistrer)
         {
 
             EnregistrerClient();
 
         }

vous pensez que mon problème viens de l'action performed?Merci.

**guillaumerivière7** · 14/06/2011, 15h58

Quel est le message d'erreur qui est affiché?

**supstoov** · 14/06/2011, 16h32

justement pas de message ,là je comprend rien surtout que ma connexion passe

bien avec Postgresql mais pas de resultat avec ma classe application qui hérite de la classe connect.

**supstoov** · 15/06/2011, 10h05

Alors j'attend toujours votre aide les amis

**sinok** · 15/06/2011, 12h53

Bonjour
On va faire simple.

Pour commencer, ta méthode enregistrerClient n'a aucun sens, tu initialise tes JTextFields dedans, donc par définition ces derniers seront vide.

Cette méthode ne doit contenir que le code d'insertion des données en base, et surtout pas de manipulation de ton interface graphique.

Puis dans les dernière version de ton code tu ne crées même plus la connexion à la base, tu te contentes d'un

Code :

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

Connection conn = null;

. Il est donc bien évident que tout appel à une méthode de conn te lancera une NullPointerException.

En ce qui concerne ta requête, beh...
Par définition le contenu d'une chaine n'est pas du code.
Donc java ne cherche pas à exécuter le contenu de cette chaine (c'est tout le but d'une chaine d'ailleurs: de stocker des caractères).

De plus, malgré le fait que tu aies corrigé cela sur les conseils avisés de guillaumerivière7, tu as laissé l'appel suivant dans ton code:

Code :

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

requete1.executeUpdate("INSERT INTO tebclients ('saisieIDclient.getText()','saisieNom.getText()','saisiePrenom.getText()','saisieAdresse.getText()','saisieTelephone.getText()')");

Ce qui forcément pose problème.

Donc mettre des appels de méthode dans une chaine comme tu le fais, bah ça ne sert absolument à rien (sauf à rendre ta requête absolument inexécutable par le SGBD).

Dans ton cas je te conseille très fortement de te diriger vers les PreparedStatement qui sont la façon propre de faire

Dans ton cas:

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
Connection conn = null
PreparedStatement stmt = null;
 
String data ="jdbc:odbc:vraiconnexion"; 
 
try {
    conn = DriverManager.getConnection(data); 
    stmt= conn.prepareStatement("INSERT INTO tebclients (ID,NOM,PRENOM,ADRESSE,TELEPHONE) VALUES (?,?,?,?,?)");
    stmt.setString(1,saisieIdclient.getText());
    stmt.setString(2,saisieNom.getText());
    stmt.setString(3,saisiePrenom.getText());
    stmt.setString(4,saisieAdresse.getText());
    stmt.setString(5,saisieTelephone.getText());
    stmt.executeUpdate();                    
} catch (SQLException e) {    
    e.printStackTrace();
} finally {
    try {
        if( stmt != null ) {
            stmt.close();
        }
        if( conn != null ) {
            conn.close();
        }
    } catch (SQLException e) {    
        e.printStackTrace();
    }
}

Tout le code concernant ton interface graphique devrait être exécuté dans le constructeur de ta classe, cela suffira amplement.
A noter la clause finally utilisée pour être sûr que le statement et la connexion seront fermés correctement, que l'insertion aie réussi ou non.

Donc bref, je te conseille

De relire tes cours de base de java avant de foncer tête baissée dans ton code. Définitivement il y a des notions que tu n'as pas acquis, notions qui sont un tant soit eu nécessaires quand on commence à toucher aux API avancées (genre Swing, JDBC et compagnie)
De prendre le temps de réfléchir un peu, voir de poser un petit dessin sur papier des divers éléments sur ton papier, car là tu es totalement en train de t'embrouiller.

**supstoov** · 15/06/2011, 15h16

Merci pour tes conseils Sinok mais je dois pas faire une méthode pour l'action de mon bouton enregistrer?.je te donne mon nouveau code et j'espère trouver une solution car j'avance pas.

mon nouveau code c'est:

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
private void EnregistrerClient() throws ClassNotFoundException {
			boutonenregistrer=new JButton();
			boutonenregistrer.addActionListener(this);
			Connection conn = null;
			PreparedStatement stmt = null;
 
			String data ="jdbc:odbc:vraiconnexion"; 
			Class.forName("org.postgresql.Driver");
			System.out.println("DRIVER OK ! ");
 
			String url = "jdbc:postgresql://localhost:5433/tabclient";
			String user = "postgres";
			String passwd = "anissa39";
			try {
 
 
			    conn = DriverManager.getConnection(data); 
			    stmt= conn.prepareStatement("INSERT INTO tebclients (ID,NOM,PRENOM,ADRESSE,TELEPHONE) VALUES (?,?,?,?,?)");
			    stmt.setString(1,saisieIdclient.getText());
			    stmt.setString(2,saisieNom.getText());
			    stmt.setString(3,saisiePrenom.getText());
			    stmt.setString(4,saisieAdresse.getText());
			    stmt.setString(5,saisieTelephone.getText());
			    stmt.executeUpdate();                    
			} catch (SQLException e) {    
			    e.printStackTrace();
			} finally {
			    try {
			        if( stmt != null ) {
			            stmt.close();
			        }
			        if( conn != null ) {
			            conn.close();
			        }
			    } catch (SQLException e) {    
			        e.printStackTrace();
			    }
			}
		}

Merci.

**sinok** · 15/06/2011, 17h28

Même règle que pour les JTextFields, direction le constructeur.

En effet, pour que la méthode EnregistrerClient soit appelée, il faut que le listener soit déjà enregistré sur ton bouton. De fait l'ajout du listener au bouton (qui nécessite que le JButton soit initialisé bien sûr) doit être fait avant l'appel de la méthode EnregistrerClient.

**sinok** · 15/06/2011, 17h28

Même règle que pour les JTextFields, direction le constructeur.

En effet, pour que la méthode EnregistrerClient soit appelée, il faut que le listener soit déjà enregistré sur ton bouton. De fait l'ajout du listener au bouton (qui nécessite que le JButton soit initialisé bien sûr) doit être fait avant l'appel de la méthode EnregistrerClient.

Un peu de logique que diantre...

**supstoov** · 15/06/2011, 17h47

j'ai mis un action listener pour mon bouton enregistrer avec ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
class BoutonListener implements ActionListener{
 
            public void actionPerformed(ActionEvent e) {
 
                    System.out.println("nom : "  + saisieNom.getText());
                    //System.out.println("TEXT : nom " + saisieNom.getText());
                    //System.out.println("TEXT : prenom "  + saisiePrenom.getText());
                    //System.out.println("TEXT : adresse "  + saisieAdresse.getText());
                    //System.out.println("TEXT : telephone "  + saisieTelephone.getText());
            }

et j'ai changé mon code qui est le suivant:

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
private void EnregistrerClient()   {
 
 
            PreparedStatement statement;
            Connection conn=null;
            try {
                Class.forName("org.postgresql.Driver");
 
            System.out.println("DRIVER OK ! ");
 
            String url = "jdbc:postgresql://localhost:5433/tabclient";
            String user = "postgres";
            String passwd = "anissa39";
 
            Connection    conn1 = DriverManager.getConnection(url, user, passwd);
 
 
 
 
 
 
               String requete= "INSERT INTO tebclients (ID,NOM,PRENOM,ADRESSE,TELEPHONE) VALUES (?,?,?,?,?)";
 
                   statement=conn.prepareStatement(requete);
                statement.setString(1,saisieIdclient.getText());
                statement.setString(2,saisieNom.getText());
                statement.setString(3,saisiePrenom.getText());
                statement.setString(4,saisieAdresse.getText());
                statement.setString(5,saisieTelephone.getText());
                statement.executeUpdate();                    
            } catch (SQLException e) {    
                e.printStackTrace();
 
            } catch (ClassNotFoundException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
        }

mais j'ai une exception de java.lang.NullPointerException et je vois pas la solution .une piste s'il vous plait.

**sinok** · 15/06/2011, 18h09

Dans quelle classe et à quelle ligne ?

**supstoov** · 15/06/2011, 18h22

Dans la classe application ligne 151 qui corresspond à ce code :

Code :

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

enregistrerInformationClient.addActionListener(new BoutonListener());

Sinon j'ai une seule classe avec la classe connect ,je dois peut être créer une classe insertion pour me faciliter la tâche .vous en pensez quoi pour ma méthode de travail?

**sinok** · 15/06/2011, 18h49

ton JButton enregistrerInformationClient n'a pas été initialisé. Grosso modo la ligne

Code :

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

enregistrerInformationClient = new Jbutton("Enregistrer");

n'a jamais été exécutée.

Problème insertion BDD

JDBC Java

Discussions similaires

Partager

Partager