Discussion :

[tiamat]

Bonjour à tous,

voila je vous expose mon problème...
j'ai une méthode qui déclare un object command. j'ai deux types de command possible (adminCommand et OperativeCommand) et je ne connais pas à l'avance le type de commande en question.
Je parcours un arbre XML avec JDom et au fur et à mesure j'initialise mon objet.
Or j'ai un souci car je ne sais pas comment faire bénéficier à mon object nouvellement instancié (soit AdminCommand soit OperativeCommand) des méthodes soit de l'une soit de l'autre...

Je m'explique. J'ai un objet:

Code :

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

private Object command;

au fur et à mesure du parcours de l'arbo XML je découvre le type de commande:

Code :

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

1
2
3
4
5
6
7
8
 
		      if(courant.getName().equals("command_type")){
	    	    	  //On crée notre objet admin
		    	  if(courant.getAttributeValue("value").equals("ADMIN"))
		    		  command = new AdminCommand();
		    	  else
		    		  command = new OperativeCommand();
		      }

Jusque là tout va bien.
Par contre par la suite, j'ai d'autre attributs que je dois associer à une classe ou à l'autre selon son type:

Code :

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

1
2
3
4
5
6
 
		      else if(courant.getName().equals("command")){
		    	  if(command instanceof AdminCommand){
		    		command.setCmd(courant.getAttributeValue("value"));
		    	  }
		      }

Dans cet exemple la méthode setCmd appartient à la classe AdminCommand mais le compilo me dit qu'il ne la connait pas pour le type Object (normal).
J'ai tenté de caster l'appel à la méthode setCmd:

Code :

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

1
2
 
(AdminCommand) command.setCmd(courant.getAttributeValue("value"));

Mais ça ne plait pas non plus au compilo.
Quelqu'un aurait une idée ?

Merci,
Tiamat.

[tiamat]

Bon j'ai trouvé la solution, mais à priori il faut éviter ce type d'utilisation:

Code :

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

		    		 ((AdminCommand) command).setCmd(courant.getAttributeValue("value"));

Vous savez pourquoi ?

[elmor]

Tu es obligé de faire ((AdminCommand) command).setCmd(courant.getAttributeValue("value"));
car lorsque tu fais
(AdminCommand) command.setCmd ((courant.getAttributeValue("value"));
tu es en train d'essayer de caster le restour de command.setCmd en AdminCommand.

Sinon, si je peux me permettre, j'ai une petite remarque:
Le fait que tu sois obligé de caster ton object command montre un problème dans le design dans tes classes.

Pourquoi ne pas faire une classe "Commande" avec deux sous classes "AdminCommand" et "OperativeCommand" ainsi dans ton code, command serait une instance de Commande et non d'"Object".
La code deviendrait plus lisible et surtout tu enleverais ces horribles "cast"

[bulbo]

Il y a deux problèmes dans ton code et plusieurs solutions:

1 - Tu ne devrais pas avoir besoin du instanceof dans ton parsing, ton code devrait se rappeler ce qu'il est en train de parser

2 - Comme l'a dit elmo le troll rouge, si la commande que tu appelles a le même nom dans les deux classes l'utilisation d'une super classe ou d'une interface est la solution (et plus besoin de savoir ce que tu es en train de parser).

Maintenant si les noms des méthodes que tu vas appeler sont différents pour chaque classe le coup de la super classe ne marche plus des tonnes. Pour résoudre un problème similaire lors d'un parsing XML j'avais fait un truc inspiré du design pattern visitor. Comme c'est un peu long je ne te mets pas le code tout de suite, si la super classe va c'est tout de même mieux.

Bulbo

[tiamat]

Ok merci pour les commentaires qui sont plus que bienvenus !
je vais effectivement repenser le design de mon code pour n'avoir qu'une seule classe commande et deux autres qui en héritent.

Mais j'ai tout de même un problème car je ne peux pas savoir à l'avance le type d'objet que je reçois de mon document XML. Je sais que c'est une commande mais les attributs sont différents selon les commandes admins et opératives, donc forcément les méthodes d'init de chaque objet seront elles aussi différentes. Il y en a de communes, mais au fur et à mesure du parsing XML je vais devoir initialiser mon objet final.

Donc si j'ai une classe mère Command et deux commandes AdminCommand et OperativeCommand qui étendes la classe Command, il y aura bien un moment dans mon parsing ou je vais devoir caster vers AdminCommand ou OperativeCommand pour récupérer les spécificités de mes classes finales.

J'avoue je ne sais pas trop comment faire là...

[tiamat]

Bulbo, je suis preneur pour avoir une idée de ton implémentation inspirée du design pattern visitor pour voir si cela peut être adaptable à mon code..

Merci !