Discussion :

[®om]

Salut,

Pour faire mes listeners, je fais toujours comme ça:

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
class MaClass {
    private Collection<XListener> listeners = new ArrayList<XListener>();
    ...
 
    public void addXListener(XListener listener) {
        listeners.add(listener);
    }
 
    public void removeXListener(Listener listener) {
        listeners.remove(listener);
    }
 
    protected void fireXXX(...) {
        for(XListener listener : listeners)
            listener.XXX(...);
    }
 
}

Mais je viens de découvrir que dans l'API (et c'est ce qui est utilisé par les listeners de l'API standard), il y a une classe EventListenerList

ce qui donnerait:

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
class MaClass {
    private EventListenerList listenerList = new EventListenerList();
    ...
    public void addXListener(XListener listener) {
        listenerList.add(XListener.class, listener);
    }
 
    public void removeXListener(XListener listener) {
        listenerList.remove(XListener.class, listener);
    }
 
    protected void fireXXX(...) {
        Object[] listeners = listenerList.getListenerList();
        /* elle est bizarre cette boucle, mais bon c'est dans la javadoc */
        for (int i = listeners.length-2; i>=0; i-=2) {
            if(listeners[i]==XListener.class) {
                ((FooListener)listeners[i+1]).fooXXX(...);
        }
    }
}

Laquelle est la "meilleure"? (et pourquoi?)

[adiGuba]

Salut,


L'EventListenerList te permet de n'utiliser qu'un seul conteneur pour tous tes Listener, a condition qu'il hérite bien de EventListener.

La boucle est un peu "bizarre" car elle saute un élément sur deux, parceque le tableau contient l'objet Class suivi de son implémentation...


Perso j'utilise de préférence EventListenerList à une simple ArrayList (après tout c'est fait pour cela), mais j'utilise la méthode getListeners(Class) pour récupérer les Listeners. Avec Java 5.0 c'est beaucoup plus simple :

Code :

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

1
2
3
4
5
    protected void fireXXX(...) {
        for ( XListener listener : listenerList.getListeners(XListener.class) ) {
            listener.fooXXX(...);
        }
    }

a++

[®om]

Mais, est-ce meilleur en terme de mémoire / rapidité?

[adiGuba]

Mais, est-ce meilleure en terme de mémoire / rapidité?

Heu... aucune idée... Mais je ne pense pas que la différence soit significative...

a++

[Gfx]

Aucune difference notable en effet.

[®om]

OK merci de vos réponses

[herve91]

/* elle est bizarre cette boucle, mais bon c'est dans la javadoc */

Justement, le petit détail qui a son importance, c'est que la liste est parcourue en sens inverse... Cela permet de déclencher en premier lieu les derniers listeners ajoutés : ainsi les listeners ajoutés par le programme "client" sont appelés avant les listeners "cablés" dans l'API. Cela peut être utile par exemple pour le KeyListener, ça permet de consommer l'événement si besoin avant que celui-ci ne soit traitée par l'API.

[®om]

Justement, le petit détail qui a son importance, c'est que la liste est parcourue en sens inverse... Cela permet de déclencher en premier lieu les derniers listeners ajoutés : ainsi les listeners ajoutés par le programme "client" sont appelés avant les listeners "cablés" dans l'API. Cela peut être utile par exemple pour le KeyListener, ça permet de consommer l'événement si besoin avant que celui-ci ne soit traitée par l'API.

Comment ça, le consommer?
De toute façon tous les listeners seront avertis, non? On peut "arrêter" le signal pour les listeners non encore avertis?

[herve91]

La classe AWTEvent dispose des méthodes consume() et isConsumed().
Oui tous les listeners sont avertis mais ils peuvent décider de tester l'état de l'événement avant de réaliser leur traitement.