Как исправить ошибку jslint «Не выполнять функции в цикле»?

У Дугласа Крокфорда есть новый идиоматический способ достижения вышеизложенного - его старая техника заключалась в использовании внутренней функции для связывания переменных, но в новой технике используется создатель функций. См. слайд 74 к его выступлению "Function the Ultimate". [Этот слайд-шоу больше не существует]

Для ленивых вот код:

function make_handler(div_id) {
    return function () {
        alert(div_id);
    };
}
for (i ...) {
    div_id = divs[i].id;
    divs[i].onclick = make_handler(div_id);
}

(Я просто наткнулся на этот вопрос через много месяцев после того, как он был опубликован...)

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

Если функция на самом деле является одной и той же функцией, присваиваемой разным значениям в итерации (или объектам, созданным в итерации), то вместо этого вам нужно присвоить функцию именованной переменной и использовать этот единственный экземпляр функции в присваивании внутри петля:

handler = function (div_id) {
    return function() { alert(div_id); }
}

for (i ...) {
    div_id = divs[i].id;
    divs[i].onclick = handler(div_id);
}

Больше комментариев/обсуждений по этому поводу было сделано другими умнее меня, когда я задал аналогичный вопрос здесь, в Stack Overflow: Ошибка JSlint "Не создавайте функции внутри цикла". приводит к вопросу о самом Javascript

Что касается JSLint: да, это догматично и идиоматично. Тем не менее, обычно это «правильно» - я обнаружил, что многие люди, которые негативно отзываются о JSLint, на самом деле не понимают (тонкостей) Javascript, которых много и они тупые.

Буквально обойти проблему, выполнив следующие действия:

1. Создайте файл .jshintrc

2. Добавьте следующую строку в файл .jshintrc

   {"loopfunc" : true, // tolerate functions being defined in loops }

JSLint — это всего лишь руководство, вам не всегда нужно придерживаться правил. Дело в том, что вы не создаете функции в цикле в том смысле, о котором идет речь. Вы создаете свои классы только один раз в своем приложении, а не снова и снова.

Если вы используете JQuery, вы можете сделать что-то подобное в цикле:

for (var i = 0; i < 100; i++) {
  $("#button").click(function() {
    alert(i);
  });
}

Чтобы удовлетворить JSLint, один из способов обойти это (в JQuery 1.4.3+) — использовать дополнительный аргумент данных обработчика для .click():

function new_function(e) {
  var data = e.data; // from handler
  alert(data); // do whatever
}

for (var i = 0; i < 100; i++) {
  $("#button").click(i, new_function);
}

Просто переместите:

(function (name, func) {...})()

блок из цикла и назначьте его переменной, например:

var makeFn = function(name, func){...};

Затем в цикле есть:

prototype[prop] = makeFn(...)