for(var i in fav){

(function(){
                var p=i;
                var obj=$S.getId(fav[i]);
                $S.addHandler($S.getId(fav[i]),'click',function(){
                        $S.jsonp({url:'http://www97.xcar.com.cn/bbs/api/api.php?action=fav_forum&fid='+p,callback:function(data){
                                  if(data.error==0){
                                     //成功收藏
                                     $S.getId('S_right').style.display = "block";
                                     setTimeout("$S.getId('S_right').style.display ='none';",3000);
                                      obj.className += " "+'S_active';
                                 }else if(dta.error==1){
                                     //未登陆
                                 }else if(data.error==3){
                                     //超过20条
                                     $S.getId('S_wrong').style.display = "block";
                                     setTimeout("$S.getId('S_wrong').style.display ='none';",3000);
                                }
                        }});
                });
            })();

}

 

 

 

 

使用闭包和立即执行的方法来保存状态

分类： JavaScript 2014-11-07 10:33 102人阅读 评论(0) 收藏 举报
// 这个代码是错误的，因为变量i从来就没背locked住
// 相反，当循环执行以后，我们在点击的时候i才获得数值
// 因为这个时候i操真正获得值
// 所以说无论点击那个连接，最终显示的都是I am link #10（如果有10个a元素的话）


var elems = document.getElementsByTagName('a');
for (var i = 0; i < elems.length; i++) {
    elems[i].addEventListener('click', function (e) {
        e.preventDefault();
        alert('I am link #' + i);
    }, 'false');
}

// 这个是可以用的，因为他在自执行函数表达式闭包内部
// i的值作为locked的索引存在，在循环执行结束以后，尽管最后i的值变成了a元素总数（例如10）
// 但闭包内部的lockedInIndex值是没有改变，因为他已经执行完毕了
// 所以当点击连接的时候，结果是正确的


var elems = document.getElementsByTagName('a');
for (var i = 0; i < elems.length; i++) {
    (function (lockedInIndex) {
        elems[i].addEventListener('click', function (e) {
            e.preventDefault();
            alert('I am link #' + lockedInIndex);
        }, 'false');
    })(i);
}


// 你也可以像下面这样应用，在处理函数那里使用自执行函数表达式
// 而不是在addEventListener外部
// 但是相对来说，上面的代码更具可读性


var elems = document.getElementsByTagName('a');
for (var i = 0; i < elems.length; i++) {
    elems[i].addEventListener('click', (function (lockedInIndex) {
        return function (e) {
            e.preventDefault();
            alert('I am link #' + lockedInIndex);
        };
    })(i), 'false');

}

//也可以用闭包来保存状态

var elems = document.getElementsByTagName('a');
function keep(i){
 var keep = i;
 return function(){
   alert(keep);
 }
}
for (var i = 0; i < elems.length; i++) {
   var index
    elems[i].addEventListener('click', keep(i), 'false');
}


不管那种方法，关键是看最后方法执行时是调用的哪个变量，第一种直接调用i，i是对外面可见的，所以在for后会变化，而另外三种都是调用的非i变量，而且这个非i变量在for中是不可见的，无法改变！！