Récupération valeur sur un Iterator

Bonjour,

débutant en Java, je travaille sur les sources d'un projet pour lequel la génération de classes Java à partir de fichier XSD (modifié) a eu un impact sur d'autres classes.

Sur l'une d'elle, il y a une boucle via un "iterator". Je n'arrive pas à récupérer les données de cette boucle :
Pièce jointe 183980
Le méthode "getCeinf..." retourne bien une list. J'ai tenté de modifier l'iterator par une boucle "for : " mais cela ne fonctionne pas, un message sous Eclipse m'informe que je peux boucler uniquement sur un tableau.

Comment récupérer les éléments présents dans ElementData ?

Merci pour l'aide que vous pourrez m'apporter.

Cdlt,

Salut,

Mets-nous ton code (entre balises [code][/code] — bouton http://club.developpez.com/aidenouve...boutoncode.png), parce qu'avec une copie d'écran c'est diffiicle de saisir tout le code.

Tout ce que que je vois

• c'est que tu as une ligne incompilable (if ( instanceof ...) ;
• que tu devrais typer ton Iteror (Iterator<Le nom de la classe>) ce qui te permettra de récupérer la valeur à chaque itération sans caster (et probablement supprimer ce instanceof) ;
• qu'il manque visiblement la récupération de la valeur : valeur = document.next().

Bonjour et merci.

J'avais mis une capture d'écran pour indiquer les valeurs que je souhaite avoir.

Le code de la boucle en lui-même est le suivant :

Code:

---

1 2 3 4 5 6 7

for (final Iterator document = pPli.getCeinfformAndCoeinfformAndCeinfE101E102().iterator(); document.hasNext();) { if (document.next() instanceof CeinfformImpl){ // le instaceof me permet de savoir sur quel modèle je vais travailler // chargement du bon modèle en fonction du test ci-dessus MarshallerFactory.getUnloadingStandard(fr.cnamts.esope.Constantes.NO_MODELE_ESOPE_DETACHEMENT_CEINFFORM).marshal(document.next(), pNormeEsopeWriter, pMarshallerPostProcessor); } }

---

Ce code est le code d'origine, je n'ai rien touché. La différence se situe sur la méthode getCeinfformAndCoeinfformAndCeinfE101E102 dans la classe Pli. Elle retourne une liste.

Merci.

Ok.

1. Tu ne peux pas écrire :
   
   Code:

   ---

   1 2 3 4

   if (document.next() instanceof CeinfformImpl){ // le instaceof me permet de savoir sur quel modèle je vais travailler // chargement du bon modèle en fonction du test ci-dessus MarshallerFactory.getUnloadingStandard(fr.cnamts.esope.Constantes.NO_MODELE_ESOPE_DETACHEMENT_CEINFFORM).marshal(document.next(), pNormeEsopeWriter, pMarshallerPostProcessor); }

   ---

   next() consomme la valeur, donc le second lira le suivant, s'il existe (tu obtiendras une NoSuchElementException à la fin de l'itération)
2. Soit tu utilises une variable locale :
   
   Code:

   ---

   1 2	Object next = document.next(); if ( next instanceof CeinfformImpl ) ...

   ---

   Soit tu types ton iterator :
   
   Code:

   ---

   1 2	for (final Iterator<CeinfformImpl> document = pPli.getCeinfformAndCoeinfformAndCeinfE101E102().iterator(); document.hasNext();) { CeinfformImpl cell = document.next();

   ---

   Il te faudra peut-être typer la liste dont tu demandes l'iterateur (on pourrait caster, mais c'est toujours mieux de typer en avant, ça évite toute interprétation en aval et tout risque d'erreur, et ça simplifie le code (plus de cast et de instanceof). D'ailleurs, quand je vois CeinfformImpl, je pense implémentation, donc interface, donc il peut être judicieux de typer par l'interface (toujours typer au plus générique).

Il n'y a rien qui va...:(
Si je tente de typer l'iterator, j'ai cette erreur sous Eclipse :
The type Iterator is not generic; it cannot be parameterized with arguments <CeinfformImpl>
Si je tente avec la création de la variable, ça plante, il ne rentre même pas dans le test en mode debug.

Pour info, le code d'origine (avant modification du XSD générant des classes), la boucle était de ce type :

Code:

---

1 2 3 4

for (final Iterator iteratorDocument = pPli.getNomDuModele().iterator(); iteratorDocument.hasNext();) { MarshallerFactory.getUnloadingStandard(fr.cnamts.esope.Constantes.NO_MODELE_LIE_AU_NOM_DU_MODELE).marshal(iteratorDocument.next(), pNormeEsopeWriter, pMarshallerPostProcessor); }

---

Dans la classe Pli, nous avions autant de méthodes que de modèles possibles. Sauf que désormais, nous n'avons plus qu'une seule méthode qui retourne une liste contenant tout ces modèles.

Si ça peut te permettre d'y voir un peu plus claire. Tout détaillé serait très long et très compliqué, je te résume le principal. Je continue de chercher de mon côté.

Merci.

Citation:

---

Envoyé par nicomes371

Il n'y a rien qui va...:(
Si je tente de typer l'iterator, j'ai cette erreur sous Eclipse :
The type Iterator is not generic; it cannot be parameterized with arguments <CeinfformImpl>

---

C'est que la liste n'est pas typée. C'est pour ça que j'ai dit :

Citation:

---

Envoyé par joel.drigo

O
Il te faudra peut-être typer la liste dont tu demandes l'iterateur (on pourrait caster, mais c'est toujours mieux de typer en avant, ça évite toute interprétation en aval et tout risque d'erreur, et ça simplifie le code (plus de cast et de instanceof).

---

Citation:

---

Envoyé par nicomes371

Si je tente avec la création de la variable, ça plante, il ne rentre même pas dans le test en mode debug.

---

Quoi plante ? Quel code ? Comment (message d'erreur ?)

Code:

---

1 2	Object next = document.next(); if ( next instanceof CeinfformImpl ) ...

---

Ce code n'a aucune raison de planter (sauf si tu as déjà une variable next dans le scope, auquel cas, il suffit de changer son nom).

Citation:

---

Envoyé par nicomes371

Pour info, le code d'origine (avant modification du XSD générant des classes), la boucle était de ce type :

Code:

---

1 2 3 4

for (final Iterator iteratorDocument = pPli.getNomDuModele().iterator(); iteratorDocument.hasNext();) { MarshallerFactory.getUnloadingStandard(fr.cnamts.esope.Constantes.NO_MODELE_LIE_AU_NOM_DU_MODELE).marshal(iteratorDocument.next(), pNormeEsopeWriter, pMarshallerPostProcessor); }

---

---

En gros, tu as changé l'itérateur et il n'itère plus seulement sur des CeinfformImpl comme avant.
Donc, si tu écris :

Code:

---

1 2 3 4 5 6 7 8

for (final Iterator iteratorDocument = pPli.getCeinfformAndCoeinfformAndCeinfE101E102().iterator(); iteratorDocument.hasNext();) { Object next = iteratorDocument.next(); if( next instanceof CeinfformImpl ) { MarshallerFactory.getUnloadingStandard(fr.cnamts.esope.Constantes.NO_MODELE_LIE_AU_NOM_DU_MODELE).marshal(next, pNormeEsopeWriter, pMarshallerPostProcessor); } }

---

ça devrait fonctionner. Sinon, ce n'est pas des instances de CeinfformImpl qui sont traitées par marshal.

Au passage, le typage en Java, ce n'est pas un artifice esthétique ! Même marshal() prend de l'Object en premier argument !

Merci merci !

C'est bon, ça fonctionne. A force de faire des essais et des bidouilles dans le code, c'était le foutoir.

J'ai tout repris, remis au propre avec tes explications et c'est bon, j'ai le résultat souhaité.

Merci pour le temps que tu as pris à m'aider.

@+