JCombobox + ItemListener

Version imprimable

Bonjour,

j'ai une combobox et des actions qui s'effectuent lorsque l'on sélectionne un "item".

Or quand je remplis ma combobox, itemStateChanged est appelée à chaque élément ajouté et ça plante l'affichage !

Donc je charge ma combobox par un thread (car mes valeurs sont dans une base de données) :

Code:

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
 
    /**
     * Classe Thread
     * Chargement des paramétrages Saisons
     */
    private class LoadSaisonsThread extends Thread {
 
        @Override
        public void run() {
 
            final SaisonsListe<Saison> saisons = new SaisonsListe<Saison>(fiche.getAdresse());
            saisons.load();
 
        //  on renvoie le rafraichissement à l'EDT
            SwingUtilities.invokeLater(
                    new Runnable() {
 
                        @Override
                        public void run() {
 
                            for (Saison saison : saisons)
                                cboSaisons.addItem(saison);
 
                            cboSaisons.setSelectedItem(null);
 
                            int count = 0;
                            if ( (count=cboSaisons.getItemCount())>0 )
                                cboSaisons.setSelectedIndex(count-1);
 
                        }
 
                });
        }
 
    }

Et quand on choisit manuellement la saison, ça exécute un chargement spécifique :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
    @Override
    public void itemStateChanged(ItemEvent e) {
 
        if ( e.getStateChange()==ItemEvent.SELECTED  ) {
 
        //  chargement d'une saison
            if ( cboSaisons.getSelectedIndex()>=0 )
                grille.load((Saison) cboSaisons.getSelectedItem());
 
        }
    }

Donc chaque fois que j'ajoute un élément

Code:

cboSaisons.addItem(saison);

ma fonction itemStateChanged est appelée !

Comment y remédier proprement ?
:(

bourrin mais pas très "beau"...

Pour empêcher le chargement, j'ai alors utilisé une variable de type boolean :

Code:

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
    /**
     * Classe Thread
     * Chargement des paramétrages Saisons
     */
    private class LoadSaisonsThread extends Thread {

        @Override
        public void run() {

            final SaisonsListe<Saison> saisons = new SaisonsListe<Saison>(fiche.getAdresse());
            saisons.load();

        //  on renvoie le rafraichissement à l'EDT
            SwingUtilities.invokeLater(
                    new Runnable() {

                        @Override
                        public void run() {

                            enChargement = true;
                            for (Saison saison : saisons)
                                cboSaisons.addItem(saison);

                            cboSaisons.setSelectedItem(null);
                            
                            enChargement = false;
                            int count = 0;
                            if ( (count=cboSaisons.getItemCount())>0 )
                                cboSaisons.setSelectedIndex(count-1);
                            
                        }

                });
        }

    }

Donc pour charger ma combobox, j'initialise ma variable à true, et quand j'ai terminé l'ajout de mes éléments, je passe à false.

Ainsi, dans ma gestion de l'evènement, si en cours de chargement de la combo, je ne fais pas mon traitement :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
    @Override
    public void itemStateChanged(ItemEvent e) {

        if ( e.getStateChange()==ItemEvent.SELECTED  ) {

        //  chargement d'une saison
            if ( !enChargement && cboSaisons.getSelectedIndex()>=0 )
                grille.load((Saison) cboSaisons.getSelectedItem());
            
        }
    }

Ca marche très bien, mais peut-on faire quelque chose de plus "propre" ?

:roll:

Merci pour vos suggestions

Ce qui m'étonne, c'est que ce bout de code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
        final JComboBox mCombo = new JComboBox();
        mCombo.addItem("Toto");
        mCombo.addItemListener(new ItemListener()
        {
            public void itemStateChanged(ItemEvent e)
            {
                System.out.print(e.getItem());
                if(e.getStateChange() == ItemEvent.SELECTED) {
                    System.out.println(" Selected");
                } else {
                    System.out.println(" Deselected");
                }
            }
        });
 
        mCombo.addItem("Toto2"); // Aucun évènement
        mCombo.addItem("Toto3"); // Aucun évènement
        mCombo.setSelectedIndex(2); // Deselect puis Select
        mCombo.addItem("Toto4"); // Aucun évènement

Montre que l'ajout d'élément ne provoque pas d'évènement.
Essaie de le comparer à ton code pour voir la différence.

Tout d'abord... merci pour ta réponse ! 8-)

Ensuite, en regardant ton code de plus près, je m'aperçois que tu ajoutes ton listener une fois après avoir ajouté un élt dans ta combo :

Code:

1
2
3
 
mCombo.addItem("Toto");
mCombo.addItemListener(

Donc si je reprends ton exemple tel que :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
        final JComboBox mCombo = new JComboBox();
 
        mCombo.addItemListener(new ItemListener()
        {
            public void itemStateChanged(ItemEvent e)
            {
                System.out.print(e.getItem());
                if(e.getStateChange() == ItemEvent.SELECTED) {
                    System.out.println(" Selected");
                } else {
                    System.out.println(" Deselected");
                }
            }
        });
 
        mCombo.addItem("Toto");   //Select !
        mCombo.addItem("Toto2"); // Aucun évènement
        mCombo.addItem("Toto3"); // Aucun évènement
        mCombo.setSelectedIndex(2); // Deselect puis Select
        mCombo.addItem("Toto4"); // Aucun évènement

Ainsi j'ai pu modifier mon code en initialisant ma combo :

Code:

1
2
3
4
 
        cboSaisons = new JComboBox();
        cboSaisons.addItem("");                  //un élt vide (qui sert à rien)
        cboSaisons.addItemListener(this);    //je rajoute après mon listener

Puis dans mon thread :

Code:

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
 
    /**
     * Classe Thread
     * Chargement des paramétrages Saisons
     */
    private class LoadSaisonsThread extends Thread {
 
        @Override
        public void run() {
 
            final SaisonsListe<Saison> saisons = new SaisonsListe<Saison>(fiche.getAdresse());
            saisons.load();
 
        //  on renvoie le rafraichissement à l'EDT
            SwingUtilities.invokeLater(
                    new Runnable() {
 
                        @Override
                        public void run() {
 
                            for (Saison saison : saisons)
                                cboSaisons.addItem(saison);     //plus d'evt à l'ajout !
 
                            int count = 0;
                            if ( (count=cboSaisons.getItemCount())>0 )
                                cboSaisons.setSelectedIndex(count-1);     //je sélectionne enfin celui que je veux afficher qui génère un evt
 
                            cboSaisons.removeItemAt(0);      //je supprime l'elt vide préalablement ajouté à l'initialisation
 
                        }
 
                });
        }
 
    }

Et l'evt normal, déclenché comme suit :

Code:

1
2
3
4
5
6
7
8
 
    @Override
    public void itemStateChanged(ItemEvent e) {
 
        if ( e.getStateChange()==ItemEvent.SELECTED  )
                grille.load((Saison) e.getItem());
 
    }

En résumé :

j'ajoute un elt "vide" que je supprimerai plus tard
j'ajoute le listener
je charge ma combo comme avant, et comme il y a déjà un élt, il ne sélectionne pas et ne déclenche pas d'evt
je sélectionne l'élt que je veux bien afficher qui déclencle l'evt associé (normal !)
et je supprime le 1er élt vide

:ccool: