Problème pour supprimer attribut dans un fichier XML

Version imprimable

Bonjour à tous !

Tout est dans le titre ... Non ! Je plaisante mais là je craque ! Ça fait 4 heures que je cherche dans tous les sens (forum, bouqin, etc) et je n'arrive pas à comprendre d'où vient mon maudit problème.

J'ai un fichier XML qui contient des éléments auxquels sont associés des attributs (cf. ci-dessous) et je souhaiterai supprimer un de ces éléments :

Exemple l'id 105 mais je n'y arrive pas et ça me G..... !!!!

Je vous met aussi mon code.

Je vous remercie d'avance à tous pour votre aide.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<rooms>
<room id="100" state="available" type="rt1"/>
<room id="101" state="available" type="rt1"/>
<room id="102" state="available" type="rt1"/>
<room id="103" state="available" type="rt1"/>
<room id="104" state="available" type="rt2"/>
<room id="105" state="available" type="rt2"/>
<room id="106" state="available" type="rt2"/>
<room id="107" state="available" type="rt2"/>
<room id="200" state="available" type="rt1"/>
<room id="201" state="available" type="rt1"/>
<room id="202" state="available" type="rt2"/>
<room id="203" state="available" type="rt2"/>
<room id="204" state="available" type="rt3"/>
<room id="205" state="available" type="rt3"/>
<room id="206" state="available" type="rt2"/>
<room id="207" state="available" type="rt3"/>
<room id="300" state="available" type="rt1"/>
<room id="301" state="available" type="rt1"/>
<room id="302" state="available" type="rt2"/>
<room id="303" state="available" type="rt2"/>
<room id="304" state="available" type="rt2"/>
<room id="305" state="available" type="rt2"/>
<room id="400" state="available" type="rt2"/>
<room id="401" state="available" type="rt2"/>
<room id="402" state="available" type="rt2"/>
<room id="403" state="available" type="rt3"/>
<room id="404" state="available" type="rt3"/>
<room id="405" state="available" type="rt3"/>
<room id="406" state="available" type="rt4"/>
<room id="407" state="available" type="rt4"/>
<room id="1000" state="available" type="rt1"/>
</rooms>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
        final DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
 
        try {
 
            final DocumentBuilder builder = factory.newDocumentBuilder();		
            final Document document= builder.parse(new File(XmlDoc));
 
            NodeList Element = document.getElementsByTagName("room");
 
            if (Element != null && Element.getLength() > 0) {
 
                for (int i = 0; i < Element.getLength(); i++) {
 
                    Node node = Element.item(i);
                    Element e = (Element) node;
 
                    String id = Element.item(i).getChildNodes().item(0).getNodeValue();
 
 
                    if (id.equals(dataRoom)) {
                        node.getParentNode().removeChild(node);
                     } 
                }
            }
 
            //Sauvegarde du DOM dans un fichier XML
            transformerXml(document, XmlDoc);
 
        } catch (ParserConfigurationException | SAXException | IOException | DOMException e) {
            //Boîte du message d'erreur
            jop1.showMessageDialog(null, "Erreur durant la suppression d'un client dans le fichier XML\n" + e.toString(), "Erreur", JOptionPane.ERROR_MESSAGE);
        }

18/02/2015, 17h56
joel.drigo

Salut,

Code:

NodeList Element = document.getElementsByTagName("room");

Te donne directement une liste d'élements "room" : pas besoin d'aller chercher des éléments enfants dedans pour récupérer l'id, puisque c'est un attribut. C'est Element.getAttribute() pour récupérer un attribut (enfin, si c'est bien org.w3c.dom.Element que tu utilises comme je pense). Ensuite, attention à ne pas continuer à parcourir la NoteList après avoir supprimé l'élément : fait un break, pour arrêter.

Par ailleurs, utiliser xpath pourrait te simplifier ce type de traitement.
18/02/2015, 18h11
Jb_One73

Salut Joël !

Merci de m'avoir répondu aussi rapidement, effectivement j'utilise org.w3c.dom.Element, je viens de regarder ce que tu m'as noté mais je sais pas trop comment le mettre en place ?

Si j'écris ce que tu as marqué Element.getAttribute() il m’insulte donc j'ai mis document à la place là il me dit rien mais c'est moi qui ne sait plus quoi faire ... :calim2:

Pourquoi c'est si simple pour certains et compliqué pour d'autres ... J'en ai marre !!!
18/02/2015, 18h16
joel.drigo

Citation:

Envoyé par Jb_One73

Si j'écris ce que tu as marqué Element.getAttribute() il m’insulte donc j'ai mis document à la place là il me dit rien mais c'est moi qui ne sait plus quoi faire ... :calim2:

Code:

String id = e.getAttribute("id");

Tout simplement. Il faut regarder la doc aussi.

Euh ! J'ai écris mon dernier message trop vite je viens de trouver la solution alors je précise quand même que c'est grâce à toi
Je vais mettre ce que j'ai fait et qui fonctionne correctement (peut-être que cela en aidera d'autres).

J'ai une autre question mais cette fois ci sur la modification donc je ne sais pas si je peux la mettre dans ce post ?

Ah ! Oui ! Je voulais apporter une petite précision au break que tu m'as demandé de mettre, en fait je fais une boucle sur une collection qui peut contenir plusieurs éléments à supprimer donc je ne peux pas mettre de break.

Qu'en penses-tu ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
 
        final DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
 
        try {
 
            final DocumentBuilder builder = factory.newDocumentBuilder();		
            final Document document= builder.parse(new File(XmlDoc));
 
            NodeList Element = document.getElementsByTagName("room");
 
            if (Element != null && Element.getLength() > 0) {
 
                for (int i = 0; i < Element.getLength(); i++) {
 
                    Node node = Element.item(i);
                    Element e = (Element) node;
 
                    String id = e.getAttribute("id");
 
                    if (id.equals(dataRoom)) {
                        node.getParentNode().removeChild(node);
                     } 
                }
            }
 
            //Sauvegarde du DOM dans un fichier XML
            transformerXml(document, XmlDoc);
 
        } catch (ParserConfigurationException | SAXException | IOException | DOMException e) {
            //Boîte du message d'erreur
            jop1.showMessageDialog(null, "Erreur durant la suppression d'un client dans le fichier XML\n" + e.toString(), "Erreur", JOptionPane.ERROR_MESSAGE);
        }

Et pour xpath, tu trouveras tout ce qu'il faut ici : http://java.developpez.com/faq/xml/?...uoi-l-utiliser

Ce qui te permettra d'aller directement chercher l'élément avec
Code:

1 2 3 XpathExpression exp = xpath.compile(""/rooms/room[id='"+ dataRoom + "']"); // voir dans la faq comment obtenir cette variable xpath ! Element roomElement = (Element)exp.evaluate(document,XPathConstants.NODE);

Citation:

Envoyé par Jb_One73

J'ai une autre question mais cette fois ci sur la modification donc je ne sais pas si je peux la mettre dans ce post ?

Vas-y !

Citation:

Envoyé par Jb_One73

Ah ! Oui ! Je voulais apporter une petite précision au break que tu m'as demandé de mettre, en fait je fais une boucle sur une collection qui peut contenir plusieurs éléments à supprimer donc je ne peux pas mettre de break.

Bizarre...tu veux dire que tu peux avoir plusieurs chambres avec le même id ? (l'exemple avec le xpath ne marche plus dans ce cas, il faut utilise XPathConstants.NODESET, ce qui te donenra un NodeList au lieu d'un Element, qui contiendra que les rooms avec id= dataRoom)

Sinon, tu peux légèrement alléger le code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 NodeList roomElements = document.getElementsByTagName("room"); // il y a des conventions de nommage en Java, donc j'ai renommé Element en rooms if (roomElements!= null ) { for (int i = 0; i < roomElements.getLength(); i++) { Element roomElement = (Element) roomElements.item(i); String id = roomElement.getAttribute("id"); if (id.equals(dataRoom)) { roomElement.getParentNode().removeChild(roomElement ); } } }

Merci pour toutes ses infos, j'en prend bien note une fois mais j'ai commencé avec DOM et j'ai peur de perdre trop de temps "xpath", je ne suis pas retissant mais vu que j'ai pas mal de classe avec .... ??? pour l'avenir je vais m'y investir.

Mon autre question est cette fois-ci avec la modification, toujours dans le même cas donc j'ai fais ça, ça fonctionne presque bien sauf à la fin il me rajoute le n° de la chambre donc pour que ça soit explicite je met mon code et le résulta obtenu et celui que j'attend.

Merci pour tout.

Le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
        final DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
 
        try {
 
            final DocumentBuilder builder = factory.newDocumentBuilder();		
            final Document document= builder.parse(new File(XmlDoc));
 
            for (Iterator<String[]> iterator = dataRoom.iterator(); iterator.hasNext();) {
 
                String[] next = iterator.next();
 
                NodeList Element = document.getElementsByTagName("room");
 
                if (Element != null && Element.getLength() > 0) {
 
                    for (int i = 0; i < Element.getLength(); i++) {
 
                        Node node = Element.item(i);
                        Element e = (Element) node;
 
                        String id = e.getAttribute("id");
 
                        if (id.equals(next[0])) {
 
                            // Element id
                            node.setTextContent(next[0]);
 
                            e.setAttribute("type", next[1]);
                            e.setAttribute("state", next[2]);
                         } 
                    }
                }
            }

Le résultat obtenu :

Code:

1
2
 
<room id="100" state="unavailable" type="rt1">100</room>

Le résultat que j'attend :

Code:

1
2
 
<room id="100" state="unavailable" type="rt1"/>

19/02/2015, 07h44
joel.drigo
Bha, c'est le code suivant qui met l'id dans l'élément, sous forme de texte :
Code:

1 2 // Element id node.setTextContent(next[0]);
Si tu ne veux pas le faire, enlève cette ligne.
19/02/2015, 09h53
Jb_One73

Bonjour Joël !

Effectivement ta réponse est toute juste ! J'avais indiqué hier que le post était résolu mais j'ai omis de mettre le code final c'est à dire fonctionnel.
Tu as donc mis le source à ma place et je t'en remercie.

Encore merci et à très bientôt ;-)
19/02/2015, 10h08
thelvin

À noter que ces classes, le DOM fourni avec Java, sont très compliquées pour tout le monde, c'est pas que toi. Et cette complexité ne sert pas à grand-chose.

On t'a parlé de XPath pour sélectionner directement l'élément que tu veux,
moi je vais plutôt de parler de JDOM, pour manipuler du XML facilement en Java. Alors bon, en XML "facile" est une notion toute relative, mais ça reste infiniment plus confortable que le DOM fourni avec Java.

À réfléchir la prochaine fois que tu travailleras du XML en Java. Pas la peine de se coltiner des mauvais outils.