http://nagoya.apache.org/bugzilla/show_bug.cgi?id=2480 *** shadow/2480 Fri Jul 6 07:37:30 2001 --- shadow/2480.tmp.16408 Fri Jul 6 07:37:30 2001 *************** *** 0 **** --- 1,293 ---- + +============================================================================+ + | org/apache/xerces/dom/TreeWalkerImpl.java is buggy | + +----------------------------------------------------------------------------+ + | Bug #: 2480 Product: Xerces-J | + | Status: NEW Version: 1.4.1 | + | Resolution: Platform: All | + | Severity: Major OS/Version: All | + | Priority: Other Component: DOM | + +----------------------------------------------------------------------------+ + | Assigned To: [EMAIL PROTECTED] | + | Reported By: [EMAIL PROTECTED] | + +----------------------------------------------------------------------------+ + | URL: | + +============================================================================+ + | DESCRIPTION | + import java.io.*; + import javax.xml.parsers.*; + import org.w3c.dom.*; + import org.w3c.dom.traversal.*; + import org.xml.sax.*; + + + /** + * Class TreeWalker141 tests org/apache/xerces/dom/TreeWalkerImpl.java + * + * This one does the correct solution under Xerces 1.4.0 and goes wrong under + Xerces 1.4.1. Output of the both different class pathes at the end of the + listing + * Classpath was: + * + * xerces-1_4_0/xerces.jar xor xerces-1_4_1/xerces.jar + * xalan-j_2_2_D6/bin/xalan.jar + * + * This class generates a Document and traverses it with the process2 method. + * + * @author Christian Geuer-Pollmann <[EMAIL PROTECTED]> + * @version %I%, %G% + */ + public class TreeWalker141 { + + /** + * Method main + * + * @param args + * @throws Exception + */ + public static void main(String args[]) throws Exception { + + System.out.println(""); + System.out.println(" --- " + + org.apache.xerces.framework.Version.fVersion + + " --- "); + + Document doc = getNodeSet1(); + NodeFilter nodefilter = null; + boolean entityReferenceExpansion = true; + int whatToShow = NodeFilter.SHOW_ALL; + TreeWalker treewalker = + ((DocumentTraversal) doc).createTreeWalker(doc, whatToShow, + nodefilter, entityReferenceExpansion); + ByteArrayOutputStream bytearrayoutputstream = + new ByteArrayOutputStream(); + PrintWriter printwriter = + new PrintWriter(new OutputStreamWriter(bytearrayoutputstream, + "UTF8")); + + process2(treewalker, printwriter); + printwriter.flush(); + System.out.println(); + System.out.println(new String(bytearrayoutputstream.toByteArray())); + System.out.println(); + } + + /** + * Method getNodeSet1 + * + * @return + * @throws ParserConfigurationException + */ + private static Document getNodeSet1() + throws ParserConfigurationException { + + DocumentBuilderFactory dfactory = + DocumentBuilderFactory.newInstance(); + + dfactory.setValidating(false); + dfactory.setNamespaceAware(true); + + DocumentBuilder db = dfactory.newDocumentBuilder(); + Document doc = db.newDocument(); + Element root = doc.createElement("RootElement"); + Element e1 = doc.createElement("Element1"); + Element e2 = doc.createElement("Element2"); + Element e3 = doc.createElement("Element3"); + Text e3t = doc.createTextNode("Text in Element3"); + + e3.appendChild(e3t); + root.appendChild(e1); + root.appendChild(e2); + root.appendChild(e3); + doc.appendChild(root); + + String s1 = + "<RootElement><Element1/><Element2/><Element3>Text in + Element3</Element3></RootElement>"; + + return doc; + } + + /** + * recursively traverses the tree + * + * for simplicity, I don't handle comments, Attributes, PIs etc. + * Only Text, Document and Element + * + * @param treewalker + * @param printwriter + */ + private static void process2(TreeWalker treewalker, + PrintWriter printwriter) { + + Node currentNode = treewalker.getCurrentNode(); + + switch (currentNode.getNodeType()) { + + case Node.TEXT_NODE : + case Node.CDATA_SECTION_NODE : + printwriter.print(currentNode.getNodeValue()); + break; + + case Node.ENTITY_REFERENCE_NODE : + case Node.DOCUMENT_NODE : + case Node.ELEMENT_NODE : + default : + if (currentNode.getNodeType() == Node.ELEMENT_NODE) { + printwriter.print('<'); + printwriter.print(currentNode.getNodeName()); + printwriter.print(">"); + } + + Node node1 = treewalker.firstChild(); + + if (node1 == null) { + System.out.println(getNodeTypeString(currentNode.getNodeType()) + + "_NODE parent: " + + currentNode.getNodeName() + + " has no childs "); + } else { + System.out.println(getNodeTypeString(currentNode.getNodeType()) + + "_NODE parent: " + + currentNode.getNodeName() + + " has childs "); + + while (node1 != null) { + { + String qStr = ""; + + for (Node q = node1; q != null; q = q.getParentNode()) { + qStr = q.getNodeName() + "/" + qStr; + } + + System.out + .println(getNodeTypeString(currentNode.getNodeType()) + + "_NODE process child " + qStr); + } + + + // recursion !!! + process2(treewalker, printwriter); + + node1 = treewalker.nextSibling(); + + if (node1 != null) { + System.out.println("treewalker.nextSibling() = " + + node1.getNodeName()); + } + } // while(node1 != null) + } + + System.out.println("setCurrent back to " + + currentNode.getNodeName()); + treewalker.setCurrentNode(currentNode); + + if (currentNode.getNodeType() == Node.ELEMENT_NODE) { + printwriter.print("</"); + printwriter.print(currentNode.getNodeName()); + printwriter.print(">"); + } + + break; + } + } + + /** Field nodeTypeString */ + private static String[] nodeTypeString = new String[]{ "", "ELEMENT", + "ATTRIBUTE", + "TEXT_NODE", + "CDATA_SECTION", + "ENTITY_REFERENCE", + "ENTITY", + "PROCESSING_INSTRUCTIO + N", + "COMMENT", + "DOCUMENT", + "DOCUMENT_TYPE", + "DOCUMENT_FRAGMENT", + "NOTATION" }; + + /** + * Transforms <code>org.w3c.dom.Node.XXX_NODE</code> NodeType values into + * XXX Strings. + * + * @param nodeType as taken from the {@link org.w3c.dom.Node#getNodeType} + function + * @return the String value. + * @see org.w3c.dom.Node#getNodeType + */ + public static String getNodeTypeString(short nodeType) { + + if ((nodeType > 0) && (nodeType < 13)) { + return nodeTypeString[nodeType]; + } else { + return ""; + } + } + + } + + /* + --- Xerces 1.4.0 --- + DOCUMENT_NODE parent: #document has childs + DOCUMENT_NODE process child #document/RootElement/ + ELEMENT_NODE parent: RootElement has childs + ELEMENT_NODE process child #document/RootElement/Element1/ + ELEMENT_NODE parent: Element1 has no childs + setCurrent back to Element1 + treewalker.nextSibling() = Element2 + ELEMENT_NODE process child #document/RootElement/Element2/ + ELEMENT_NODE parent: Element2 has no childs + setCurrent back to Element2 + treewalker.nextSibling() = Element3 + ELEMENT_NODE process child #document/RootElement/Element3/ + ELEMENT_NODE parent: Element3 has childs + ELEMENT_NODE process child #document/RootElement/Element3/#text/ + setCurrent back to Element3 + setCurrent back to RootElement + setCurrent back to #document + + <RootElement><Element1></Element1><Element2></Element2><Element3>Text in + Element3</Element3></RootElement> + + + --- Xerces 1.4.1 --- + DOCUMENT_NODE parent: #document has childs + DOCUMENT_NODE process child #document/RootElement/ + ELEMENT_NODE parent: RootElement has childs + ELEMENT_NODE process child #document/RootElement/Element1/ + ELEMENT_NODE parent: Element1 has childs + ELEMENT_NODE process child #document/RootElement/Element2/ + ELEMENT_NODE parent: Element2 has childs + ELEMENT_NODE process child #document/RootElement/Element3/ + ELEMENT_NODE parent: Element3 has childs + ELEMENT_NODE process child #document/RootElement/Element3/#text/ + setCurrent back to Element3 + setCurrent back to Element2 + treewalker.nextSibling() = Element3 + ELEMENT_NODE process child #document/RootElement/Element3/ + ELEMENT_NODE parent: Element3 has childs + ELEMENT_NODE process child #document/RootElement/Element3/#text/ + setCurrent back to Element3 + setCurrent back to Element1 + treewalker.nextSibling() = Element2 + ELEMENT_NODE process child #document/RootElement/Element2/ + ELEMENT_NODE parent: Element2 has childs + ELEMENT_NODE process child #document/RootElement/Element3/ + ELEMENT_NODE parent: Element3 has childs + ELEMENT_NODE process child #document/RootElement/Element3/#text/ + setCurrent back to Element3 + setCurrent back to Element2 + treewalker.nextSibling() = Element3 + ELEMENT_NODE process child #document/RootElement/Element3/ + ELEMENT_NODE parent: Element3 has childs + ELEMENT_NODE process child #document/RootElement/Element3/#text/ + setCurrent back to Element3 + setCurrent back to RootElement + setCurrent back to #document + + <RootElement><Element1><Element2><Element3>Text in + Element3</Element3></Element2><Element3>Text in + Element3</Element3></Element1><Element2><Element3>Text in + Element3</Element3></Element2><Element3>Text in + Element3</Element3></RootElement> + */ --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
