Discussion :

[Iori Yagami]

Bonsoir,

Je developpe un jeu de carte en ligne, où les actions sont un peu divers. Je suis donc dans l'obligation de leurs associer chacun un type d'évenement souris (drag&drop, double clique, clique droit)
Mon application est bien organisée, j'essai de respecter le module MVC le plus possible. Donc les différents événements je les gère dans une classe Control. (classe imposée par l'API que j'utilise).
Le problème est que la gestion des différents événements est trop compliquée. la méthode mouseReleased me prend environ 300 ligne de code.

Je voulais savoir s'il y a de bonnes pratiques permettant d'améliorer mon code, ou des design patterns particuliers à ce genre de problèmes.

Merci.

[tchize_]

pour les listeners, comme tu peux en mettre plusieurs par objet graphique, pourquoi ne pas séparer en plusieurs listener plutot que d'avoir un gros listener qui essaie de gérer toutes les conditions?

[Iori Yagami]

La partie graphique est un composant ou il y a plusieurs dedans c'est vrai, mais les événements surviennent sur la totalité du composant.
C'est à dire, j'ai un composant de type HandView par exemple (où il y a des carte de jeu), et un autre de type ThrownView, les deux font partie du composant majeur contrôlé par la classe Control.

Si je fait un double clique sur une des cartes, la carte devrai se retrouver dans le composant ThrownView, bien évidemment après avoir rempli quelques conditions nécessaires :s

Si je drag une carte, je peux changer l'ordre des cartes dans le composant HandView.

Si je fait un clique droit, c'est toute une autre opération qui a ses conditions et ses conséquences.

le problème est que toutes ses opérations doivent être faites dans la méthode mouseReleased. (la méthode actionPerformed ne marche qu'avec des boutons je crois et déjà elle quoi à faire).

J'ai essayé de rendre le code plus lisible en créant des méthodes privés qui chacune regroupe un ensemble d'instruction.
Mais je me sens encore dans le chaos, déjà j'ai un blocage (sans aucune exception générée), mais je peux pas en déterminer la cause. Parce que c'est compliqué à suivre.

[tchize_]

pourquoi ne pas créer une classe séparé par action possible?

[Iori Yagami]

Oui, j'aime bien cette idée, mais je vois pas comment?

Je veux dire, les paramètres de cette classe? ou encore les classes dont elle peut hériter..

[tchize_]

ben c'est un actionlistener, il a pas besoin d'hérite de quoi que ce soit, il implémente acitonlistener et c'est tout. Qaudn au paramètre, tu dois lui passer dans le constructeur ou autre, tout ce dont il aura besoin pour faire son travail.