Is this Node retrieval optimal

Question

Hello all I have the following xml document
<?xml version="1.0" encoding="utf-8" ?>
<iptablesfile name="Untitled.xml">
<CHAIN name="ACCEPT">
<RULE>Lets</RULE>
<RULE>OK</RULE>
</CHAIN>
<CHAIN name="REJECT">
<RULE>Whatever</RULE>
<RULE>Day</RULE>
</CHAIN>
<CHAIN name="FORWARD">
<RULE>Light</RULE>
<RULE>Cancel</RULE>
</CHAIN>
</iptablesfile>
And the following code to get the values from this document
Element root = document.getDocumentElement();
root.normalize();
NodeList list = root.getElementsByTagName("CHAIN");
Node fileName = list.item(0).getParentNode();
System.out.println(((Element)fileName).getAttribute("name"));
for (int i = 0; i<list.getLength(); i++){
     Node n= list.item(i);
     if(n.getNodeType() == ELEMENT_TYPE){
          System.out.println(i+"-"+((Element)n).getAttribute("name"));
     NodeList ipRules = ((Element)n).getElementsByTagName("RULE");
     for (int x = 0; x< ipRules.getLength(); x++){
          Node w = ipRules.item(x);
          System.out.println((w.getChildNodes()).item(0).getNodeValue());
And the question is, how optimal is this, I feel this is very redudant code.
Thank you for any help,
Gyftus     

Answer 1

<?xml version="1.0" encoding="utf-8" ?>
<iptablesfile name="Untitled.xml">
<CHAIN name="ACCEPT">
<RULE>Lets</RULE>
<RULE>OK</RULE>
</CHAIN>
<CHAIN name="REJECT">
<RULE>Whatever</RULE>
<RULE>Day</RULE>
</CHAIN>
<CHAIN name="FORWARD">
<RULE>Light</RULE>
<RULE>Cancel</RULE>
</CHAIN>
</iptablesfile>
And the following code to get the values from this document
Element root = document.getDocumentElement();
root.normalize();
NodeList list = root.getElementsByTagName("CHAIN");
Node fileName = list.item(0).getParentNode();
System.out.println(((Element)fileName).getAttribute("name"));
for (int i = 0; i<list.getLength(); i++){
Node n= list.item(i);
if(n.getNodeType() == ELEMENT_TYPE){
System.out.println(i+"-"+((Element)n).getAttribute("name"));
NodeList ipRules = ((Element)n).getElementsByTagName("RULE");
for (int x = 0; x< ipRules.getLength(); x++){
Node w = ipRules.item(x);
System.out.println((w.getChildNodes()).item(0).getNodeValue());
}