Как передать значение (а не ссылку) переменной JS в функцию?

Вот упрощенная версия чего-то, что я пытаюсь запустить:

for (var i = 0; i < results.length; i++) {
    marker = results[i];
    google.maps.event.addListener(marker, 'click', function() { 
        change_selection(i);
    }); 
}

но я обнаружил, что каждый слушатель использует значение results.length (значение, когда цикл for завершается). Как добавить слушателей так, чтобы каждый из них использовал значение i в момент добавления, а не ссылку на i?

Источник

6 ответов

Решение

В современных браузерах вы можете использовать let или же const ключевые слова для создания блочной переменной:

for (let i = 0; i < results.length; i++) {
  let marker = results[i];
  google.maps.event.addListener(marker, 'click', () => change_selection(i));
}

В старых браузерах вам нужно создать отдельную область видимости, которая сохраняет переменную в ее текущем состоянии, передавая ее в качестве параметра функции:

for (var i = 0; i < results.length; i++) {
  (function (i) {
    marker = results[i];
    google.maps.event.addListener(marker, 'click', function() { 
      change_selection(i);
    }); 
  })(i);
}

Создавая анонимную функцию и вызывая ее с переменной в качестве первого аргумента, вы передаете функцию по значению и создаете замыкание.

Источник

Как и закрытия, вы можете использовать function.bind:

google.maps.event.addListener(marker, 'click', change_selection.bind(null, i));

передает значение i в качестве аргумента функции при вызове. (null для привязки this, который вам не нужен в этом случае.)

function.bind был представлен платформой Prototype и был стандартизирован в пятом издании ECMAScript. Пока все браузеры не поддерживают его изначально, вы можете добавлять свои собственные function.bind поддержка использования замыканий:

if (!('bind' in Function.prototype)) {
    Function.prototype.bind= function(owner) {
        var that= this;
        var args= Array.prototype.slice.call(arguments, 1);
        return function() {
            return that.apply(owner,
                args.length===0? arguments : arguments.length===0? args :
                args.concat(Array.prototype.slice.call(arguments, 0))
            );
        };
    };
}
Источник

Закрытие:

for (var i = 0, l= results.length; i < l; i++) {
    marker = results[i];
    (function(index){
        google.maps.event.addListener(marker, 'click', function() { 
            change_selection(index);
        }); 
    })(i);
}

РЕДАКТИРОВАТЬ, 2013: Теперь они обычно называют IIFE

Источник

Вы заканчиваете с закрытием. Вот статья о замыканиях и о том, как с ними работать. Проверьте Пример 5 на странице; это сценарий, с которым вы имеете дело.

РЕДАКТИРОВАТЬ: Четыре года спустя, эта ссылка мертва. Корень проблемы выше в том, что for петли образуют замыкания (особенно на marker = results[i]). Как marker передается в addEventListener вы видите побочный эффект замыкания: общая "среда" обновляется с каждой итерацией цикла, прежде чем она будет окончательно "сохранена" с помощью замыкания после последней итерации. MDN объясняет это очень хорошо.

Источник 
for (var i = 0; i < results.length; i++) {
    marker = results[i];
    google.maps.event.addListener(marker, 'click', (function(i) {
        return function(){
            change_selection(i);
        }
    })(i)); 
}
Источник

Я думаю, что мы можем определить временную переменную для хранения значения i.

for (var i = 0; i < results.length; i++) {
 var marker = results[i];
 var j = i;
 google.maps.event.addListener(marker, 'click', function() { 
   change_selection(j);
 }); 
}

Я не проверял это все же.

Источник
Другие вопросы по тегам