函数调用作为循环 ES6 中的可迭代

Question

寻找最佳实践建议。

将for of中的函数调用作为可迭代对象是一种好习惯吗？

const obj = {key1:'a', key2:'b'};

for(const i of Object.keys(obj)){ ... }

最好有

const obj = {key1:'a', key2:'b'};
const allKeys = Object.keys(obj);
for(const i of allKeys){ ... }

不是在循环的每次迭代中调用 Object.keys 函数吗？

Answer 1

你可以通过模拟一个类似的案例来检查它

function A(){
    console.log('function called');
    return [1, 2, 3, 4, 5];
}

for(let i of A()){
    console.log(i);
}

如你所见，A() 只被调用一次

Answer 2

您的第一个示例与第二个示例执行相同的操作，只是没有allKeys 常量。在您的第一个示例中：

const obj = {key1:'a', key2:'b'};

for(const i of Object.keys(obj)){ ... 

...Object.keys 只会被调用一次，然后它的迭代器将被检索并用于循环。 （这在规范中的ForIn/OfHeadEvaluation 中有介绍，但它...很繁重。）所以你可以使用这种形式，它并不是低效的。

Answer 3

对Object.keys 的调用在两种变体中只调用一次。

但也可以考虑这个不依赖于Object.keys 的for 循环：

for (const prop in obj) {  } 

Object.keys 创建一个数组，而上面的for 循环没有。后者仅在必要时检索项目。当然，如果您的 for 循环从未过早退出，它仍会检索所有这些，因此只有在您计划过早退出循环时才会有所不同。在这种情况下，相应的迭代器永远不会访问剩余的项目，而Object.keys 在创建数组时已经这样做了。

需要注意的一点：for .. in 循环还会迭代可枚举的inherited 属性，而不仅仅是拥有 属性，而Object.keys 只会列出拥有可枚举的属性。您可以在for 循环中添加额外的测试以排除非自己的属性：

if (obj.hasOwnProperty(prop))