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 !

[elmor]

Tout d'abord je tiens à préciser que je ne suis pas rouge mais vert

J'ai une petite propotion mais je ne sais pas si ca convient à ton problème.
Tu as une classe abstraite Command et deux sous classes AdminCommand et OperativeCommand.

Dans ta classe command tu as une methode abstraite init qui prend en parametre un Node.

Code :

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

public abstract void init (org.w3c.dom.Node Node);

Tu crées la bonne classe des que tu connais le type de la commande. Ensuite tu initializes la commandes en appalant la method init

Un moyen pas mal serait aussi de créer un factory de commande a qui tu fournirais un org.w3c.dom.Node en parametre.

Par exemple on pourrait avoir:

Code :

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

1
2
3
4
5
6
7
8
CommandFactory{
public Command getCommand (Node node){
     //Cree la commande en fonction du type 
     Command command = newCommand (node);   
    command.init (node);     
    return command;        
}
}

Est-ce que tu peux nous fournir un exemple d'XML pour voir comment le parser au mieux ?

[bulbo]

Tu as un DTD, un schéma ou quelque chose pour me donner une idée de la structure de ton XML ?

Ma méthode est utile en cas d'élément imbriqué, par exemple:

Code :

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

1
2
3
4
5
<AdminCommand name="" code="" ...>
  <SectionSpecial>
    ....
  </SectionSpecial>
</AdminCommand>

Elle implique un mapping "un objet = un élément" et permet de passer à l'objet AdminCommand l'objet SectionSpecial et que celui ci sache quoi en faire sans cast ou test avec instanceof.

Pour ce qui est de traiter les attributs, la méthode la plus simple est d'avoir un constructeur prenant en argument l'objet Attribute de SAX et initialisant correctement tes objets.

Si ton XML correspond à ce que j'ai expliquer je recherche mon bout de code et je te le poste ici.

[Edit] Oups je viens de voir que tu utilises JDom, ma méthode est valide pour un parsing SAX [/Edit]

Bulbo