2011-09-13 125 views
0
if(properties != undefined) 
{ 
    foreach(key in properties) 
    { 
     dialogProperty.key = property[key]; 
    } 
    alert(dialogProperty.close); 
} 

如何获取/修复上述代码?我认为上面的代码是自我解释的。在JavaScript中循环播放对象

回答

4

我想你的意思是for而不是foreach。你也应该停止key是全球和使用Object.prototype.hasOwnProperty

if(properties != undefined) 
{ 
    for (var key in properties) 
    { 
     if (properties.hasOwnProperty(key) { 
      dialogProperty[key] = properties[key]; // fixed this variable name too 
     } 
    } 
    alert(dialogProperty.close); 
} 

NB股份有限公司Kobi's fix了。

+0

优秀点':)' – Kobi

+0

或使用'Object.keys(属性).forEach' – Raynos

+1

@Raynos如果您认为ECMAScript 5,是的。我不是100%相信OP是在这个阶段...... – lonesomeday

3

假设你要复制的所有属性,你可能在寻找:

dialogProperty[key] = property[key]; 

dialogProperty.key不是动态的,它设置每次key财产,以同样的方式dialogProperty["key"]会。

2
properties && Object.keys(properties).forEach(function(key) { 
    dialogProperty[key] = properties[key]; 
}); 
console.log(dialogProperty.close); 

properties &&检查是为了确保属性不是虚假的。

Object.keys调用返回properties对象具有的所有键的数组。

.forEach为数组中的每个元素运行函数。

dialogProperty[key] = properties[key] set将dialogProperty的值设置为属性的值。

+1

有趣的是,浏览器在其优化中可能有多不同。 http://jsperf.com/object-keys-vs-hasownproperty +1这种方法,但我只是澄清说'Object.keys'只返回可枚举的键,这当然会在这里。 – user113716