Problème jComboBox.removeAllItem() avec actionListener() génère erreur de type Null pointeurException

**j-narimane** · 13/01/2020, 15h39

Nom : liste2.png
Affichages : 124
Taille : 167,6 Ko

Nom : liste3.png
Affichages : 139
Taille : 166,9 Ko

**joel.drigo** · 13/01/2020, 15h58

action1 c'est la réaction à la sélection dans combo1
action2 c'est la réaction à la sélection dans combo2

on le voit à ce code :

Code :

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

1
2
3
comboBox1.addActionListener(new action1()); 
comboBox2.addActionListener(new action2());
comboBox1.setSelectedIndex(-1); // ce code annule la sélection automatique dans combo1 du premier choix due au remplissage de combo1

Voici le code des action1 et action2 que tu as écrit :

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
class action1 implements ActionListener{
@Override
public void actionPerformed(ActionEvent ae){
liste = getSelection( comboBox1 ) ;
   remplirCombo2(comboBox2, liste);
    //remplirCombo2(comboBox2, comboBox1.getSelectedItem().toString() );
 
}
}
class action2 implements ActionListener{
@Override
public void actionPerformed(ActionEvent ae){
          client = getSelection( comboBox2 );
 
           if ( client!=null && liste!=null ) {
         // les deux choix sont non "null"
               comboBox2.setEnabled(true);
               comboBox2.setSelectedIndex(-1);
         System.out.println( liste+ " - " + client);
 
 
     }

C'est dans action1, à la sélection d'un choix dans combo1 et après qu'on ait remplit combo2 qu'il faut annuler la sélection de combo2 (qui a été faite automatiquement lorsqu'on l'a remplie).

Donc 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
class action1 implements ActionListener{
    @Override
    public void actionPerformed(ActionEvent ae){
        liste = getSelection( comboBox1 ) ;
        remplirCombo2(comboBox2, liste);
        comboBox2.setSelectedIndex(-1); // on annule la sélection automatique du premier élément de combo2, après avoir rempli la combo 2
    }
}
class action2 implements ActionListener{
    @Override
    public void actionPerformed(ActionEvent ae){
          client = getSelection( comboBox2 );
 
           if ( client!=null && liste!=null ) {
               // les deux choix sont non "null"
               System.out.println( liste+ " - " + client);
           }
    }
}

dans remplir combo1 il est inutile de rappeler liste=getSelection(comboBox1); : value c'est déjà cette valeur

Code :

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

1
2
3
4
5
private void remplirCombo2(JComboBox comboBox, String value) {
 
     // liste=getSelection(comboBox1); // INUTILE
 
     /* etc */

En revanche, à la construction, on fait comboBox2.setEnabled(false);, ce qui fait qu'on ne peut pas faire de choix dans cette combo. C'est juste pour empêcher de faire un choix dedans quand il n'y a aucun choix de fait dans combo1. Tu peux supprimer cette ligne si ça ne te dérange pas qu'on puisse le faire. Sinon, si tu le supprimes aps, il faut activer comboBox2, lorsqu'on la remplit sinon on ne pourra pas faire de choix, donc ajouter "comboBox.setEnabled(true)" :

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
private void RemplirComboBox1(JComboBox comboBox) throws SQLException{
 
    Statement stm=con.createStatement();
        String sql1 ="select * from liste ";
 
         rs=stm.executeQuery(sql1);
 
 
        try {
 
            while(rs.next()){
 
                 String link=rs.getString(1);
 
                comboBox1.addItem(link); 
 
                }
 
 
 
 
        } catch (SQLException ex) {
            System.out.println("impossible de charger les items de BDD "+ex);
    } 
 
      }
 
 
 
 
 
private void remplirCombo2(JComboBox comboBox, String value) {
 
     // liste=getSelection(comboBox1); // INUTILE
     if ( value!=null ) {
        switch (liste) {
        case "choix 1":
            comboBox2.removeAllItems();
            try {
                Statement stm=con.createStatement();
                String sql2="select client from client where choix=1";
 
                rs1=stm.executeQuery(sql2);
 
                while(rs1.next()){
 
 
                    comboBox2.addItem( rs1.getString(1));
 
 
                }
 
            }
            catch (SQLException ex) {
                Logger.getLogger(admin.class.getName()).log(Level.SEVERE, null, ex);
            }
 
            break;
        case "choix 2":
            comboBox2.removeAllItems();
 
            try {
                Statement stm=con.createStatement();
                String sql2="select client from client where choix=2";
 
                rs2=stm.executeQuery(sql2);
 
                while(rs2.next()){
 
 
                    comboBox2.addItem( rs2.getString(1));
 
                }
 
 
            } catch (SQLException ex) {
                Logger.getLogger(admin.class.getName()).log(Level.SEVERE, null, ex);
            }
 
            break;
        case "choix 3":
            comboBox2.removeAllItems();
            try {
                        Statement stm=con.createStatement();
                        String sql2="select client from client where choix=3";
 
                        rs3=stm.executeQuery(sql2);
 
                        while(rs3.next()){
 
                            comboBox2.addItem( rs3.getString(1)); }
 
            } catch (SQLException ex) {
                Logger.getLogger(admin.class.getName()).log(Level.SEVERE, null, ex);
            }
 
            break;
        }
 
 
        comboBox.setEnabled(true); // on permet la sélection dans combo2
 
    }
    else {
        comboBox.setEnabled(false); // on ne permet pas la sélection dans combo2 (ça ne sert à rien, y'a rien de sélectionné dans combo1)
    }
 
}

**joel.drigo** · 13/01/2020, 16h11

Nom : Capture.JPG
Affichages : 88
Taille : 10,6 Ko

ça c'est l'affichage en console dû à System.out.println( liste+ " - " + client);.

C'est ça qui te gêne ? Dans ce cas, c'est un plus compliqué.

Il faut stocker l'instance de action2 pour pouvoir la mettre et l'enlever au besoin.

Code :

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

1
2
3
4
5
comboBox1.addActionListener(new action1()); 
actionComboBox2 = new action2();
//on ne fait plus ça à la construction // comboBox2.addActionListener(new action2());
comboBox1.setSelectedIndex(-1); // ce code annule la sélection automatique dans combo1 du premier choix due au remplissage de combo1
comboBox2.setEnabled( false );

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
class action1 implements ActionListener{
    @Override
    public void actionPerformed(ActionEvent ae){
        comboBox2.removeActionListener( actionComboBox2 ); // on ne doit plus changer les événements de sélection dans combo2 pendant qu'on modifie ses choix, on enlève donc l'écouteur action2
        liste = getSelection( comboBox1 ) ;
        remplirCombo2(comboBox2, liste);
        comboBox2.setSelectedIndex(-1); // on annule la sélection automatique du premier élément de combo2, après avoir rempli la combo 2        
        comboBox2.addActionListene( actionComboBox2 ); // on écoute les événements de sélection dans combo2 seulement à partir de là
    }
}
class action2 implements ActionListener{
    @Override
    public void actionPerformed(ActionEvent ae){
          client = getSelection( comboBox2 );
 
           if ( client!=null && liste!=null ) {
               // les deux choix sont non "null"
               System.out.println( liste+ " - " + client);
           }
    }
}

**j-narimane** · 13/01/2020, 16h22

voici mon 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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
 
/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package admin;
 
 
import java.awt.Panel;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.logging.Level;
import java.util.logging.Logger;
 
import javax.swing.JComboBox;
 
 
/**
 *
 * @author Inspiron
 */
public class admin extends javax.swing.JFrame {
 
    /**
     * Creates new form admin
     */
    JComboBox<String> comboBox1;
    JComboBox<String> comboBox2;
    Panel p;
    Connection con =null;
    ResultSet rs,rs1,rs2,rs3,rs4;
    String liste , client;
 
    public admin() {
        initComponents();
 
 
 
 
 
 
 
 
        initComponents();
          comboBox1 = new JComboBox<>();
          comboBox2 = new JComboBox<>();
 
 
           p=new Panel();
           p.add(comboBox1);
           p.add(comboBox2);
 
           this.setVisible(true);
           this.setContentPane(p);
        try{
 
        Class.forName("org.sqlite.JDBC");
        con=DriverManager.getConnection("jdbc:sqlite:"+"C:\\work\\admin.db");
        System.out.println("bdd est connecter ");
 
        RemplirComboBox1(comboBox1);
 
         }
        catch(Exception ex){
 
        System.out.println(ex);
 
        }  
     comboBox1.addActionListener(new action1()); 
     comboBox2.addActionListener(new action2());
     comboBox1.setSelectedIndex(-1);
 
  }
//les fonctions getselected*******************************************************
private static String getSelection(JComboBox comboBox) {
     Object selected = comboBox.getSelectedItem();
     if ( selected != null ) {
           return selected.toString();
     }
     else {
           return null;
     }
}
private void RemplirComboBox1(JComboBox comboBox) throws SQLException{
 
    Statement stm=con.createStatement();
        String sql1 ="select * from liste ";
 
         rs=stm.executeQuery(sql1);
 
 
        try {
 
            while(rs.next()){
 
                 String link=rs.getString(1);
 
                comboBox1.addItem(link); 
 
                }
 
 
 
 
        } catch (SQLException ex) {
            System.out.println("impossible de charger les items de BDD "+ex);
    } 
 
      }
 
 
 
 
 
private void remplirCombo2(JComboBox comboBox, String value) {
 
 
     if ( value!=null ) {
        switch (liste) {
        case "choix 1":
            comboBox2.removeAllItems();
            try {
                Statement stm=con.createStatement();
                String sql2="select client from client where choix=1";
 
                rs1=stm.executeQuery(sql2);
 
                while(rs1.next()){
 
 
                    comboBox2.addItem( rs1.getString(1));
 
 
                }
 
            }
            catch (SQLException ex) {
                Logger.getLogger(admin.class.getName()).log(Level.SEVERE, null, ex);
            }
 
            break;
        case "choix 2":
            comboBox2.removeAllItems();
 
            try {
                Statement stm=con.createStatement();
                String sql2="select client from client where choix=2";
 
                rs2=stm.executeQuery(sql2);
 
                while(rs2.next()){
 
 
                    comboBox2.addItem( rs2.getString(1));
 
                }
 
 
            } catch (SQLException ex) {
                Logger.getLogger(admin.class.getName()).log(Level.SEVERE, null, ex);
            }
 
            break;
        case "choix 3":
            comboBox2.removeAllItems();
            try {
                        Statement stm=con.createStatement();
                        String sql2="select client from client where choix=3";
 
                        rs3=stm.executeQuery(sql2);
 
                        while(rs3.next()){
 
                            comboBox2.addItem( rs3.getString(1)); }
 
            } catch (SQLException ex) {
                Logger.getLogger(admin.class.getName()).log(Level.SEVERE, null, ex);
            }
 
            break;
        }comboBox.setEnabled(true);
     } else {
 
     comboBox.setEnabled(false);
 
     } 
 
 
 
}
 
// les actions de listeners *********************************************
 
class action1 implements ActionListener{
@Override
public void actionPerformed(ActionEvent ae){
liste = getSelection( comboBox1 ) ;
 comboBox2.setSelectedIndex(-1);
   remplirCombo2(comboBox2, liste);
    //remplirCombo2(comboBox2, comboBox1.getSelectedItem().toString() );
 
}
}
class action2 implements ActionListener{
@Override
public void actionPerformed(ActionEvent ae){
          client = getSelection( comboBox2 );
 
           if ( client!=null && liste!=null ) {
         // les deux choix sont non "null"
 
 
         System.out.println( liste+ " - " + client);
 
 
     }
         }}
class buttonAction implements ActionListener {
 
 
 
    @Override
    public void actionPerformed(ActionEvent ae) {
        //throw new UnsupportedOperationException("Not supported yet."); //To change body of generated methods, choose Tools | Templates.
 
    }
 
}
 
 
 
    /**
     * This method is called from within the constructor to initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is always
     * regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {
 
        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
 
        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 400, Short.MAX_VALUE)
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 300, Short.MAX_VALUE)
        );
 
        pack();
    }// </editor-fold>                        
 
    /**
     * @param args the command line arguments
     */
    public static void main(String args[]) {
        /* Set the Nimbus look and feel */
        //<editor-fold defaultstate="collapsed" desc=" Look and feel setting code (optional) ">
        /* If Nimbus (introduced in Java SE 6) is not available, stay with the default look and feel.
         * For details see http://download.oracle.com/javase/tutorial/uiswing/lookandfeel/plaf.html 
         */
        try {
            for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    javax.swing.UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (ClassNotFoundException ex) {
            java.util.logging.Logger.getLogger(admin.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (InstantiationException ex) {
            java.util.logging.Logger.getLogger(admin.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (IllegalAccessException ex) {
            java.util.logging.Logger.getLogger(admin.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (javax.swing.UnsupportedLookAndFeelException ex) {
            java.util.logging.Logger.getLogger(admin.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        }
        //</editor-fold>
 
        /* Create and display the form */
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new admin().setVisible(true);
            }
        });
    }
 
    // Variables declaration - do not modify                     
    // End of variables declaration                   
}

**joel.drigo** · 13/01/2020, 16h52

Ton nouveau code ne correspond pas au code que je t'ai donné.

**j-narimane** · 14/01/2020, 14h07

Merci pour votre réponse le problème est réglé

Problème jComboBox.removeAllItem() avec actionListener() génère erreur de type Null pointeurException

AWT/Swing Java

Discussions similaires

Partager

Partager