var函数=[];//让我们创建3个函数对于(变量i=0;i<3;i++){//并将其存储在funcs中funcs[i]=函数(){//每个都应该记录其值。console.log(“我的值:”,i);};}对于(变量j=0;j<3;j++){//现在让我们逐一查看函数[j]();}

它输出以下内容:

我的价值:3我的价值:3我的价值:3

而我希望它输出:

我的值:0我的价值:1我的价值:2


当使用事件侦听器导致函数运行延迟时,也会出现同样的问题:

var buttons=document.getElementsByTagName(“button”);//让我们创建3个函数for(var i=0;i<buttons.length;i++){//作为事件侦听器buttons[i].addEventListener(“单击”,函数(){//每个都应该记录其值。console.log(“我的值:”,i);});}<按钮>0</按钮><br/><button>1</button><br/><按钮>2</按钮>

…或异步代码,例如使用Promises:

//一些异步等待函数const wait=(ms)=>new Promise((解析,拒绝)=>setTimeout(解析,ms));对于(变量i=0;i<3;i++){//一旦每个承诺得到解决,就记录“i”。等待(i*100)。然后(()=>console.log(i));}

在for in和for of循环中也很明显:

常量arr=[1,2,3];常量fns=[];用于(arr中的变量i){fns.push(()=>console.log(“index:”,i));}对于(arr的var v){fns.push(()=>console.log(“值:”,v));}for(arr常量){var obj={number:n};//或新的MyLibObject({…})fns.push(()=>console.log(“n:”,n,“|”,“obj:”,JSON.stringify(obj)));}for(fns的var f){f();}

这个基本问题的解决方案是什么?


当前回答

让我们利用新功能。因此,i停止成为闭包的变量,并成为文本的一部分:

var funcs = [];
for (var i = 0; i < 3; i++) {
    var functionBody = 'console.log("My value: ' + i + '");';
    funcs[i] = new Function(functionBody);
}

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

其他回答

这是异步代码中经常遇到的问题,变量i是可变的,在调用函数时,使用i的代码将被执行,并且i将变为其最后一个值,因此意味着在循环中创建的所有函数都将创建一个闭包,i将等于3(for循环的上限+1)。

解决此问题的一个方法是创建一个函数,该函数将为每次迭代保存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。

为什么不在创建第一个(也是唯一的)循环之后,简单地调用它们,例如:

 var funcs = [];
    for (var i = 0; i < 3; i++) {
    // let's create 3 functions
    funcs[i] = function() {
    // and store them in funcs
    console.log("My value: " + i); // each should log its value.
    };
    funcs[i]();// and now let's run each one to see
    }

聚会的时间有点晚了,但我今天正在探讨这个问题,并注意到许多答案并没有完全解决Javascript如何处理作用域的问题,这本质上就是问题的根源。

所以正如许多其他人提到的,问题是内部函数引用的是同一个i变量。那么为什么我们不在每次迭代时都创建一个新的局部变量,并让内部函数引用它呢?

//覆盖console.log(),以便可以看到控制台输出console.log=函数(消息){document.body.innerHTML+='<p>'+msg+'</p>';};var funcs={};对于(变量i=0;i<3;i++){var ilocal=i//创建新的局部变量funcs[i]=函数(){console.log(“我的值:”+ilocal)//每个都应该引用自己的局部变量};}对于(变量j=0;j<3;j++){函数[j]();}

就像之前一样,每个内部函数输出分配给i的最后一个值,现在每个内部函数只输出分配给ilocal的最后一值。但每个迭代不应该有自己的ilocal吗?

事实证明,这就是问题所在。每个迭代都共享相同的范围,因此第一次迭代之后的每个迭代都只是覆盖ilocal。来自MDN:

重要提示:JavaScript没有块范围。随块引入的变量的作用域是包含函数或脚本,设置这些变量的效果将持续到块本身之外。换句话说,块语句不引入范围。尽管“独立”块是有效的语法,但您不希望在JavaScript中使用独立块,因为如果您认为它们在C或Java中做类似的事情,它们不会做您认为它们做的事情。

再次强调:

JavaScript没有块范围。随块引入的变量的作用域是包含函数或脚本

通过在每次迭代中声明ilocal之前检查ilocal,我们可以看到这一点:

//覆盖console.log(),以便可以看到控制台输出console.log=函数(消息){document.body.innerHTML+='<p>'+msg+'</p>';};var funcs={};对于(变量i=0;i<3;i++){console.log(ilocal);var ilocal=i;}

这正是这个bug如此棘手的原因。即使您正在重新定义变量,Javascript也不会抛出错误,JSLint甚至不会抛出警告。这也是为什么解决这一问题的最佳方法是利用闭包,这本质上就是在Javascript中,内部函数可以访问外部变量,因为内部作用域“包围”外部作用域。

这也意味着内部函数“抓住”外部变量并保持它们的活力,即使外部函数返回。为了利用这一点,我们创建并调用一个包装函数来创建一个新的作用域,在新作用域中声明ilocal,并返回一个使用ilocal的内部函数(下面有更多解释):

//覆盖console.log(),以便可以看到控制台输出console.log=函数(消息){document.body.innerHTML+='<p>'+msg+'</p>';};var funcs={};对于(变量i=0;i<3;i++){funcs[i]=(function(){//使用包装函数创建新范围var ilocal=i//将i捕获到本地var中return function(){//返回内部函数console.log(“我的值:”+ilocal);};})(); //记住运行包装器函数}对于(变量j=0;j<3;j++){函数[j]();}

在包装器函数内创建内部函数为内部函数提供了一个只有它才能访问的私有环境,即“闭包”。因此,每次调用包装器函数时,我们都会创建一个新的内部函数,并使用它自己的独立环境,确保ilocal变量不会相互冲突和覆盖。一些小的优化给出了许多其他SO用户给出的最终答案:

//覆盖console.log(),以便可以看到控制台输出console.log=函数(消息){document.body.innerHTML+='<p>'+msg+'</p>';};var funcs={};对于(变量i=0;i<3;i++){funcs[i]=包装器(i);}对于(变量j=0;j<3;j++){函数[j]();}//为内部函数创建单独的环境函数包装器(ilocal){return function(){//返回内部函数console.log(“我的值:”+ilocal);};}

使现代化

随着ES6成为主流,我们现在可以使用新的let关键字来创建块范围的变量:

//覆盖console.log(),以便可以看到控制台输出console.log=函数(消息){document.body.innerHTML+='<p>'+msg+'</p>';};var funcs={};for(设i=0;i<3;i++){//使用“let”声明“i”funcs[i]=函数(){console.log(“我的值:”+i)//每个都应该引用自己的局部变量};}对于(var j=0;j<3;j++){//我们可以在这里使用“var”函数[j]();}

看看现在有多容易!有关更多信息,请参阅此答案,我的信息基于此。

只需将var关键字更改为let。

var是函数范围的。

let是块范围的。

当您开始编写代码时,for循环将迭代并将i的值赋值为3,在整个代码中该值将保持为3。我建议您阅读更多关于节点中作用域的信息(let、var、const和其他)

funcs = [];
for (let i = 0; i < 3; i++) {      // let's create 3 functions
  funcs[i] =async function() {          // and store them in funcs
    await console.log("My value: " + i); // each should log its value.
  };
}
for (var j = 0; j < 3; j++) {
  funcs[j]();                      // and now let's run each one to see
}