var или нет var в цикле for-in JavaScript?

Используйте var, это уменьшает область действия переменной, в противном случае переменная ищет ближайшее замыкание в поисках оператора var. Если он не может найти var, то он глобальный (если вы находитесь в строгом режиме, using strict, глобальные переменные выдают ошибку). Это может привести к следующим проблемам.

function f (){
    for (i=0; i<5; i++);
}
var i = 2;
f ();
alert (i); //i == 5. i should be 2

Если вы напишете var i в цикле for, предупреждение покажет 2.

Обзор и подъем JavaScript

Первая версия:

for (var x in set) {
    ...
}

объявляет локальную переменную с именем x. Вторая версия:

for (x in set) {
    ...
}

не.

Если x уже является локальной переменной (т.е. у вас есть var x; или var x = ...; где-то раньше в вашей текущей области (т.е. текущей функции)), то они будут эквивалентны. Если x еще не является локальной переменной, то использование второй неявно объявит глобальную переменную x. Рассмотрим этот код:

var obj1 = {hey: 10, there: 15};
var obj2 = {heli: 99, copter: 10};
function loop1() {
    for (x in obj1) alert(x);
}
function loop2() {
    for (x in obj2) {
        loop1(); 
        alert(x);
    }
}
loop2();

вы могли бы ожидать, что это предупредит hey, there, heli, hey, there, copter, но поскольку x одно и то же, оно будет предупреждать hey, there, there, hey, there, there. Вы не хотите этого! Используйте var x в своих for петлях.

В довершение ко всему: если цикл for находится в глобальной области видимости (т.е. не в функции), то локальная область видимости (область видимости x объявляется, если вы используете var x) совпадает с глобальной областью видимости (область x неявно объявлен в, если вы используете x без var), поэтому две версии будут идентичными.

Вы действительно должны объявлять локальные переменные с помощью var, всегда.

Вы также не должны использовать циклы "for... in", если вы не абсолютно уверены, что хотите сделать именно это. Для перебора реальных массивов (что довольно часто) всегда следует использовать цикл с числовым индексом:

for (var i = 0; i < array.length; ++i) {
  var element = array[i];
  // ...
}

Перебор простого массива с помощью for... in может привести к неожиданным последствиям, потому что ваш цикл может выбрать атрибуты массива помимо численно проиндексированных.

редактировать здесь, в 2015 году, также можно использовать .forEach() для перебора массива:

array.forEach(function(arrayElement, index, array) {
  // first parameter is an element of the array
  // second parameter is the index of the element in the array
  // third parameter is the array itself
  ...
});

Метод .forEach() присутствует в прототипе Array, начиная с IE9.

На самом деле, если вам не нравится объявление в заголовке for, вы можете сделать:

var x;
for (x in set) {
    ...
}

Как упоминалось в других ответах на этот вопрос, полное отсутствие использования var приводит к ненужным побочным эффектам, таким как назначение глобального свойства.

Используйте тот, где вы объявляете переменную цикла с помощью var. Неявно объявленные переменные имеют другую область видимости, которая, вероятно, не соответствует вашим намерениям.

for(var i = 0; ...)

это обычно наблюдаемый шаблон, но он отличается от

for(int i; ...)

в C++ тем, что переменная не привязана к блоку for. Фактически, var поднимается на вершину объемлющей области (функции), поэтому локальная i будет эффективно доступна как до цикла for (после начала текущей области/функции), так и после него.

Другими словами, делая:

(function(){ //beginning of your current scope;
 //...
 for(var i in obj) { ... };
})();

такой же как:

(function(){ //beginning of your current scope;
 var i;
 //...
 for(i in obj) { ... };
})();

В ES6 есть ключевое слово let (вместо var), чтобы ограничить область действия блоком for.

Конечно, вы ДОЛЖНЫ использовать локальные переменные (объявленные либо с var, либо с let, либо с const (в ES6)) вместо неявных глобальных переменных.

for(i=0; ...) или for(i in ...) потерпит неудачу, если вы используете "use strict"; (как и следует) и i не объявлено.

Использование var — самый чистый способ, но оба работают так, как описано здесь: https://developer.mozilla.org/en/JavaScript/Reference/Statements/for...in

По сути, используя var, вы гарантируете, что создадите новую переменную. В противном случае вы можете случайно использовать ранее определенную переменную.

Я думаю, что var хорош по соображениям производительности.

Javascript не будет просматривать всю глобальную область видимости, чтобы увидеть, существует ли где-то еще x.

С общей точки зрения, первая версия будет для индекса, который должен находиться в пределах области действия цикла, а другая будет любой переменной в области, в которой был вызван конструктор цикла.

Если вы собираетесь использовать индекс цикла внутри цикла for, и это не потребуется другим в следующих строках, лучше объявите переменную с помощью «var», чтобы вы были уверены, что «x» — это индекс цикла, инициализированный 0, в то время как другой, если в этом контексте доступна другая переменная "x", она будет перезаписана индексом цикла - у вас будут логические ошибки -.

Я всегда использую область видимости блока let, представленную в ES2015.

for (let x in set) {
    ...
}

Дополнительная литература и примеры