5

我很困惑为什么当我尝试在 for 循环中附加多个元素时只插入最后一个元素。

我创建了一个JsFiddle 来展示我无法让它工作。我希望插入 100 个锚标记，但只插入最后一个元素。

为了后面的缘故，这里是相关的 JavaScript，TODO 标记了相关部分：

Math.randomNumber = function(max) {
    return Math.round(Math.random() * max % max);
}


var Door = {
    $el: $('<a>', {
        class: 'door selectable'
    }),
    number: null,
    isSelected: false,
    containsZonk: true,
    bind: function () {
        var that = this;
        this.$el.on('click tap', function () {
            that.isSelected = true
        });
    }
}

var Platform = {
    $el: null,
    doorCount: null,
    jackpotNumber: null,
    doors: [],
    init: function($el, doorCount) {
        this.$el = $el;
        this.doorCount = doorCount;

        for (var i = 0; i <= doorCount - 1; i++) {
            var door = Object.create(Door);
            door.number = i;
            door.$el.html(i.toString());
            this.doors.push(door);

            /* TODO: wtf why is only last one inserted? */
            this.$el.append(door.$el);
        }

        this.jackpotNumber = Math.randomNumber(doorCount);
        this.doors[this.jackpotNumber].containsZonk = false;
    }
}
    $(document).ready(function(){
        var platform = Object.create(Platform);
        var $game = $('.door_game');
        platform.init($game, 100);
    });

我想将所有 100 个元素插入div.door_game：

<body>
    <h1>Zonk!</h1>
        <div class="door_game" data-doors="10">
        </div>
</body>
4

2 回答 2

6

这是因为你所有的门都共享同一个$el。

http://jsfiddle.net/U9swZ/9/

var Door = {
    number: null,
    isSelected: false,
    containsZonk: true,
    bind: function () {
        var that = this;
        this.$el.on('click tap', function () {
            that.isSelected = true
        });
    },
    init: function () {
        this.$el = $('<a>', {
            class: 'door selectable'
        });
        return this;
    }
}
于 2013-04-14T14:05:28.917 回答
2

Door $el 被创建一次，这就是为什么它被追加并再次追加（即一次又一次地移动），因此您只能看到最后一个。@plalx 是对的。

此外，由于您附加了数百个门，因此非常重要，var frag = document.createDocumentFragment()在循环之前使用 documentFragment ( ) 来收集您的门 ( frag.appendChild(...)) 并在循环 ( ) 之后将此 documentFragment 附加到 Platform $el this.$el.appendChild(frag)。对于如此数量的元素，性能增益非常重要。

而且，不要犹豫，克隆一个元素而不是重建它。它要快得多。您甚至可以使用 DOM APIcloneNode(false)方法对原始元素进行非深度复制。

于 2013-04-14T14:20:41.087 回答