JavaScript：JSLint 错误“for in 的主体应包含在 if 语句中，以从原型中过滤掉不需要的属性”

Question

我正在使用 JSLint 工具来确保我的 JavaScript 是“严格的”。

我收到以下错误，但不明白如何修复它：

The body of a for in should be wrapped in an if statement to filter unwanted properties from the prototype

对于以下代码：

for (var i in keypairs) {
    ...
}

任何人都知道如何修复此错误，因为它是 JavaScript“严格”并且不会被 JSLint 标记

Answer 1

如果 keypairs 是一个数组，那么您应该真正迭代这些元素，例如：

for(var i = 0; i < keypairs.length; i++) {
  ...
}

如果 keypairs 是一个散列，那么 JSLint 正确地建议您检查您是否正在操作适当的密钥类型（即，确认哈希是预期的类型），

因此类似

for(var i in keypairs) {
  if(keypairs.hasOwnProperty(i)) {
    ...
  }
}

if 验证任何标准的情况可确保您不会访问原型函数等。

Answer 2

它希望您使用hasOwnProperty。

for (var i in keypairs) {
    if(keypairs.hasOwnProperty(i))
    {
        // Use i
    }
}

与 JSLint 的大部分内容一样，这是一个建议，其适用性取决于您的情况。如果对象的原型中有不需要的可枚举属性，那么它很有用。例如，如果您使用某些 JavaScript 库，则可能会出现这种情况。

Answer 3

for...in 的问题是您还将遍历原型的属性，而大多数时候这不是您想要的。这就是为什么您应该使用 hasOwnProperty 测试该属性：

for (var i in keypairs) {
    if(keypairs.hasOwnProperty(i) {
        //...
    }
}

Answer 4

for (var i in keypairs) {
   if (keypairs.hasOwnProperty(i)) {
       ...
   }
}

这是因为 for/in 循环可能会迭代由第 3 方库扩展的某些方法，例如，如果存在

Object.prototype.clone = function() { ... }

没有 .hasOwnProperty() 条件下，.clone 方法也会在 ... 中迭代。

http://yuiblog.com/blog/2006 进一步解释了这一点/09/26/for-in-intrigue/，从 JSLint 页面本身链接。

您可以通过选中“容忍未过滤的内容”来关闭此警告。

Answer 5

看一下 jslint 自己的文档：
http://www.jslint.com/lint.html
跳到该部分

，

他们执行以下操作：
for (对象中的名称) { if (object.hasOwnProperty(name)) { .... } }