Discussion :

[patnership]

Bonjour j'ai écrit ce code pour vérifier l'existence d'un enregistrement avant l'insertion mais il ne fonctionne que dans la 1ere partie du if (en gras) le else ne marche pas
Je ne sais pas où se situe le problème . please help me !

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 DefaultTableModel dt;
try{
                     
                    if (cbvn.isSelected()){  
                    pst=con.prepareStatement(sqlattvn);
                                         
                        boolean b=false;
                        for(int j=0; j<dt.getRowCount() ;j++){
                            if ((txtnumattestvn.getText())==dt.getValueAt(j,0)){
                                b=true;
                                break;
                            }
                        }

                        if (b==false){
                             
                            
                            pst.setString(1,txtnumattestvn.getText());
                            pst.setString(2,date.toString());                            
                            pst.setString(3,datef.toString());
                            pst.setString(4,txtpolicevn.getText());
                            
                            pst.execute();                             
                            affichermodelvn();
                            
                            JOptionPane.showMessageDialog(null, "Enregistrement effectué avec succès !");
                        }
                        else{
                            JOptionPane.showMessageDialog(null, "Ce numero d'attestation existe déja");
                            
                                txtnumattestvn.setText("");
                                txtdateffet.setText("");
                                txtdatefin.setText("");
                                txtpolicevn.setText("");
                                  
                            
                        }
                    }

[thelvin]

Hello,

cette ligne ne convient pas :

Code :

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

if ((txtnumattestvn.getText())==dt.getValueAt(j,0)){

En Java les objets, dont notamment les String, doivent être comparés avec equals(), pas avec ==

[patnership]

Merci pour votre réponse rapide
J'ai modifié le code comme ceci

Code :

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

if ((txtnumattestvn.getText()).equals(dt.getValueAt(j,0))){

Mais le résultat est le même
Merci de votre aide

[MasterMbg]

Merci pour votre réponse rapide
J'ai modifié le code comme ceci

Code :

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

if ((txtnumattestvn.getText()).equals(dt.getValueAt(j,0))){

Mais le résultat est le même
Merci de votre aide

Bonjour,
Essaie de faire ceci :

Code :

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

if ((txtnumattestvn.getText().trim()).equals(dt.getValueAt(j,0).toString().trim())){

[hwoarang]

Tout d'abord, ton code verifie que la donnée dans le textbox est présent dans le DefaultTableModel et pas dans la BDD. Ca veut dire que si tu lances 2 fois ton executable et que tu inseres 2 fois le meme enregistrement, ca va pas marcher.
Le mieux, c'est de verifier l'existence de l'entrée dans la requete. Un truc du genre "INSERT INTO MATABLE(MAVALEUR) (SELECT 'coco' WHERE NOT EXISTS SELECT MAVALEUR FROM MATABLE WHERE MAVALEUR = 'coco')"

Comme ca, si la valeur a été insérée, tout va bien. Et si elle n'est pas insérée, message d'erreur. Tu profites des mécanismes de la BDD qui evitent les modifications simultanées.

Pour revenir à ton probleme, le mieux, c'est de faire un System.out.println() du text de la textbox ainsi que des données récupérées de ton model. Tu as probablement, comme ca a été sous entendu dans les messages précédents, un probleme d'espaces qui trainent ou quelque chose comme ca.

[patnership]

Merci ça marche avec cette modification

Code :

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

if ((txtnumattestvn.getText().trim()).equals(dt.getValueAt(j,0).toString().trim())){