Discussion :

[Fahelis]

Bonjour,
Mon problème est le suivant.
Je possède une classe Personnage et plusieurs classes dérivés de celle-ci (Guerrier, Archer, Mage).

Dans mon programme j'instancie l'une des trois classes fille (en fonction du choix du joueur) et je stocke l'instance dans un objet de type Personnage (puisque je ne sais pas à l'avance lequel sera utilisé.)
Mon code se résume en gros à ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
Personnage perso;
if (event.getSource() == boutonGuerrier){
      perso = new Guerrier();
}
else if (event.getSource() == boutonArcher){
      perso = new Archer();
}
else if (event.getSource() == boutonMage){
      perso = new Mage();
}

Tout ceci fonctionne très bien pour tout ce que j'ai à faire, mais un seul petit truc me manque.
J'ai dans Archer une variable de plus (une arme à distance). Comme ni le guerrier ni le mage ne peuvent en avoir, je l'avais déclaré dans Archer.
Mais du coup je ne peux pas y accéder depuis "perso".

Existe t-il un moyen de contourner ce problème ou dois-je déclarer mon arme à distance dans personnage, même si seul mon archer l'utilisera ?

[Lucio_]

Cela dépend un peu des méthodes et des besoins des classes en question et surtout de l'interface, mais généralement il vaut mieux trouver un moyen générique depuis l’interface.
Pourquoi ne pas créer une interface arme, ainsi que plusieurs armes ayant chacune leurs propres caractéristiques? Chaque constructeurs de personnages sera en charge de créer sa propre instance d'arme.

[thelvin]

On peut pas connaître la réponse parfaite à ce problème, je saurais quoi faire si c'était moi qui étais en train de faire ce programme, mais ce n'est pas moi, c'est toi.

La question que tu dois te poser, est "pourquoi et quand je veux accéder à l'arme à distance."

- La solution où tout le monde a la variable, n'est pas très Orientée Objet au sens Java. Mais elle est simple et pragmatique. Et si tu fais ça tes classes ne servent pas à grand-chose, il faudrait juste un enum ClasseDePersonnage qui a les valeurs GUERRIER, ARCHER, MAGE et dans ta classe Personnage tu mets juste une variable de type ClasseDePersonnage. Normalement c'est plutôt comme ça qu'on fait quand on débute... Et souvent, même quand on débute pas.

- La solution Orientée Objet, seul l'archer a une variable arme à distance, exploite le polymorphisme. Par exemple si tu as besoin d'afficher les armes dispos d'un personnage, tu fais dans ta classe Personnage :

Code :

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

1
2
3
protected void afficherLesArmes(InterfaceDuJeu idj) {
  idj.afficherUneArme(arme);
}

et dans ta classe Archer tu la redéfinis :

Code :

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

1
2
3
4
@Override
protected void afficherLesArmes(InterfaceDuJeu idj) {
  idj.afficherDeuxArmes(arme, armeADistance);
}

Et même raisonnement pour tous les cas où tu as besoin d'accéder à l'arme à distance s'il y en a une : faire une méthode dans Personnage et la redéfinir dans Archer, et appeler cette méthode pour qu'elle fasse ce qui doit être fait avec ou sans arme à distance.

[Fahelis]

Cela dépend un peu des méthodes et des besoins des classes en question et surtout de l'interface, mais généralement il vaut mieux trouver un moyen générique depuis l’interface.
Pourquoi ne pas créer une interface arme, ainsi que plusieurs armes ayant chacune leurs propres caractéristiques? Chaque constructeurs de personnages sera en charge de créer sa propre instance d'arme.

J'ai pris ici l'exemple de l'arme à distance, mais j'ai d'autres paramètres qui sont propres aux sous classes. Du coup il faudrait que je fasse une interface pour chaque ? (Je n'ai encore jamais utilisé d'interface jusque là, donc je suis pas trop au point sur le sujet).

On peut pas connaître la réponse parfaite à ce problème, je saurais quoi faire si c'était moi qui étais en train de faire ce programme, mais ce n'est pas moi, c'est toi.

La question que tu dois te poser, est "pourquoi et quand je veux accéder à l'arme à distance."

- La solution Orientée Objet, seul l'archer a une variable arme à distance, exploite le polymorphisme. Par exemple si tu as besoin d'afficher les armes dispos d'un personnage, tu fais dans ta classe Personnage :

Code :

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

1
2
3
protected void afficherLesArmes(InterfaceDuJeu idj) {
  idj.afficherUneArme(arme);
}

et dans ta classe Archer tu la redéfinis :

Code :

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

1
2
3
4
@Override
protected void afficherLesArmes(InterfaceDuJeu idj) {
  idj.afficherDeuxArmes(arme, armeADistance);
}

Et même raisonnement pour tous les cas où tu as besoin d'accéder à l'arme à distance s'il y en a une : faire une méthode dans Personnage et la redéfinir dans Archer, et appeler cette méthode pour qu'elle fasse ce qui doit être fait avec ou sans arme à distance.

Là ça fonctionne car la methode afficherLesArmes est utiles à tous. Mais qu'en est-il pour quelque chose de vraiment spécifique à une seule classe fille.
Exemple : J'ai aussi mes classes ennemis qui dérivent de Personnage (Gobelin, Squelette et Sorcier), le sorcier est le boss final et il possède une grosse attaque qui nécessite qu'il se concentre pendant un tour avant de la lancer.
J'ai donc tout naturellement créer une variable booléenne "concentre" dans ma classe Sorcier, qui doit passer à true quand il lance son sort de concentration.
Mais il est le seul concerné par cette variable. Du coup si je met ma méthode "concentration" (qui passe la variable à true) dans Personnage, c'est pas super logique car elle ne sert qu'à lui. Mais si je la met dans Sorcier (ce que j'avais fait au départ), ben du coup je ne peux pas y accéder depuis ma variable "perso".

[tchize_]

Forcément, toutes tes classes ont des comportement différents, ce qui est liée à l'usage objet et ce qui est une bonne chose. Là où te pêche, selon moi, c'est que tu essaie d'utiliser cette information à l'extérieur de la classe. Le moteur s'en fous que ton personnage se concentre, ce n'est pas à lui à gérer les spécificité de la concentration (oui ce n'est pas évident à faire en pratique comme gestion). Ce qui lui importe, c'est que le personnage a une action "concentre" et qu'elle prend un tour. Tout comme l'archer a surement une action "tir à l'arc" et qu'elle prend 0.5 tour, et donc il lui importe de savoir que l'utilisateur, peux choisir de préparer plusieurs actions (se concenter (1), lancer un sort (1) ou "tirer vaguement (0.3), tirer vaguement (0.3), tirer vaguement (0.3)" pour avoir trois tirs sur le même tour). Là t'as quelque chose de générique: getListeActionsPossible(), getListeActionsEnAttente(), setFutureActions()

[Fahelis]

Forcément, toutes tes classes ont des comportement différents, ce qui est liée à l'usage objet et ce qui est une bonne chose. Là où te pêche, selon moi, c'est que tu essaie d'utiliser cette information à l'extérieur de la classe. Le moteur s'en fous que ton personnage se concentre, ce n'est pas à lui à gérer les spécificité de la concentration (oui ce n'est pas évident à faire en pratique comme gestion). Ce qui lui importe, c'est que le personnage a une action "concentre" et qu'elle prend un tour. Tout comme l'archer a surement une action "tir à l'arc" et qu'elle prend 0.5 tour, et donc il lui importe de savoir que l'utilisateur, peux choisir de préparer plusieurs actions (se concenter (1), lancer un sort (1) ou "tirer vaguement (0.3), tirer vaguement (0.3), tirer vaguement (0.3)" pour avoir trois tirs sur le même tour). Là t'as quelque chose de générique: getListeActionsPossible(), getListeActionsEnAttente(), setFutureActions()

La pour le coup ce n'est pas la question du choix des actions (mais j'ai peut-être mal compris ce que tu essaye de me dire). C'est juste l'accès à la methode concentration.
Elle n'a pas lui d'exister dans "Personnage" puisque seul le Sorcier l'utilise. Elle est donc dans la classe "Sorcier".
Mais quand je suis dans ma classe "Combat" (qui me sert à gérer le combat, logique tu me diras), j'ai mon joueur "perso" de type "Personnage" (puisqu'il peut être soit Guerrier, soit Archer, soit Mage) et son ennemi "ennemi" de type "Personnage" aussi car il peut être soit Gobelin, Squelette ou Sorcier (il est différent selon la pièce dans laquelle on se trouve).
Du coup via ma variable "ennemi", je n'ai pas accès à la méthode "concentration". Je pourrais la mettre dans "Personnage", ça réglerait facilement mon problème, mais ça me parait un peu bourrin, parce que en faisant ça, me classe filles ne servent plus à grand chose (enfin si à définir les caractéristiques au moment de l'instanciation, mais c'est tout)