正确的 for/in 变量声明

Question

在 for/in 循环中声明循环特定变量的正确语法是什么？

前两个似乎都可以工作（并且不会在 Google Closure 编译器中引发任何标志），但只有第三个通过了 Crockford 的 JS Lint。我不愿意使用它，主要是因为它不紧凑。

JSLint 抱怨 val 是一个坏变量（当我不添加 var 时），或者声明应该被移动。

第一个或第二个选项有什么缺点吗？我应该使用什么？ （示例假设 str 是声明的字符串，vals 是声明的对象）

1. 无声明：

for(val in vals)
{
    if(vals.hasOwnProperty(val))
    {
        str += val;
    }
}

2. 在 'for' var 声明中：

for(var val in vals)
{
    if(vals.hasOwnProperty(val))
    {
        str += val;
    }
}

3. 在循环之外 var 声明：

var val;
for(val in vals)
{
    if(vals.hasOwnProperty(val))
    {
        str += val;
    }
}

Answer 1

请随意忽略 JSLint。这比什么都重要。第二个和第三个在功能上是相同的，随意使用其中一个（我使用第二个）。第一个暴露了全局“val”变量，所以不要这样做:)

更难发现/查找。

var a, b, c, d, e;

仅供参考，第三个背后的原因是内联变量声明比在函数顶部

Answer 2

除了变量声明背后的 cwolves 推理之外，JSLint 还考虑 变量提升。

话虽如此，在编写 for 循环时我更喜欢选项#2。不过，我确实在函数的顶部定义了其余的变量。所以我喜欢：

function foo(a) {
  var b, c = {d: "e", f: "g"};

  for (var i = 0, j = a.length; i < j; i += 1) {
    ...
  }

  for (var h in c) {
    if (c.hasOwnProperty(h)) {
      ...
    }
  }      
}

Answer 3

1. 选项

val 变量成为隐式全局变量。
隐式全局变量 100% 不好，应该避免。

2. 和 3. 选项

val 变量成为包含范围的变量。如果您的代码位于函数内部，则 val 变量将成为该函数的局部变量。两个选项是等效的。 （我更喜欢 2. 选项。）

JSLint 会抛出异常，因为 Crockford 的想法是始终在代码顶部声明所有变量。这对于较大的代码（例如 > 100 行）更有意义 - 在这种情况下，您可以考虑将声明移出 for 和 for-in 语句，并将它们放在代码的顶部。这样您就可以很好地了解所有局部变量。