Discussion :

[Itof]

Salut à tous!

J'aimerais créer une méthode pour supprimer un élément dans un fichier XML. En fait, je voudrais supprimer un élément particulier, càd, celui dont l'attribut "name" correspond au String elt.

Le problème est que j'utilise le type "Element" pour faire la recherche du bon élément, et le type "Node" pour réaliser la suppression.

Je ne connais pas d'autres méthodes...

Voici ma méthode JAVA:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public void removeElt(String elt){
	NodeList list= document.getElementsByTagName("uIndex");
	int nb= list.getLength();
	int i=0;
	while (i<nb){
		Node child = list.item(i);
		Element childElt = (Element)child;
		String atr = childElt.getAttribute("name");
		if (atr.equals(elt)){
			root.removeChild(child);
		}
		i=i+1;
	}
	save();
}

et mon fichier XML:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
<uIndexList xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="./xsd/uIndexList.xsd">
	<uIndex name="un"/>
	<uIndex name="coucou"/>
	<uIndex name="salut"/>
	<uIndex name="bijour"/>
	<uIndex name="hello"/>
	<uIndex name="rigole"/>
	<uIndex name="ralout"/>
</uIndexList>

Merci d'avance!

[zoltar]

Salut,

Il faut utiliser la méthode 'getAttribute' (méthode de l'objet Node dont hérite l'objet Element) :

Code :

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

getAttribute("name")

Tu récupères dans une variable la valeur de l'attribut 'name' et tu fais un test :

Code :

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

if ma_variable=="elt"

Zoltar.

[Itof]

Merci Zoltar, mais c'est un peu ce que je fais, je pense...
Non???

Je ne peux pas utiliser la méthode getAttribute(String) sur un Node, elle n'est valable que pour le type Element.
Et je dois travailler sur un Node pour faire ma suppression!
De plus, je ne peux pas "caster" mon Node en un Element.

Si j'ai mal compris ta répnose, dis le moi...

Merci quand même

Salut,

Il faut utiliser la méthode 'getAttribute' (méthode de l'objet Node dont hérite l'objet Element) :

Code :

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

getAttribute("name")

Tu récupères dans une variable la valeur de l'attribut 'name' et tu fais un test :

Code :

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

if ma_variable=="elt"

Zoltar.

[zoltar]

Effectivement, c'était une coquille, désolé

C'est la méthode removeChild qui est une méthode de Node dont hérite element. La méthode getAttributes n'est pas une méthode de Node.

Tu peux essayer la propriété 'attributes' de Node : tu obtiens une collection de type NamedNodeMap des noeuds attributs, enfants du noeud spécifié ou null s'il n'en existe pas.

Tu devrais avoir un code :

Code :

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

1
2
3
4
 
attribs=le_Node.attributes;
if(attribs(0).value=="elt")
...

Zoltar.

[Itof]

Oui, je veux bien essayer, mais ça me permettra encore de supprimer le Node???

Je crains qu'en faissant ça, je ne sois plus capable de supprimer le Node. Car il faut que le Node que je veux supprimer soit rattaché à un Node père. => pour pouvoir utiliser la méthode Node removeChild(Node oldchild)

Or, en faisant ce que tu me dis, je vais récupérer une liste de Nodes.
Est-ce que les Nodes de cette liste entretiennent toujours des relations entre-eux, ou avec leur origine??? Est-ce qu'ils sont associés au père???

Merci

[zoltar]

Salut,

Oui, je veux bien essayer, mais ça me permettra encore de supprimer le Node???

En faisant :

Code :

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

childElt.attributes;

Tu récupères la liste (ou collection des noeuds attributs associés à un noeud de type élément. Au lieu d'utiliser la méthode getAttributes qui va pointer sur un attribut nommé, tu récupères avec cette méthode l'ensemble des attributs. Donc attribs(0) est le premier attribut de la liste.
Au niveau de la suppression, rien ne change.

[philemon_siclone]

Salut,

Je ne comprend pas où est ton problème Itof : ton code fonctionne ou pas? Moi le seul problème que je vois c'est que si les fils de uIndex ne sont pas tous des elements ton code génère une exception lors du transtypage. Par contre si tu es sur que tous les fils de "uIndex" sont des elements, alors il faut qu'on m'explique ou est le problème.

Cordialement,

[Itof]

Salut,

Mon code ne fonctionne pas... il me crache une "CastException". Comme toi, je me demande pourquoi, vu le format de mon fichier XML. Juges en par toi même!
Si tu vois où pourrais se trouver l'erreur, n'hésite pas à me le faire savoir!

Merci!

Salut,

Je ne comprend pas où est ton problème Itof : ton code fonctionne ou pas? Moi le seul problème que je vois c'est que si les fils de uIndex ne sont pas tous des elements ton code génère une exception lors du transtypage. Par contre si tu es sur que tous les fils de "uIndex" sont des elements, alors il faut qu'on m'explique ou est le problème.

Cordialement,

[philemon_siclone]

t'as essayé de debugger ? d'afficher le nodeName() et le nodeType() du noeud avant transtypage, par exemple ?

Sinon, si tu veux juste securiser ton code sans comprendre je suppose qu'il suffit de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public void removeElt(String elt){ 
   NodeList list= document.getElementsByTagName("uIndex"); 
   int nb= list.getLength(); 
   int i=0; 
   while (i<nb){ 
      Node child = list.item(i); 
      if (child.getNodeTYPE() == ELEMENT_NODE)
      {
      Element childElt = (Element)child; 
      String atr = childElt.getAttribute("name"); 
      if (atr.equals(elt)){ 
         root.removeChild(child); 
}
      } 
      i=i+1; 
   } 
   save(); 
}

Cordialement,

PS : code pas compilé, pas testé (désolé, c'est juste que j'ai pas le jdk, je code en C++).

[philemon_siclone]

j'suis con...

C'est bien sur le fait de boucler de 0 à n-1 alors que tu supprimes des elements dans la boucle qui fait tout foirer, non?

[philemon_siclone]

j'suis con...

C'est bien sur le fait de boucler de 0 à n-1 alors que tu supprimes des elements dans la boucle qui fait tout foirer, non?

[philemon_siclone]

Je propose le code suivant (pas compilé, pas executé):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public void removeElt(String elt){ 
   NodeList list= document.getElementsByTagName("uIndex"); 
   int i=list.getLength(); 
   while (i>0){ 
      i--;
      Node child = list.item(i); 
      Element childElt = (Element)child; 
      String atr = childElt.getAttribute("name"); 
      if (atr.equals(elt)){ 
         root.removeChild(child); 
      } 
   } 
   save(); 
}

[Itof]

Oh Putain!!!!!!!!!

je n'ai pas d'autres mots!!!! MERCI!!!!

Ca fonctionne, c'est parfait! Si tu pouvais juste m'expliquer ce qui diffère avec mon code... ça pourrait m'aider pour la suite de mon job

Merci beaucoup, c'est cool que des mecs comme toi passent un peu de leur temps pour aider des mecs paumés comme moi!!!

A+ Itof

Je propose le code suivant (pas compilé, pas executé):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public void removeElt(String elt){ 
   NodeList list= document.getElementsByTagName("uIndex"); 
   int i=list.getLength(); 
   while (i>0){ 
      i--;
      Node child = list.item(i); 
      Element childElt = (Element)child; 
      String atr = childElt.getAttribute("name"); 
      if (atr.equals(elt)){ 
         root.removeChild(child); 
      } 
   } 
   save(); 
}

[philemon_siclone]

Bon déjà as-tu compris le problème ?
Si tu as 4 fils (indexés de 0 à 3), mettons que l'avant dernier soit celui qu'il faille supprimer. Lors de la suppression i vaut donc 2. Tu supprimes le noeud, tu incrementes i (qui vaut maintenant 3) et tu essais d'acceder à item(3) alors qu'il n'y plus que 3 fils dans ta liste de noeud. Donc il doit renvoyer Null; et transtyper Null en "Element", ça marche pas .

Maintenant mes modifs doivent te paraitre limpides...

Tcho,

[Itof]

Bien vu!!!

C'est tout de suite plus clair

Merci beaucoup, et au plaisir de te revoir (qui sais?! )