инструмент качества кода JSLint

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

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

Лично я в настоящее время предпочитаю использовать JSHint, так как Crockfords Lint, на мой взгляд, меньше связан с правильным кодом, а больше — с Crockfords. мнение о том, как должен выглядеть JavaScript, к черту мнение всех остальных.

JSLint рекомендует заключать операторы if и for в фигурные скобки, потому что это может привести к трудно отслеживаемым ошибкам, если вы не будете осторожны с отступами. Например:

if (somecondition)
    step1();
    step2();

Судя по отступам, шаги step1() и step2() будут выполняться только в том случае, если какое-то условие истинно, но на самом деле step2() будет выполняться всегда, поскольку фигурные скобки отсутствуют.

Если вас действительно беспокоит размер файла, я бы посоветовал изучить JS-компрессор, такой как yuicompressor. Большинство из них удалит фигурные скобки вокруг операторов if/for, которые в них не нуждаются. Тогда вы сможете получить лучшее из обоих миров: удобочитаемый код и небольшой размер файла при его обслуживании.

Что касается второй проблемы, JSlint рекомендует ее, потому что переменные javascript всегда имеют область действия, а не область действия блока. Делая это:

function func(){
  var i;
  alert('foo');
  for(i = 0; i<3; i++){
     alert(i);
  }
  alert(i);
}

точно (100% точно!!) такой же, как:

function func(){
  alert('foo');
  for(var i = 0; i<3; i++){
     alert(i);
  }
  alert(i);
}

Таким образом, i не предназначен для (как и в других языках). Вот почему jslint рекомендует помещать все переменные поверх функций, чтобы вы не запутались, думая, что есть переменные с блочной областью видимости.

Уважаю первый вопрос, согласен с вами. Я не вижу в этом большой проблемы. Но JSLint видит в этом проблему, потому что, если вы позже добавите строки в свой оператор if и забудете поставить фигурные скобки, у вас будет логическая ошибка, как указал Джефф.

До:

if (x > 10)
    alert("it's obvious x is positive");

После некоторых изменений в коде:

if (x > 10)
    alert("it's obvious x is positive");  
    alert("x is greater than 10");  //Logic error

Проблема, которая не возникла бы при использовании фигурных скобок.

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

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

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

В любом случае, просто игнорируйте ошибки, которые вам не нужны.

Первое, фигурные скобки в операторах if, отражает убеждение автора JSLint, что отсутствие фигурных скобок приводит к проблемам с обслуживанием (вера, которую я разделяю, но это ни здесь, ни там).

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

Вы спросите: "Почему это считается ошибкой?" Это гораздо больше, чем плохая практика; они определенно вредоносны. Это потому, что они поощряют более поздние ошибки, которые будет почти невозможно найти. Такое же предсмертное опущение фигурных скобок разрешено и в C, и в C++, и там это такая же ошибка.

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