Problem with adding an XML node to an existent XML File

[Hugo Alberto Bedolla]

I got a problem when trying to write a new node to an xml file, basically the new node isn't appended to the file. I tried to write to another file, but in both cases, only the old data appears... what am I doing wrong? :S


Here's my code:

public void addXmlData(XmlData xmlData) throws XmlDataException {
        
        synchronized (this) {
            try {
                DocumentBuilderFactory dbfac = DocumentBuilderFactory.newInstance();
                DocumentBuilder docBuilder = dbfac.newDocumentBuilder();
                
                File file = new File("/home/myhome/an_xml_file.xml");
                
                Document doc = docBuilder.parse(file);
                
                XPath xpath = XPathFactory.newInstance().newXPath();
                
                String XmlDataName = xmlData.getXmlDataName();
                
                String search = "//XmlData[@name='" + XmlDataName + "']";
                
                XPathExpression expr = xpath.compile(search);
                
                Node result = (Node)expr.evaluate(doc, XPathConstants.NODE);
                
                if(result == null) {
                    Element XmlDataElement = doc.createElement("XmlData");
                    XmlDataElement.setAttribute("name", xmlData.getXmlDataName());
                    
                    Element descriptionElement = doc.createElement("XmlData-description");
                    descriptionElement.setTextXmlData(xmlData.getXmlDataDescription());
                    XmlDataElement.appendChild(descriptionElement);
                    
                    Element baseLocaleElement = doc.createElement("base-locale");
                    baseLocaleElement.setAttribute("locale", xmlData.getXmlDataLocale());
                    XmlDataElement.appendChild(baseLocaleElement);                    
                    
                    //set up a transformer
                    TransformerFactory transfac = TransformerFactory.newInstance();
                    Transformer trans = transfac.newTransformer();
                    trans.setOutputProperty(OutputKeys.OMIT_XML_DECLARATION, "yes");
                    //trans.setOutputProperty(OutputKeys.INDENT, "yes");

                    //create string from xml tree
                    //File file2 = new File("/home/myhome/an_xml_file.xml");
                    //FileOutputStream output = new FileOutputStream(file);
                    //Result streamResult = new StreamResult(output);
                    
                    Result streamResult = new StreamResult(file);
                    DOMSource source = new DOMSource(doc);
                    trans.transform(source, streamResult);
                    
                    //output.close();

                    
                } else {
                    throw new XmlDataException("XmlData already exists"); 
                }


            } catch (TransformerException ex) {
                Logger.getLogger(XmlDataManager.class.getName()).log(Level.SEVERE, null, ex);
            } catch (XPathExpressionException ex) {
                Logger.getLogger(XmlDataManager.class.getName()).log(Level.SEVERE, null, ex);
            } catch (SAXException ex) {
                Logger.getLogger(XmlDataManager.class.getName()).log(Level.SEVERE, null, ex);
            } catch (FileNotFoundException ex) {
                Logger.getLogger(XmlDataManager.class.getName()).log(Level.SEVERE, null, ex);
            } catch (IOException ex) {
                Logger.getLogger(XmlDataManager.class.getName()).log(Level.SEVERE, null, ex);
            } catch (ParserConfigurationException ex) {
                Logger.getLogger(XmlDataManager.class.getName()).log(Level.SEVERE, null, ex);
            }

            
        }
        
}

Thanks in advance.

[Pranav Raulkar]

Have a look at this

http://www.daniweb.com/software-development/java/threads/255757

Hope this helps.

[Rob Spoor]

Hugo, please UseOneThreadPerQuestion. This is the same question as here. Since the other thread has more replies I'm closing this one.