Créer ses propres listeners

Bonjour

J'ai lu l'excellent tuto sur Developpez pour créer ses propres listeners. J'ai réussi à faire marcher le programme mais j'ai dû utiliser une "astuce" qui casse tout le design pattern MVC...

En fait, mon interface graphique lance un thread.

Code:

---

1 2 3 4

public void run() { Partie p = new Partie(1,4); }

---

Le problème, c'est que 'p' lance une boucle infinie (c'est voulu) et donc si je fais :

Code:

---

1 2 3 4 5

public void run() { Partie p = new Partie(1,4); p.addMyListener(this); }

---

évidemment, le listener n'est jamais crée... Donc je ne vois pas comment faire... Ce que j'ai réussi à faire pour au moins tester que tout marche au niveau du code, c'est faire addMyListener dans le constructeur de Partie et de passer au constructeur de Partie une référence sur l'interface graphique et donc j'ai :

Code:

---

1 2 3 4 5

public Partie(int a, int b, Gui ig) { // du code... // avant la boucle infinie addMyListener(ig);

---

Et là, ça marche ... Seulement je perds l'intérêt du listener puisque mon modèle dépend de l'interface graphique...

La solution est sûrement évidente, c'est peut-être l'heure tardive qui me rend c*n :aie:

Nas'

Il faut que tu lances la boucle infinie dans un autre thread ; ainsi, le thread de base continuera à tourner, et donc aura la possibilité d'inscrire tous les listeners qu'il voudra.

Avoir une boucle infinie n'est pas en soi un problème (c'est peu ou prou le choix de swing, et de tous les systèmes client / serveur). Ici il faut que tu voyes si c'est un simple choix de mise en oeuvre, une facilité technique que tu as trouvée, auquel cas il est préférable de masquer au maximum ce point pour l'extérieur, ou bien si c'est justifié par ton domaine d'application, auquel cas tu auras probablement avantage à le rendre plus explicite, et à faire que ton Runnable soit public (sur le modèle du thread swing, par exemple, que chacun peut manipuler).

Ce que tu peux faire, c'est utiliser le polymorphisme

Code:

---

1 2 3 4 5

public Partie(int a, int b, Gui ig) { // du code... // avant la boucle infinie addMyListener(ig);

---

Devient

Code:

---

1 2 3 4 5

public Partie(int a, int b,MyListenerInterface ig) { // du code... // avant la boucle infinie addMyListener(ig);

---

Où MyListenerInterface est ton interface d'écoute

Ta boucle infinie pourrait être dans une méthode "start()" (ou autre, mais une méthode) plutôt que dans le constructeur directement.
Ainsi, tu pourrais faire:

Code:

---

1 2 3

Partie p = new Partie(...); p.addXListener(...); p.start();

---

Sinon, si tu veux absolument garder ta partie telle quelle, tu peux utiliser une (horrible) astuce:

Code:

---

Partie p = new Partie(...) {{addXListener(...);}};

---

Attention cependant, le addXListener a lieu AVANT le constructeur, donc il faut bien initialiser EventListenerList avant le constructeur (comme dans le tuto).

Donc la première solution est meilleure je pense...

Merci pour vos réponses.

En fait, juste après avoir posté mon message hier, j'ai pensé à la première méthode de ®om qui me semble la plus simple à mettre en oeuvre.

Belegkarnil, je ne savais pas qu'on pouvait faire ça, évidemment si j'avais su ... :mrgreen:

gifffftane, la boucle infinie c'est pour l'écoute/client serveur. Et je préfère éviter d'avoir encore un thread, j'en ai tellement pour l'instant :aie:

Merci à vous.

Nas'

D'ailleurs, pour ajouter tous tes listeners au constructeur, tu peux faire:

Code:

---

public Partie(int a, int b,MyListenerInterface... ig)

---

Comme ça t'as le droit de faire:

Code:

---

1 2 3

new Partie(1,2); // pas de listener new Partie(1,2,ig1,ig2,ig3); // 3 listeners new Partie(1,2,ig); // 1 listener

---

Je me rends compte que je n'ai pas très compris en fait.
Dans la classe Gui qui ressemble à ça :

Code:

---

1 2 3 4

public class Gui extends JFrame implements MyListener { // du code ... new Partie(1,4,this); }

---

ce n'est pas "this" que je suis supposé passer, si ? Sachant que :

Code:

---

public Partie(int a, int b, MyListener listen){

---

:?

Nas'

Si, mais quel est le type de la variable surlaquel tu affectes listen? Son type doit être MyListener pas GUI

Code:

---

1 2	public Partie(int a, int b, MyListener listen){

---

Dans ton cas, si, mais il est plutôt conseiller de faire un listener autre que ta classe graphique...

Par exemple

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public class Gui extends JFrame { class Controller implements MyListener { ... }   private final Controller controller = new Controller();   public void uneMethode() { new Partie(1,2,controller); } }

---

D'accord, merci pour les précisions ;)

Nas'