Question

我无法理解这里的错误。我从数据库中获取了一个事件列表，我正在尝试使用回调onclick方法创建一个自定义列表来显示它们。我得到正确的值，我正确显示它们但是onClick方法获取最后一个项目ID的值，而不是它自己的值。

因此，如果我点击列表中的任何项目，我的提醒就会显示

show itinerary for event: 10

而不是正确的ID（我现在有10个事件）

这就是我处理db中的项目的方法：

function gotEvents(transaction, result) {
             var eventList;
             var myDbElements = $('#myDbElements');            
             var params = {
                items:result.rows.length,
                hasImage:false,
                content:[]
             }
                   if (result != null && result.rows != null) {

                       for (var i = 0; i < result.rows.length; i++) {
                          var row = result.rows.item(i);
                          var rowText = row["Name"] + '<br/>' +  row["Start"].slice(0,9);                    


                          var rowId = row["id"];
                          var contentItems = {
                            text: rowText,
                            onClick: function(){ showEventItinerary(rowId) }

                          }
                           params.content.push(contentItems);                      
                          }
                          for (var j=0; j<result.rows.length;j++){
                            PrecAppPersistence.debugObject(params.content[i]);
                          }
                          eventList = PrecAppComponents.myList(params,$('#eventsList'));
                   }
                PrecApp.I_SCROLL.refresh();   
               }


            function showEventItinerary(event_ID) {
                alert("show itinerary for event:"+event_ID);
            }

以下是我创建列表的方法：

function List() {

var self;
var items;
var firstRun = true;

this.initialize = function(params,cont) {
    self = this;
    var container = cont;

    var items;
    container.empty();
    opt = {
            items:0,
            content:[], // array with the content. Default is = {text:text,icon:imageURL,onClick: function(){}}  itemId // not sure if we need this yet. supposed to be a manual ID for the item. EX: id taken from the db.                                         
            width: 300,
            hasImage: false, //set true if you want to show an images
            margin:10, //margin between items               


    }
    for (i in params) opt[i] = params[i];

    items = opt.items;
    container.width(opt.width);

    for (i=0;i<items;i++){

        itemToAdd = '<div class="listItem" id="listItem'+i+'">'; //add IDS to add click                        
        if (opt.hasImage) {
            itemToAdd += "<img src=" + opt.content[i].image + " />";
        }               
        itemToAdd += "<p>" + opt.content[i].text + "</p>";
        itemToAdd += "<div class='clearFix'></div>";
        itemToAdd += '</div>';      
        container.append(itemToAdd); //original #scroller might change?     

        listItem = $('#listItem'+i);    
        listItem.css('margin',opt.margin + 'px 0');
        listItem.on('click',opt.content[i].onClick);            

    }


}}

我正在记录我的变量，我的rowId具有正确的ID。 onClick日志显示所有项目的以下内容：

onClick:function (){ showEventItinerary(rowId )}（我不确定这是否正确，是否应该显示rowID的值而不是变量rowID？）

我在这里错过了什么吗？

由于

Answer 1

你需要创建一个闭包，因为你的索引我用循环更新了：

for (var i = 0; i < result.rows.length; i++) {(function(index){
// use index instead of i inside your loop
...
// end closure
})(i);
// end loop
}

将使用index = i执行自调用函数。我是循环的一部分，它的值会改变，而循环中的每个函数都会保留自己的索引。

Answer 2

rowID没有块级范围使用数组来存储循环中的每个项目，或使用closure为每次迭代创建单独的范围。

检查the infamous loop problem

JavaScript中的意外参数值

2 个答案: