Discussion :

[gerard kester]

Bonjour

Je mets cette discussion ici mais je ne suis pas sûr d'être au bon endroit.

Je suis en train de développer un gros programme avec beaucoup de fenêtres.
Je me pose une questions sur la façon d'écouter les évènements.
il y a deux façons de faire :
- soit chaque composant utilise la fenêtre comme listener;
- soit chaque composant décrit son propre listener (voir code).
Code :

Code :

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

1
2
3
4
5
6
7
8
 
  btNouveau.addActionListener(new ActionListener()
  {
    public void actionPerformed(ActionEvent evenement)
    {
    ...
    }
});

La première façon a l'avantage de traiter les évènements (un seul composant écoute les évènements, la fenêtre) à un seul endroit mais impose des tests pour savoir quel est le composant qui a reçu cet évènement.
La deuxième façon est plus simple à écrire, le traitement d'un évènement est décrit avec la déclaration du composant mais génère un listener par composant.
Pour cette deuxième syntaxe, on peut aussi créer une classe qui gère les évènements mais je trouve que cela complique le code, il faut que cette classe dialogue avec toutes les autres classes, je ne suis pas fan de cette écriture. D'autant plus qu'elle fait écrire encore plus de classes. je ne veux pas tomber dans une profusion de classes (j'ai déjà 160 classes).

J'hésite entre les deux façons de faire.

Quelle est la meilleure façon de procéder, en terme de performance principalement ?

Merci pour vos commentaires.

[tchize_]

si tes traitement n'ont rien à voir entre eux, il es plus lisible de faire des actionlistener personnalisés (deuxième méthode) que d'avoir un seul gros actionlistener bourré de if/else if/ ....
De plus la deuxième méthode a l'avantage d'associer visuellement la logique de traitement au composant, ce qui est moins évident quand tout est déporté dans une classe plus loin

[gerard kester]

C'est ce que je pense. Mais je voudrais savoir si le fait de déclarer autant de listener que d'objet n'est pénalisant en terme de temps de traitement, charge de la machine virtuelle, ...
Je suis un vieil informaticien (j'ai appris ce métier en 1982) et mon soucis est l'optimisation du programme. Je cherche à obtenir un code rapide et propre sans tomber dans l'obsession.

[tchize_]

au démarrage d'un JVM t'as environ 10.000 instances présentes en mémoire. A mois d'avoir 10.000 listener, ça ne devrais pas pénaliser outre mesure

[sinok]

Et de toute façon les listeners présent en mémoire ne sont que ceux qui correspondent aux éléments graphiques instanciés, le garbage collector ayant par définition une certaine tendance à vouloir faire le ménage des objets déréférencés....

De plus ces listeners à durée de vie courte sont moins risqués au niveau fuites mémoire.


Puis bon en général, mieux vaut passer par des AbstractAction que des ActionListeners en général, plus génériques, réutilisables, s'intégrant avec les mécanismes d'ActionMap/InputMap...

Et il faut bien se dire que de toute façon, on créera moins de listeners que ne peut le faire les implémentations du Composant dans tel et tel Look&Feel...

[gerard kester]

OK, merci. J'ai bien compris la situation.