Discussion :

[Népomucène]

Pour le PreparedStatement c'est bien comme cela qu'il faut faire.
Le gros intérêt du truc c'est que cela te gère automatiquement les conversions de Date et de String qui comporte des "'

Bon de toute manière les utilisateurs ne verront rien... Et cela n'a pas l'air d'entraver le fonctionnement de l'application...

N'imagine pas une seconde que tu vas laisser cette bombe à retardement derrière toi !!!
Continue à faire les autres tests lundi sur le reste de l'application mais après il est obligatoire de trouver la cause du plantage.

[Spiicky]

Bonjour Népo, tu as passé un bon week end ?
De mon coté, j'ai continué à faire quelques testes...

Alors voilà, j'ai une fonction insertChefChantier() qui bloque.
Voici l'erreur :

Code :

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

1
2
3
4
5
java.lang.NullPointerException
        at coactivite2.ChefChantier.insertChefChantier(ChefChantier.java:149)
        at coactivite2.Fenetre.jButtonValiderAjoutCCActionPerformed(Fenetre.java:7343)
        at coactivite2.Fenetre.access$100(Fenetre.java:30)
        at coactivite2.Fenetre$2.actionPerformed(Fenetre.java:675)...

La ligne 149 correspond à ceci :

Code :

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

1
2
3
4
5
Connexion connexion = new Connexion();
            connexion.execSql(
                    "INSERT INTO ChefChantier (NomCC, PrenomCC, FonctionCC, TelCC, MailCC, IdSociete) " +
                    "VALUES ('"+nom+"', '"+prenom+"', '"+fonction+"', '"+telCC+"', '"+mail+"', "+idSociete+");");
connexion.closeConn();

Alors je me suis dit que c'était parce qu'elle n'était pas prepared ! (Vu que je test avec des mots de type "L'église"

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Connexion connexion = new Connexion();
            PreparedStatement ps = connexion.con.prepareStatement(
                    "INSERT INTO ChefChantier (NomCC, PrenomCC, FonctionCC, TelCC, MailCC, IdSociete) "
                    + "VALUES (?, ?, ?, ?, ?, ?);");
            ps.setString(1, nom);
            ps.setString(2, prenom);
            ps.setString(3, fonction);
            ps.setString(3, telCC);
            ps.setString(3, mail);
            ps.setInt(3, idSociete);
            ps.executeUpdate();
 
            connexion.closeConn();

Toujours le même problème, alors j'ai séparé la déclaration de l'instanciation :

Code :

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

1
2
3
4
PreparedStatement ps;
            ps = connexion.con.prepareStatement(
                    "INSERT INTO ChefChantier (NomCC, PrenomCC, FonctionCC, TelCC, MailCC, IdSociete) "
                    + "VALUES (?, ?, ?, ?, ?, ?);");

Et la ben, je sèche, ça ne va toujours pas...

Je me demande pourquoi je ne comprend pas ce genre d'erreur
Tu pourrais me mettre sur la piste stp ?

[Népomucène]

Code :

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

1
2
3
4
5
6
            ps.setString(1, nom);
            ps.setString(2, prenom);
            ps.setString(3, fonction);
            ps.setString(3, telCC);
            ps.setString(3, mail);
            ps.setInt(3, idSociete);

ça ne peut pas marcher !!!
tu mets plusieurs fois le même numéro !!!
(rassure-toi ça m'arrive aussi après un copié/collé mal fait)

[Spiicky]

Oulala... cette honte qui s’abat sur moi dès le matin...

J'ai fait un nouveau test et je retrouve la même erreur.
J'ai aussi essayer en partant de 0 mais non ...

MODIFIED :
Pour complément d'info, je récupère bien la valeur des variables.

Code :

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

1
2
3
4
5
6
System.out.print(nom+"\n");
            System.out.print(prenom+"\n");
            System.out.print(fonction+"\n");
            System.out.print(telCC+"\n");
            System.out.print(mail+"\n");
            System.out.print(idSociete+"\n");

Résultat:

Code :

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

1
2
3
4
5
6
test1
wdsfg2
qdsfvgwqs3
0606060606
test@gmail.com
21

[Spiicky]

J'ai trouvé mon problème !
J'ai été fouillé un peut dans ma classe Connexion et c'est vrai qu'on déclare Connexion.con mais la connexion est dans getConnexion()...

Du coup j'ai simplement remplacé :

Code :

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

PreparedStatement ps = connexion.con.prepareStatement(

par :

Code :

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

1
2
 
PreparedStatement ps = connexion.getConnexion().prepareStatement(

[Népomucène]

Connexion.con est déclarée "public" ?
Il faut la mettre en "private" pour que ce genre de blague ne se reproduise plus

[Spiicky]

Absolument... en public => correction effectuée.

Après modification, mon tableau devient totalement vide.
Est-ce un bug récurrent ou c'est propre à mon appli ?

[Népomucène]

Si ça se compile, alors le bug n'est pas du à la modification "public" -> "private"

[Spiicky]

En fait il faudrait que je fasse soit :
- Trouver ce bug ?
- Ou on s'embête pas et on créer une fonction de type RefreshTable qu'on appellerais après les manip ?

[Népomucène]

Ah je n'avais pas saisi ... c'est après modification d'une fiche société ?

[Spiicky]

A l'ajout, tout se passe bien, mais à la modification, les items n'apparaissent plus que l'ont décidé de valider la modification ou d'annuler !

Voici le code du bouton Modifier (si ca peut aider dans les recherches) :

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
    private void jButtonModifierChargeActionPerformed(java.awt.event.ActionEvent evt) {                                                      
        jTextFieldModifMatriculeCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 0));
        jTextFieldModifNomCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 1));
        jTextFieldModifPrenomCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 2));
        jTextFieldModifFonctionCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 3));
        jTextFieldModifTelCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 4));
        jTextFieldModifPortCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 5));
        jTextFieldModifFaxCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 6));
        jTextFieldModifMailCharge.setText((String) jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 7));
        if(jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 8) == "Non")
        {
            jCheckBoxModifAdminCharge.setSelected(false);
        }
        else
        {
            jCheckBoxModifAdminCharge.setSelected(true);
        }
        jDialogModifCharge.setVisible(true);
}

[Népomucène]

Et il n'y a pas de trace d'erreur ? Elle est peut-être interceptée avec un "catch" vide.
En principe il doit y avoir une méthode pour charger le tableau et cette méthode est appelée à l'ouverture puis en cas de modification / ajout / suppression.
Peux-tu l'identifier ?

[andry.aime]

Bonjour

J'ai pas lu la discussion depuis le début mais juste une remarque:

Code :

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

jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 8) == "Non"

Code :

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

"Non".equals(jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 8) ) // ou equalsIgnoreCase (dépendant de ton besoin)

A+.

[Népomucène]

Bonjour

J'ai pas lu la discussion depuis le début mais juste une remarque:

Code :

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

"Non".equals(jTableCharge.getValueAt(jTableCharge.getSelectedRow(), 8) ) // ou equalsIgnoreCase (dépendant de ton besoin)

A+.

Oui en effet, il faut faire cette modification.

[Spiicky]

Merci andry.aime, je suppose que cela permet d'avoir un code plus propre...

Et il n'y a pas de trace d'erreur ?

Non, je n'ai aucune StackTrace

En principe il doit y avoir une méthode pour charger le tableau et cette méthode est appelée à l'ouverture puis en cas de modification / ajout / suppression.
Peux-tu l'identifier ?

J'essaye de trouver au plus !

[Spiicky]

Bon voici ce que j'ai trouvé en arrivant :

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
/* Méthode permettant d'afficher les sociétés dans le tableau du panneau d'administration */
    public static void AffichageSociete(JTable table)
    {
        try
        {
            DefaultTableModel modele = (DefaultTableModel) table.getModel();
            Vector<Vector<String>> data = null;
            Connexion connexion = new Connexion();
            Connection conn = connexion.getConnexion();
 
            Statement stmt = conn.createStatement(ResultSet.TYPE_SCROLL_SENSITIVE, ResultSet.CONCUR_UPDATABLE);
 
            ResultSet resultats = stmt.executeQuery("SELECT * FROM Societe;");
 
            int i = 0;
            Utilitaire.centrerTable(table);
            while(resultats.next())
            {
                modele.addRow(data);
                table.setValueAt(resultats.getString(2),i,0);
                table.setValueAt(resultats.getString(3),i,1);
                table.setValueAt(resultats.getString(4),i,2);
                table.setValueAt(resultats.getString(5),i,3);
                table.setValueAt(resultats.getString(6),i,4);
                table.setValueAt(resultats.getString(7),i,5);
                table.setValueAt(resultats.getString(8),i,6);
                table.setValueAt(resultats.getString(9),i,7);
                table.setValueAt(resultats.getString(10),i,8);
                i++;
            }
            resultats.close();
            connexion.closeConn();
        }
        catch(Exception e)
        {
            e.printStackTrace();
        }
    }

J'ai biensure modifié pour avoir ceci :

Code :

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

1
2
3
 
Connexion connexion = new Connexion();
ResultSet resultats = connexion.getResultSet("SELECT * FROM Societe;");...

Mais ce n'est pas la cause du problème.
C'est bien la méthode qui nous intéresse ?

Et après vérification, c'est à l'ajout, la modification (ou pas) et à la suppression que le tableau ne présente plus les items !!!

[Népomucène]

Oui c'est cela.
Il me semble que si cette méthode était déclenchée après chaque modification (ajout/suppr/modif) alors on devrait avoir un tableau à jour de ce qui se trouve réellement dans la base.

[Spiicky]

D'accord, pour exemple, j'ai été dans Fenetre.java et repéré jButtonSupprimerSocieteActionPerformed:

Code :

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

1
2
3
    private void jButtonSupprimerSocieteActionPerformed(java.awt.event.ActionEvent evt) {                                                        
        Societe.deleteSociete(jTableSociete);
}

J'y ai donc ajouté :

Code :

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

Societe.AffichageSociete(jTableSociete);

Mais cela n'a pas l'air d'opérer

[Népomucène]

Bon, ben il faut vérifier avec le mode débug si AffichageSociete est bien exécuté quand tu cliques sur le bouton supprimer

[Spiicky]

Rolala je sens que ca va être méga casse tête là...
En débug tout se passe bien, jusqu'a mon affichage puis après, ca part dans les méandres de NB le tout accompagné de mon appli en blackscreen...
Avant que tout revienne (mais vide), j'étais sur cette méthode de EventDispatchThread.java.... :

Code :

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

1
2
3
4
5
6
7
8
9
    void pumpEventsForFilter(int id, Conditional cond, EventFilter filter) {
        addEventFilter(filter);
        while (doDispatch && cond.evaluate()) {
            if (isInterrupted() || !pumpOneEventForFilters(id)) {
                doDispatch = false;
            }
        }
        removeEventFilter(filter);
    }

Faudrait pas plutot contourner le problème ? mais je n'ai aucune idée du comment