将对象列表转换为嵌套的UL不起作用

时间:2013-02-20 11:54:46

标签: javascript recursion

我试图将通过ajax获取的对象列表转换为类别菜单的嵌套ul。看完这个网站后,我找到了一个PHP函数并将其转换为javascript。

不幸的是它不起作用:

function has_children(data, id) {
    for(a=0; a<data.length; a++) {
        if (data[a].ParentID == id) return true;
    }
    return false;
}

function renderCategoryTree(data, parent) {
    result = "<ul>";
    for(index=0; index<data.length; index++) {
        if (data[index].ParentID == parent) {
            result = result + "<li>" + data[index].Name;
            if (has_children(data, data[index].ID)) 
                result = result + renderCategoryTree(data, data[index].ID);
            result = result + "</li>";
        }
    }
    result = result + "</ul>";

    return result;
}

var stuff= [ { ID: 1, ParentID: 0, Name: 'Development' },
{ ID: 2, ParentID: 0, Name: 'Databases' },
{ ID: 3, ParentID: 0, Name: 'Systems' },
{ ID: 4, ParentID: 1, Name: 'java' },
{ ID: 5, ParentID: 1, Name: 'c++' },
{ ID: 6, ParentID: 1, Name: 'python' },
{ ID: 7, ParentID: 1, Name: 'ruby' },
{ ID: 8, ParentID: 2, Name: 'mysql' },
{ ID: 9, ParentID: 2, Name: 'oracle' },
{ ID: 10, ParentID: 2, Name: 'sqlite' },
{ ID: 11, ParentID: 3, Name: 'linux' },
{ ID: 12, ParentID: 3, Name: 'windows' } ];

  alert(renderCategoryTree(stuff, 0));

渲染停在&#34; ruby​​&#34;。我认为问题在于javascript输入的工作方式,但我不确定。有人可以帮忙吗?谢谢!

2 个答案:

答案 0 :(得分:4)

我不是百分百肯定,但我认为你的循环计数器是全局的。所以,如果你写

for(var a=0; a<data.length; a++) {

而不是

for(a=0; a<data.length; a++) {

for(var index=0; index<data.length; index++) {

而不是

for(index=0; index<data.length; index++) {

它会起作用。

答案 1 :(得分:1)

我发现了问题:因为javascript变量在嵌套函数调用中存活,所以必须使用var使它们成为本地。

只需将var添加到索引声明就可以了!