这描述了在JavaScript中使用闭包的常见错误。

函数定义新环境

考虑：

function makeCounter()
{
  var obj = {counter: 0};
  return {
    inc: function(){obj.counter ++;},
    get: function(){return obj.counter;}
  };
}

counter1 = makeCounter();
counter2 = makeCounter();

counter1.inc();

alert(counter1.get()); // returns 1
alert(counter2.get()); // returns 0

每次调用makeCounter时，｛counter:0｝都会创建一个新对象。另外，obj的新副本也被创建以引用新对象。因此，计数器1和计数器2彼此独立。

回路中的闭合

在循环中使用闭包很棘手。

考虑：

var counters = [];

function makeCounters(num)
{
  for (var i = 0; i < num; i++)
  {
    var obj = {counter: 0};
    counters[i] = {
      inc: function(){obj.counter++;},
      get: function(){return obj.counter;}
    }; 
  }
}

makeCounters(2);

counters[0].inc();

alert(counters[0].get()); // returns 1
alert(counters[1].get()); // returns 1

请注意，计数器[0]和计数器[1]不是独立的。事实上，它们对同一个对象进行操作！

这是因为在循环的所有迭代中只有一个obj副本共享，可能是出于性能原因。即使｛counter:0｝在每次迭代中都创建了一个新对象，obj的同一副本也只会用对最新对象的引用。

解决方案是使用另一个助手函数：

function makeHelper(obj)
{
  return {
    inc: function(){obj.counter++;},
    get: function(){return obj.counter;}
  }; 
}

function makeCounters(num)
{
  for (var i = 0; i < num; i++)
  {
    var obj = {counter: 0};
    counters[i] = makeHelper(obj);
  }
}

这是因为直接分配了函数范围中的局部变量以及函数参数变量新副本。

另一种说法是，函数中的i在执行函数时绑定，而不是在创建函数时绑定。

当您创建闭包时，i是对外部范围中定义的变量的引用，而不是创建闭包时的副本。将在执行时进行评估。

大多数其他答案提供了通过创建另一个不会改变值的变量来解决问题的方法。

我只是想给你解释清楚。就我个人而言，为了解决这个问题，我会选择Harto的方法，因为从这里的答案来看，这是最不言自明的方法。发布的任何代码都可以使用，但我宁愿选择闭包工厂，而不必写一堆注释来解释为什么我要声明一个新变量（Freddy和1800）或具有奇怪的嵌入闭包语法（apphacker）。

最简单的解决方案是，

而不是使用：

var funcs = [];
for(var i =0; i<3; i++){
    funcs[i] = function(){
        alert(i);
    }
}

for(var j =0; j<3; j++){
    funcs[j]();
}

警报“2”3次。这是因为在for循环中创建的匿名函数共享相同的闭包，并且在该闭包中，i的值是相同的。使用此选项可防止共享关闭：

var funcs = [];
for(var new_i =0; new_i<3; new_i++){
    (function(i){
        funcs[i] = function(){
            alert(i);
        }
    })(new_i);
}

for(var j =0; j<3; j++){
    funcs[j]();
}

这背后的想法是，用IIFE（立即调用函数表达式）封装for循环的整个主体，并将new_i作为参数传递，并将其捕获为i。由于匿名函数是立即执行的，因此匿名函数内部定义的每个函数的i值都不同。

此解决方案似乎适合任何此类问题，因为它需要对受此问题困扰的原始代码进行最小的更改。事实上，这是故意的，根本不应该是一个问题！

JavaScript函数在声明时“关闭”它们有权访问的作用域，并保留对该作用域的访问权，即使该作用域中的变量发生变化。

var函数=[]对于（变量i=0；i<3；i+=1）{funcs[i]=函数（）{控制台日志（i）}}对于（var k=0；k<3；k+=1）{函数[k]（）}

上面数组中的每个函数都在全局范围（全局，因为这恰好是它们声明的范围）上关闭。

稍后调用这些函数，记录全局范围中i的最新值。这就是结束的魔力和挫折。

“JavaScript函数在声明它们的作用域上关闭，并保留对该作用域的访问权，即使该作用域内的变量值发生变化。”

使用let而不是var可以解决这个问题，方法是在每次for循环运行时创建一个新的作用域，为每个要关闭的函数创建一个单独的作用域。其他各种技术对额外的功能也有同样的作用。

var函数=[]for（设i=0；i<3；i+=1）{funcs[i]=函数（）{控制台日志（i）}}对于（var k=0；k<3；k+=1）{函数[k]（）}

（让变量成为块范围。块用大括号表示，但在for循环的情况下，初始化变量i在我们的情况下被认为是在大括号中声明的。）

假设您不使用ES6；您可以使用IIFE：

var funcs = [];
for (var i = 0; i < 13; i++) {      
    funcs[i] = (function(x) {
      console.log("My value: " + i)
     })(i);
   }

但情况会有所不同。

我很惊讶还没有人建议使用forEach函数来更好地避免（重新）使用局部变量。事实上，出于这个原因，我不再使用for（var I…）。

[0,2,3].forEach(function(i){ console.log('My value:', i); });
// My value: 0
// My value: 2
// My value: 3

//编辑为使用forEach而不是map。