我在这个简单的Closure上做错了什么

kev*_*ler 3 javascript closures javascript-events

它可能是睡眠剥夺,但我不明白为什么这不起作用.我希望onclick从创建元素的for循环返回i的值并应用事件处理程序.把它放在一个闭包中,它仍然是转动迭代器的最大数量.

window.onload = function(){
  var arbitrary_amount = 100;
  var the_list = document.getElementsByTagName('ul')[0];

  for(var i = 0; i < arbitrary_amount; i++){
    var natural_index = i + 1;
    var list_item = document.createElement('li');
    var inner_link = document.createElement('a');
    inner_link.setAttribute('href', '#');
    inner_link.innerHTML = "Link "+natural_index;

    inner_link.onclick = function(){
      return function(link_num){
        alert('You clicked link '+link_num);
      }(i);
    };

    list_item.appendChild(inner_link);
    the_list.appendChild(list_item);
  }


};
Run Code Online (Sandbox Code Playgroud)

spa*_*tel 8

你正在以错误的方式使用闭包......我不能给你一个关于实际发生的事情的大师类型的答案,但这是一个工作(没有测试)闭包:

inner_link.onclick = (function(link_num){
   return function(){
       alert('You clicked link '+link_num);
   };
})(i);
Run Code Online (Sandbox Code Playgroud)

  • 大师类型的答案是:`i`被一个闭包捕获,然后被传递给事件处理函数内的自调用函数.这不是我们想要的.相反,`i`应该通过值传递给自调用函数,该函数返回一个事件处理函数,它将`link_num`捕获为闭包.嗯..我应该考虑把它放在答案而不是这个评论中吗? (4认同)