• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 235
  • Last Modified:

How to build multiple XML documents by mergeing two XML documents using JDOM.

I am building multiple dynamic xml documents from result set and storing them to an ArrayList called 'xmlDocumentList'.
I have another static xml document called 'staticDocument'. Where static document looks like:

<Transmission>
      <TransmissionHeader>
              <UserName />
              <Password />
        </TransmissionHeader>
      <TransmissionBody>
      <!----- here goes GLogXMLElement XML element ---->
      </TransmissionBody>
   </Transmission>

Now I want to create another ArrayList 'finalDocumentList' out of mergeing 'staticDocument' with each element of 'xmlDocumentList'. That will be like each element of 'finalDocumentList' is made from merging 'staticDocument' with 'xmlDocumentList[i]' (one of dynamic document element) at <TransmissionBody>. When I do that with attached code snippet, it creates the GlogFinal0.xml, GlogFinal1, GlogFinal2..etc file with all having same content in them. It uses only first element of 'xmlDocumentList' to build all elements of 'finalDocumentList'.



ArrayList dynamicDocumentList = xmlBuilder.getDynamicXmlDocuments();
        Document staticDocument = xmlBuilder.getStaticXmlWrapper();
        ArrayList finalDocumentList = new ArrayList(); 
        Element staticRoot = staticDocument.getRootElement();
        Element staticChild = staticRoot.getChild("TransmissionBody");
        Element dynamicRoot = null;
        for(int i = 0; i < dynamicDocumentList.size(); i++)
        {
            dynamicRoot = ((Document)dynamicDocumentList.get(i)).getRootElement();
            staticChild.addContent(dynamicRoot.detach()); 
           finalDocumentList.add(staticDocument);
        }
          FileOutputStream finalXmlFile = null;
          XMLOutputter xmlOutPutter = null;
          for (int i = 0; i < finalDocumentList.size(); i++)
          {
            finalXmlFile =  new FileOutputStream("F:\\JavaProjects\\WorkspaceDev\\OTM\\xsl\\GLogFinal"+i+".xml");
            xmlOutPutter = new XMLOutputter(Format.getPrettyFormat());
            xmlOutPutter.output((Document)(finalDocumentList.get(i)), finalXmlFile);
          }

Open in new window

0
MehtaJasmin
Asked:
MehtaJasmin
  • 2
1 Solution
 
kawasCommented:
Its copying the whole thing because you are attaching the root nodes of a Document.

I think that xmlBuilder.getDynamicXmlDocuments()  is incorrect. Perhaps the method should create new documents each containing the node of interest. The rest of the code looks correct.
0
 
MehtaJasminAuthor Commented:
I know that xmlBuilder.getDynamicXmlDocuments()  is creating correct ArrayList out of the result set. I have printed the list before. It shows correct results.

 Attached is the code for that method.

The problem is merging of static doc with dynamic element in loop.
    private ArrayList getDynamicXmlDocuments()
    {
        Statement stmt = null;
        ResultSet rs = null;
        org.w3c.dom.Document document = null;
        oracle.xml.parser.v2.XMLDocument xmlDocument = null;
        org.jdom.input.DOMBuilder domBuilder = null;
        ArrayList xmlDocumentList = new ArrayList();
        Document jdomDocument = null;
        XMLType xmlType = null;
        String sql = "select OTM from OTM_XML";
 
        try
        {
            stmt = 
(ConnectionManager.getInstance()).getConnection().createStatement();
            logger.info("SQL: " + sql);
            rs = stmt.executeQuery(sql);
            OracleResultSet orset = (OracleResultSet)rs;
            while (orset.next())
            {   
                xmlType = XMLType.createXML(orset.getOPAQUE(1));
                document = xmlType.getDOM();
                xmlDocument = (oracle.xml.parser.v2.XMLDocument)document;
 
                domBuilder = new org.jdom.input.DOMBuilder();
                jdomDocument = domBuilder.build(document);
                xmlDocumentList.add(jdomDocument);
            }
            System.out.println("xml document list size : "+xmlDocumentList.size());
            FileOutputStream out = null;
            XMLOutputter xmlOutPutter = null;
            for (int i = 0; i < xmlDocumentList.size(); i++)
            {
              out =  new FileOutputStream("F:\\JavaProjects\\WorkspaceDev\\OTM\\xsl\\GLogDynamic"+i+".xml");
              xmlOutPutter = new XMLOutputter(Format.getPrettyFormat());
              xmlOutPutter.output((Document)(xmlDocumentList.get(i)), out);
            }    
        }
        catch (SQLException se)
        {
            logger.error(RootException.getStackTraceString(se));
        }
        catch (Exception se)
        {
            logger.error(RootException.getStackTraceString(se));
        }
        finally
        {
            try
            {
                if (stmt != null)
                {
                    stmt.close();
                    stmt = null;
                }
            }
            catch (SQLException se)
            {
                System.out.println("SQLException in finally of statement: " + 
                                   se.getMessage());
                logger.info("SQLException in finally of statement: " + 
                            se.getMessage());
            }
            try
            {
                if (rs != null)
                {
                    rs.close();
                    rs = null;
                }
            }
            catch (SQLException se)
            {
                System.out.println("SQLException in finally of resultset: " + 
                                   se.getMessage());
                logger.info("SQLException in finally of resultset: " + 
                            se.getMessage());
            }
        }
 
        return xmlDocumentList;
    }

Open in new window

0
 
kawasCommented:
Hmm, they are all the same, because in your for loop, you are adding the same element every time

staticChild.addContent(dynamicRoot.detach());

All of the elements are 'live' (changes occur even after you add them to other documents) so for each iteration of the for loop staticChild keeps adding more children and all the nodes in the list finalDocumentList are getting modified.

Does this make sense?

I think that the best solution would be to create new elements in the for loop so that they have limited scope and you cant modify them inadvertently.

0

Featured Post

Free Tool: Subnet Calculator

The subnet calculator helps you design networks by taking an IP address and network mask and returning information such as network, broadcast address, and host range.

One of a set of tools we're offering as a way of saying thank you for being a part of the community.

  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now