Home/ Questions/Q 6565589
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-25T14:06:45+00:00

I will keep this very short: I’m trying to make a loop through an

  • 0

I will keep this very short:
I’m trying to make a loop through an xml-document for a gallery. I got a script that should work, but doesn’t. Can anyone please tell me where I did wrong?

I didn’t want to make this longer because the problem is simple and have been pondering over this since yesterday and this is the closest I get.

I want to loop through the xml-file and print out “path” and “file” first and most. I’m building a gallery and thought that the best way to save all the data for the images was an xml-file, but now I can’t get it to loop correctly. In the script I made the page print out both x and i, which resulted with x being 1 and i being 0, hence it hasn’t worked through the for-loop at all as I see it.

Any help would be appreciated, because I’m stuck. Been trying so many solutions that my head is spinning and I can’t get any further without a nudge in the right direction.

The html/javascript:

        <script type="text/javascript">
            function displayResult()
            {
                if (window.XMLHttpRequest)
                {// code for IE7+, Firefox, Chrome, Opera, Safari
                    xmlhttp=new XMLHttpRequest();
                }
                else
                {// code for IE6, IE5
                    xmlhttp=new ActiveXObject("Microsoft.XMLHTTP");
                }
                xmlhttp.open("GET","../gallery/gallery.xml",false);
                xmlhttp.send();
                xmlDoc=xmlhttp.responseXML;


                x=xmlDoc.getElementsByTagName("session");
                for (i=0;i<x.length;i++)
                { 
                    img = "<img src='";
                    path = (x[0].getElementsByTagName("path")[0].childNodes[0].nodeValue);
                    file = (x[i].getElementsByTagName("file")[i].childNodes[0].nodeValue);
                    end = "' /><br />";
                    name = (x[i].getElementsByTagName("name")[i].childNodes[0].nodeValue);
                    txt = "x:" + x.length + "| i " + i + "<br />" + img + path + file + end + name + "<br />";
                    document.getElementById("content").innerHTML = txt;
                    //document.write(txt);
                }
            }
        </script>
    </head>
    <body onload="displayResult()">
        <div id='content'></div>
    </body>
</html>

xml-file:

<gallery>
    <session>
        <path>../gallery/beauty/</path>
        <item>
            <file>_DSC2331.jpg</file>
            <name>Picture 1</name>
        </item>
        <item>
            <file>_DSC2339.jpg</file>
            <name>Picture 2</name>
        </item>
        <item>
            <file>_DSC2350.jpg</file>
            <name>Picture 3</name>
        </item>
        <date>2011-08-03</date>
    </session>
</gallery>
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    If I can make some suggestions:

    1. Use the var keyword within functions to make those variables local to that function. The code you have at the moment would set values in the global namespace, which is often considered bad practice (e.g. you could overwrite other people’s variables, or other people could overwrite yours). Also declare your variables at the start of a function, as they will be hoisted there anyway.
    2. Split your code up into more meaningful functions. This way they become easier to read and often then become more reusable.
    3. Make sure you loop through items as well as sessions.
    4. Consider using a Javascript framework like jQuery. They can often simplify the code you have to write, and you will usually end up writing less code yourself.

    .

    <html>
    <head>
        <script type="text/javascript">
            function loadDoc(url) {
                var xmlhttp = null;
                if (window.XMLHttpRequest) {
                    // code for IE7+, Firefox, Chrome, Opera, Safari
                    xmlhttp = new XMLHttpRequest();
                } else {
                    // code for IE6, IE5
                    xmlhttp = new ActiveXObject("Microsoft.XMLHTTP");
                }
                xmlhttp.open("GET", url, false);
                xmlhttp.send();
                return xmlhttp.responseXML;
            }

            function getContent(sessions) {
                var items = null,
                    i = 0,
                    j = 0,
                    img = "",
                    path = "",
                    file = "",
                    end = "",
                    name = "",
                    txt = "";
                for (i = 0; i < sessions.length; i++) { 
                    items = sessions[i].getElementsByTagName("item");
                    path = sessions[i].getElementsByTagName("path")[0].childNodes[0].nodeValue;
                    for (j = 0; j < items.length; j++) {
                        img = "<img src='";
                        file = items[j].getElementsByTagName("file")[0].childNodes[0].nodeValue;
                        end = "' /><br />";
                        name = items[j].getElementsByTagName("name")[0].childNodes[0].nodeValue;
                        txt += "session[" + i + "] item[" + j + "]<br />" + img + path + file + end + name + "<br />";
                    }
                }
                return txt;
            }

            function displayResult()
            {
                var xmlDoc = loadDoc("../gallery/gallery.xml");
                var sessions = xmlDoc.getElementsByTagName("session");
                var txt = getContent(sessions);
                document.getElementById("content").innerHTML = txt;
            }
        </script>
    </head>
    <body onload="displayResult()">
        <div id='content'></div>
    </body>
</html>
    • 0