Discussion :

[nicomes371]

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 :

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,

[joel.drigo]

Salut,

Mets-nous ton code (entre balises [code][/code] — bouton ), 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().

[nicomes371]

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 :

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

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.

[joel.drigo]

Ok.

1. Tu ne peux pas écrire :
   

   Code :

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

   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 :

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

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

   Soit tu types ton iterator :
   

   Code :

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

   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).

[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>
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 :

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

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.

[joel.drigo]

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 :

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).

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 :

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

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).

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

Code :

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

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 :

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

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 !