Помилка JSLint: Перемістіть усі декларації 'var' у верх функції

Сайт JSLint оновлено, і я більше не можу перевіряти сценарії JS. Для мене це попередження не є критичним, і я не хочу проходити через тисячі рядків, щоб це виправити, я хочу знайти більше критичних проблем.

Хтось знає, як вимкнути цю помилку або використовувати застарілий JSLint?

ОНОВЛЕННЯ

Приклад:

function doSomethingWithNodes(nodes){
  this.doSomething();

  for (var i = 0; i < nodes.length; ++i){
    this.doSomethingElse(nodes[i]);
  }

  doSomething(); // want to find this problem
}

Вивід jslint.com:

Error:
Problem at line 4 character 8: Move all 'var' declarations to the top of the function.

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

Problem at line 4 character 8: Stopping, unable to continue. (44% scanned).

Проблема:

Наявність змінних поверх функцій - нова вимога. Я не можу використовувати JSLINT для тестування коду, оскільки він зупиняє сканування сценарію щодо цієї помилки.

У мене багато коду, і я не хочу загрожувати цьому попередженню критичною помилкою.

ОНОВЛЕННЯ 22.08.2011: знайдено http://jshint.com , він виглядає набагато краще, ніж http://jslint.com/

Оновлення, червень 2017 р .: За умови підтримки (наприклад, якщо ви не використовуєте JavaScript в Internet Explorer 10 або старішій версії), вам слід розглянути можливість використання let замість var .

Наприклад: for(let i=0; ...; i++)

Я ніяк не збираюся ставити var i;з a for(var i=0; ...; i++)у верхній частині своїх функцій. Особливо, коли Специфікація JavaScript має його як прийнятний синтаксис у forрозділі (12.6). Крім того, це синтаксис, який Брендан Ейх використовує у своїх прикладах.

Ідея переміщення декларації вгору полягає в тому, що вона повинна точніше відображати те, що відбувається під капотом, однак, робити це буде лише відображати, а не впливати.

Для мене це смішне очікування forповторень. Тим більше, що JSLint припиняє обробку, коли виявляє це.

Чи є змінні, оголошені у верхній частині функції, більш читабельними, дискусійно. Я особисто вважаю за краще, щоб змінні-ітератори оголошувались при їх використанні. Мені байдуже, чи змінна вже створена всередині, я ініціалізую її тут, щоб я був у безпеці.

Я б стверджував, що оголошення змінної ітератора там, де вони використовуються, гарантує, що вони випадково не стануть глобальними (якщо ви перемістите цикл в іншу функцію, змінна ітератора рухається разом з нею). Це набагато зручніше, ніж необхідність підтримувати оголошення змінних у верхній частині функцій.

На даний момент я використовую http://www.javascriptlint.com/online_lint.php, оскільки, схоже, це зосереджено на важливих речах.

Компілятор Google Closure насправді не зможе правильно визначити тип змінної циклу циклу for ... in, якщо він не оголошений як for (var i in ...), і жодна анотація, здається, не виправляє цього, тому декларацію не можна переміщати до верху.

Ви можете будь-коли завантажити застарілі версії або змінити останню версію . Це не так складно, насправді (шукати move_var). Потім запустіть jslint локально, або за допомогою вузла, або за допомогою браузера з простою формою HTML - можливо, ви захочете скопіювати оригінал Крокфорда.

Зауважте, що попередження було введено як частина основного перезапису , і воно з’являється лише після for(, тому повідомлення трохи вводить в оману.

Зверніть увагу, що переміщення всіх vars до початку відрізняється від "дозволити один оператор var на функцію". Вимога перемістити всі змінні вгору є новою і, схоже, не має перемикача. Більше на http://groups.google.com/group/jsmentors/browse_thread/thread/5e90c25230f8e22/70e1a95a20fb829e

У мене була ця проблема в моїй кодовій базі, коли ми хотіли перейти на останню версію JSLINT. У нас їх було багато, і люди були не раді перенести декларацію. Ми фактично виявили, що найелегантнішим рішенням було використання underscore.js і замість повного багатослівного циклу використовувати функцію _.each (), яка видалила помилку JSLint і зробила наш код більш функціональним, чистим, жорсткішим та простішим для читати.

Незважаючи на те, що нова бета-версія JSLint не документує директиву про коментарі щодо множинних varдопусків у межах функції, вона , схоже, підтримує директиви з початкової версії.

Оригінальний JSLint дозволив вам зробити це:

/*jslint vars: true */

З мого досвіду це все ще працює - я думаю, для зворотної сумісності. Час написання - червень 2015 року.

Я виявив, що наступний синтаксис видалить помилку:

function doSomethingWithNodes(nodes) {
    this.doSomething();
    var i; // HERE is where you move the 'var' to the top of the function
    for (i = 0; i < nodes.length; ++i) {
        this.doSomethingElse(nodes[i]);
    }

    doSomething(); // want to find this problem
}