How to loop through xml file in javascript and skip a node in a loop

Question

0.00/5 (No votes)

See more:

Hi Everyone

I am working on a project where I am loading data from xml file to html using javascript.

What I have tried:

Javascript code

JavaScript

<script>
        var xmlhttp, xmlDoc;
        xmlhttp = new XMLHttpRequest();
        xmlhttp.open("GET", "books.xml", false);
        xmlhttp.send();
        xmlDoc = xmlhttp.responseXML;
        x = xmlDoc.getElementsByTagName("book")[0].childNodes;
        z = xmlDoc.getElementsByTagName("book")
        for (k = 0; k < x.length; k++) {          
                y = xmlDoc.getElementsByTagName("book")[k].firstChild;
            for (i = 0; i < x.length; i++) //looping xml childnodes
            {            
                if (y.nodeType == 1) {
                    document.write(y.nodeName + ":" + y.firstChild.nodeValue + "<br />");
                }
                y = y.nextSibling;
                
            }
            document.write("-------------------<br/>");
        }
    </script>

xml file

http://www.w3schools.com/xsl/books.xml[^]

Output:

C#

title:Everyday Italian
author:Giada De Laurentiis
year:2005
price:30.00
-------------------
title:Harry Potter
author:J K. Rowling
year:2005
price:29.99
-------------------
title:XQuery Kick Start
author:James McGovern
author:Per Bothner
author:Kurt Cagle
-------------------
title:Learning XML
author:Erik T. Ray
year:2003
price:39.95
-------------------

As it is seen, I have lost data from the third node, what may be the mistake.
Also is it possible to skip a node in a loop.

Please help

Thanks in advance.

Posted 24-Feb-16 5:02am

JPais

Updated 24-Feb-16 8:31am

Add a Solution

2 solutions

Solution 1

You should learn to use the debugger as soon as possible. Rather than guessing what your code is doing, It is time to see your code executing and ensuring that it does what you expect.

The debugger allow you to follow the execution line by line, inspect variables and you will see that there is a point where it stop doing what you expect.
Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]
http://docs.oracle.com/javase/7/docs/technotes/tools/windows/jdb.html[^]
https://www.jetbrains.com/idea/help/debugging-your-first-java-application.html[^]

With the debugger, you will be able to see exact how your code is handling the third node, and probably understand why it fail.

[edit]
To skip a book, you just have to put a test in the loop that scan the books tour test will say which book you skip.

Posted 24-Feb-16 5:52am

Patrice T

Updated 24-Feb-16 9:08am

v2

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Richard Deeming · Accepted Answer · 2016-02-24T08:31:00

The error is pretty obvious: both loops are limited to x.length iterations.

What is x? It's the child nodes of the first book element.

The first book element contains four child nodes. As a result, you attempt to read four book elements - that happens to be the right number, but only accidentally - and you attempt to read four child elements of each book node. Books 1, 2 and 4 have four child elements, but book 3 has eight child elements.

Fix your loops and the code will work properly, regardless of how many books there are, and how many child elements they have:

JavaScript

xmlDoc = xmlhttp.responseXML;

var z = xmlDoc.getElementsByTagName("book");
for (var k = 0; k < z.length; k++) {
    var x = z[k].childNodes;
    for (var i = 0; i < x.length; i++) {
        var y = x[i];
        if (y.nodeType == 1) {
            document.write(y.nodeName + ":" + y.firstChild.nodeValue + "<br />");
        }
    }
    document.write("-------------------<br/>");
}