3

我有小问题。我正在使用一家公司的 API。我请求活动，他们返回给我一系列活动。这是代码的一部分。

client.requestActivities(function(activities) {
        if (activities.length > 0) {
            for(i=0; i < activities.length; i++) {
                var activity = activities[i];
                activity.onStart(function() { alert(i+ " started"); });
                activity.onCredit(function() { alert(i+ " credit"); });
                activity.onClose(function() { alert(i+ " close"); });
                activity.onFinish(function() { alert(i+ " finish"); });
                            $('.Game-Screen-Footer').append('<div class="Game-Screen-Footer-Adv" id="foota'+i+'"><a href="javascript:;" ><img src="'+activity.image_url+'" alt="'+activity.display_text+'" width="190" height="110"></a></div>');
                document.getElementById('foota'+i+'').onclick = function() {
                ARNO.box.show({html:'<div id="socialVibeFancyBox0"></div>', close: false, width:activity.window_width, height:activity.window_height, openjs: 
                            function(){
                                client.loadActivityIntoContainer(activity, 'socialVibeFancyBox0');
                            }
                        });
            }
        }

}

大多数情况下，此函数返回包含 3 个元素的数组。但是当我点击第一个活动时，onStart 它应该提醒我（0 开始），但它会提醒（4 开始），这意味着它会提醒 i 的最后一个元素。我应该如何解决这个问题？我尝试了很多东西，但我没有找到解决方案，也许你可以帮助我？

4

3 回答 3

5

这是关闭问题的又一个案例。尝试这个：

for( i=0; i<activities.length; i++) {
    (function(i) {
        // your code here
    })(i);
}
于 2012-10-03T23:27:41.780 回答
0

将for循环的内容移到一个函数中，并传入需要单独限定范围的值。

for(i=0; i < activities.length; i++) {
    setUpActivity(i, activities[i]);
}

function setUpActivity(i, activity) {

    activity.onStart(function() { alert(i+ " started"); });
    activity.onCredit(function() { alert(i+ " credit"); });
    activity.onClose(function() { alert(i+ " close"); });
    activity.onFinish(function() { alert(i+ " finish"); });
    ARNO.box.show({
        html:'<div id="socialVibeFancyBox0"></div>', 
        close: false, width:activity.window_width, 
        height:activity.window_height, 
        openjs:function(){
                    client.loadActivityIntoContainer(activity, 'socialVibeFancyBox0');
                }
            });
}

函数调用会创建一个新的变量范围。在函数中创建的任何变量或参数都将在它们自己的范围内，并且在同一范围内创建的函数将永久引用这些变量。

于 2012-10-03T23:30:31.700 回答
0

如果你不关心 IE8- 那么 ES5 已经解决了这个问题forEach：

activities.forEach(function(v, i){
  // now 'i' will work anywhere within this scope
});
于 2012-10-03T23:32:45.657 回答