for 循环中的 appendChild 只添加 1 个子项

appendChild in for loop only adds 1 child

提问人:user1649326 提问时间:10/4/2012 最后编辑:Communityuser1649326 更新时间:9/11/2021 访问量:24736

问:

在 JavaScript 中,我正在使用 HTML 表格创建一个网格(您在 Photoshop 中看到的网格类型)。网格大小将是可变的,即可以由用户更改,因此必须计算每个网格正方形的大小并除以可用的像素数才能获得精确大小的网格。

我已经完成了所有这些工作,但是在添加必要的表格元素来创建网格时遇到了问题。我的代码处于完全工作状态,除非我在 for 循环中使用该函数时,它只会附加一个子项,而它应该附加到几百个子项。appendChild()

我的代码:

grid.show = function(event)
{
    var e = event;

    if(grid.show_grid == false)
    {
        grid.show_grid = true;

        parent.document.getElementById("grid_table").style.display = "block";

        // Get grid (table) and create some elements.
        grid.get_grid = parent.document.getElementById("grid_table");
        grid.tr       = parent.document.createElement("tr");
        grid.td       = parent.document.createElement("td");

        // Clear the grid of all squares so we don't have to worry about subtracting anything.
        grid.get_grid.innerHTML = "";

        // Calculate the number of horizontal and vertical squares.
        var horizontal = Math.ceil(grid.total_width / grid.size);
        var vertical   = Math.ceil(grid.total_height / grid.size);

        // This was a nested loop, removed for demonstration.
        // Attempting to add 10 "<tr><td></td></tr>" to the table.
        for(var j = 0; j < 10; j++)
        {
            grid.tr.appendChild(grid.td);
        }

        //console.log(grid.tr);

        // Add the elements to the table.
        grid.get_grid.appendChild(grid.tr);

    }
    else
    {   
        grid.show_grid = false;
        parent.document.getElementById("grid_table").style.display = "none";
    }
}

这只会返回一个包含单个表数据的表行,如下所示:

<tr>
    <td></td>
</tr>

我已经看过这个页面和这个页面,它们听起来很有前途,但我就是不知道如何让它发挥作用。

编辑:代码现在工作,解决方案:

grid.show = function(event)
{
    var e = event;

    if(grid.show_grid == false)
    {
        grid.show_grid = true;

        parent.document.getElementById("grid_table").style.display = "block";

        grid.get_grid           = parent.document.getElementById("grid_table");
        grid.tr                 = null;
        grid.td                 = null;
        grid.get_grid.innerHTML = "";

        var horizontal = Math.ceil(grid.total_width / grid.size);
        var vertical   = Math.ceil(grid.total_height / grid.size);

        for(var i = 0; i < vertical; i++)
        {
            grid.tr = parent.document.createElement("tr");

            for(var j = 0; j < horizontal; j++)
            {
                grid.td = parent.document.createElement("td");

                grid.td.width = grid.size;
                grid.td.height = grid.size;

                grid.tr.appendChild(grid.td);
            }
            grid.get_grid.appendChild(grid.tr);
        }
    }
    else
    {   
        grid.show_grid = false;
        parent.document.getElementById("grid_table").style.display = "none";
    }
}
JavaScript 附录子

评论


答:

31赞 TheZ 10/4/2012 #1

您一遍又一遍地附加相同的元素。每次希望拥有新元素时,都需要调用。document.createElement

编辑:如果元素设置非常复杂并且可能包含子元素,那么您也可以使用 Node.cloneNode

评论

2赞 user1649326 10/5/2012
谢谢,这就是问题所在。代码现在按预期工作(OP 中的解决方案)。
5赞 Mutahhir 10/4/2012 #2
for(var j = 0; j < 10; j++)
{
        grid.tr.appendChild(grid.td);
}

应该是

for(var j = 0; j < 10; j++) {
    var newTd = parent.document.createElement('td');
    grid.tds.push(newTd); // if you need it, not sure why though
    grid.tr.appendChild(newTd);
}
15赞 epascarello 10/4/2012 #3

如果要创建元素的一个副本,则需要克隆它。使用 cloneNode()

所以改变

grid.tr.appendChild(grid.td);

grid.tr.appendChild(grid.td.cloneNode(true));