jQuery $.each() multidimensional JSON array

Question

What did I manage do do wrong here? I can't see it, through the first $.each it works, get to the second it stops..

var testJSON = {"cluster":[{"node":[{"name":"one", "number":'100', "error":"none"},{"name":"two", "number":'200', "error":"none"},{"name":"three", "number":'300', "error":"found"},{"name":"four", "number":'400', "error":"none"}]}]}

if (testJSON.cluster.length != 0)
{
    $.each(testJSON.cluster, function(i, clstr)
    {
        $('.clusters').append('<ul class="nodes">');
        $.each(clstr.node, function(i, ndes)
        {
            $.find('ul').append('<li>'+ndes.name+'</li>');
        });
        $('.clusters').append('</ul>');
    });
}

Stops? We'll need a bit more than that. Did you receive any uncaught exceptions? — Delan Azabani
– Delan Azabani, Commented Aug 17, 2011 at 15:11
does the outer each work in the first place? append() is adding a DOM node, not text - append ("</ul>") seems out of place. — BrokenGlass
– BrokenGlass, Commented Aug 17, 2011 at 15:15

FishBasketGordo · Accepted Answer · 2011-08-17 15:13:56Z

7

Change your code to the following:

if (testJSON.cluster.length != 0) {
    $.each(testJSON.cluster, function(i, clstr) {
        $('.clusters').append('<ul class="nodes"></ul>');
        $.each(clstr.node, function(i, ndes) {
            $('.clusters ul.nodes').append('<li>' + ndes.name + '</li>');
        });;
    });
}

When you append an element, you don't have to later append the closing tag. Also, find cannot be called directly from the jQuery object. You need a selector.

answered Aug 17, 2011 at 15:13

FishBasketGordo

23.2k4 gold badges60 silver badges92 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

ShankarSangoli · Accepted Answer · 2011-08-17 15:26:33Z

2

What is $.find, you are getting an exception in the inner loop and then it stops.

$.find('ul').append('<li>'+ndes.name+'</li>');

Are you looking for the ul which you added in the outer loop? If yes, then use

$('.clusters ul').append('<li>'+ndes.name+'</li>')

edited Aug 17, 2011 at 15:26

answered Aug 17, 2011 at 15:12

ShankarSangoli

69.9k13 gold badges96 silver badges124 bronze badges

Collectives™ on Stack Overflow

jQuery $.each() multidimensional JSON array

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related