Question

我正在使用JQuery动态创建一个列表，并希望将每个元素绑定到＆＃34; tap＆＃34;事件。我该怎么做？以下是我尝试做的一个示例，但它不起作用。

for(var i=1; i <= 5; i++) {
        var id = "item" + i;
        var li = $("<li data-theme=\"d\" id=\"" + id + "\">Item " + i + "</li>");
        li.appendTo(ul);

        $(document).delegate("#"+id, "tap", function() {
            $("#"+id).attr({ "data-theme" : "e", "class" : "ui-li ui-li-static ui-btn-up-e" });
        });
    }

此代码在点击任何元素时触发，但它总是因某种原因修改列表中的最后一个元素。

Answer 1

您的选择

将事件处理移至循环外部

for(var i=1; i <= 5; i++) {
 ....
}
$(document).delegate("[id^=item]", "tap", function() {
});

使用bind方法并将tap事件应用于元素，而不应用于document。

for(var i=1; i <= 5; i++) {
  //append li to ul
  $("#"+id).bind("tap", function() {
    $(this).attr({ 
          "data-theme" : "e", 
          "class" : "ui-li ui-li-static ui-btn-up-e"   
    });
  });
}

但是，最好的办法是将event置于循环之外，并将事件绑定到ul，然后将其委托给li。
```
for(var i=1; i <= 5; i++) {
 ....
}
$("ul").delegate("[id^=item]", "tap", function() {
});
```

注意

如果您想更改主题，还需要更新一次布局。

$("ul").delegate("[id^=item]", "tap", function() { $(this).attr({ "data-theme" : "e", "class" : "ui-li ui-li-static ui-btn-up-e" }).parent("ul").listview().listview("refresh"); });

使用STARTS WITH SELECTOR

你已经把它放在你的代码中了：

var id = "item" + i;

这意味着对于5个元素的整个循环，你的id将如下所示：

<li id="item1">.. <li id="item2">.. <li id="item3">.. <li id="item4">..

看看这里常见的事情，我会说：

item

因此，由于您的ID以item开头，因此您可以使用starts with selector对其进行概括。所以，

id^=item

表示您正在搜索ID为item的元素。由于它是一个属性，

[id^=item]

更加模块化的方法

此方法涉及较少的HTML：

//your ul tag var ul = $("ul") //set up an array for adding li to it. var li = []; //a temporary element to store "li" var $li; for(var i=1; i <= 5; i++) { //add required attributes to the element var $li = $("<li/>", { "data-theme" : "d", "id" : "item" + i, "html" : "Item " + i }); //push that into array li.push($li); } //wait for append to finish ul.append(li).promise().done(function () { //wait for list to be added - thats why you wait for done() event in promise() //add the click events to this - event delegation - this way your click event is added only once $(this).on("tap", "[id^=item]", function () { $(this).attr({ "data-theme" : "e", "class" : "ui-li ui-li-static ui-btn-up-e" }).parent("ul").listview().listview("refresh"); }); //then refresh $(this).listview().listview("refresh"); });

这是一个演示：http://jsfiddle.net/hungerpain/TdHXL/

希望这有帮助！

Answer 2

只需将事件处理程序移动到for循环外部。

并替换

$(document).delegate("#"+id, "tap", function() {

带

$(document).delegate("[id*=item], "tap", function() {

<强> JS

for (var i = 1; i <= 5; i++) {
    var id = "item" + i;
    var li = $("<li data-theme=\"d\" id=\"" + id + "\">Item " + i + "</li>");
    li.appendTo(ul);
}

$(document).delegate("[id*=item]", "tap", function () {
    $("#" + id).attr({
        "data-theme": "e",
            "class": "ui-li ui-li-static ui-btn-up-e"
    });
});

Answer 3

这是我要做的：

var items = []; // an actual JavaScript array - a list of items.

for(var i=1;i<=5;i++){
    items.push({theme:'d'}); //add an item to our list, 
                             //Anything that actually relates to the item should be
                             //here, text, other data and such. This is our 'view model'.
}

items.forEach(function(item,i){ // for each item, bind to the dom
    var el = document.createElement("li"); //create an element
    el.textContent = i+1; // set its text

    el.onclick = function(e){ // or ontap or whatever
        el.className = "ui-li ui-li-static ui-btn-up-e";
        item.theme = "d";
    }//you mind want to use addEventListener instead at a later point

    item.el = el;
    ul.appendChild(el); // you need access to ul, currently being a jQuery object ul[0].
});

请注意，我们可以直接从我们的代码访问这些项目，我们可以直接更新它们，并直接引用它们。我们不需要查询我们自己的数据 - 我们拥有它并直接了解如何获取它。

另外 - 我们没有80kb的依赖。没有复杂的选择器，没有'魔术'绑定。一切都是直截了当的，它只是简单的'javascript。

注意：对于旧浏览器，forEach应该（很容易）为其填充。

将事件绑定到列表元素

3 个答案: