2013-02-11 99 views
3

我对下面的代码很困惑:对象函数循环覆盖javascript?

var x =[ {name : 'name1' , value : 15 },{name :'name2' , value: 60} ]; 
var y = [[1,2,3] , [4,5,6]] ; 
    for(var t in y){ 
     x[t].myFun = function(){console.log(y[t])} ; 
     } 
console.log(x[0].myFun()) ; 

不应该将此代码返回y第一阵列为什么它返回第二个阵列?

这里是jsFiddle

+0

只是单挑,for-in循环用于迭代对象中的成员。用于(var i = 0; i kimpettersen 2013-02-11 21:06:49

+1

实际的问题在于'console.log(y [t])',因为函数中't'的值总是等于它最后的值(即1) – Mahn 2013-02-11 21:09:27

回答

6

myFun函数的所有引用相同t(和y)变量。所以循环后,t1,所以它总是返回第二个值。

你需要使用闭包“关闭”周围的值(也,你不应该使用for..in数组):

var x = [{name : 'name1' , value : 15 }, {name :'name2' , value: 60}]; 
var y = [[1,2,3] , [4,5,6]]; 
for(var t = 0, len = y.length; t < len; t++){ 
    (function(t){ 
     x[t].myFun = function(){console.log(y[t])}; 
    })(t); 
} 

console.log(x[0].myFun()); 
+0

这是正确的答案。 – 2013-02-11 21:09:05

5

由于您使用JQuery,你有一个简单的方法来迭代数组无需担心在创建闭包时专门捕获索引的当前值。它是$.each

var x =[ {name : 'name1' , value : 15 },{name :'name2' , value: 60} ]; 
var y = [[1,2,3] , [4,5,6]] ; 
$.each(y, function(i,v) 
{ 
    x[i].myFun = function(){console.log(y[i])} ; 
}); 
+2

哎呀,你甚至可以用'v'代替'y [i]'。 – 2013-02-11 21:12:01

+1

@RocketHazmat对! 'v'将等于'y [i]'。 – Plynx 2013-02-11 21:13:31