javascript - JavaScript 中的意外参数值

Question

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

因此，如果我单击列表中的任何项目，我的警报会显示

显示事件行程:10

而不是正确的 ID(我现在有 10 个事件)

这是我处理从数据库中获取的项目的方式:

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

您需要创建一个闭包，因为您的索引 i 会随着循环更新:

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 执行。 i 是循环的一部分，它的值会改变，而循环中的每个函数都会保留自己的索引。