Что является причиной JSLint говоря есть «слишком много вара заявления»

Question

JSLint (с onevar флага включен) ослабевают некоторый яваскрипта кода, который я имею в следующем:

Problem at line 5 character 15: Too many var statements.

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

Я просмотрел документы JSLint для var keyword, но в нем конкретно не говорится о том, почему несколько операторов var в одной и той же функции являются плохими.

Простейший пример. Объясните, как код будет пользоваться только имея 1 уаг заявление:

function Test(arg) { 
    var x = arg + 1, 
     y = cache.GetItem('xyz'); 
    if (y !== null) { 
     // This is what would cause the warning in JSLint 
     var request = ajaxPost(/* Parameters here */); 

    } 
} 

Answer 1

Javascript не имеет блокировки. На других языках с ним (например, c), если вы объявляете переменную в выражении if, вы не можете получить к ней доступ за ее пределами, но в javascript вы можете. Автор JSLint считает, что это плохая практика, так как вы (или другие читатели) можете запутаться и подумать, что больше не можете обращаться к переменной, но на самом деле можете. Поэтому вы должны объявить все свои переменные в верхней части функции.

Answer 2

Просто угадать здесь, но это может быть время для functional decomposition. Функции должны делать один вещь и делать это хорошо.

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

Answer 3

Если для параметра «onevar» установлено значение «Истина», если допускается только один оператор var для каждой функции.

if (funct['(onevar)'] && option.onevar) { 
    warning("Too many var statements."); 
} 

Answer 4

Идея состоит в том, что вы должны использовать объект вместо отдельных варов. Так, где у вас есть:

var x = arg + 1, 
    y = cache.GetItem('xyz'); 

Изменить это:

var dimensions = {}; 
dimensions.x = arg + 1; 
dimensons.y = cache.GetItem('xyz'); 
dimensions.request = ... 

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

Answer 5

Просто объявлять ВАР в одном месте, как это:

var request,x,y; 

Answer 6

Официальная причина is here, Дуглас Crockford.

процитировать:

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

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

JSLint ожидает блоки с функцией, если, переключатель, while, for, do и пробовал заявления и нигде больше.

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

Answer 7

Обоснование уже описано.

Рекомендация использовать эту форму:

var myVar1 = document.getElementById("myDiv1"), 
    myVar2 = document.getElementById("myDiv2"); 

или это:

var myVar1, myVar2; 
myVar1 = document.getElementById("myDiv1"); 
myVar2 = document.getElementById("myDiv2"); 

Но это не выглядит очень красиво, особенно если вы хотите, чтобы документ ВАР.

Таким образом, вы можете просто отключить это предупреждение temporarly:

/*jslint vars: true*/ 
    /** 
    * @returns {HTMLDivElement} 
    */ 
    var myVar1 = document.getElementById("myDiv1"); 
    /** 
    * @returns {HTMLDivElement} 
    */ 
    var myVar2 = document.getElementById("myDiv2"); 
    /*jslint vars: false*/ 

Внимание: убедитесь, что это будет сделано в верхней части функции.

Я думаю, что это сделано, потому что jslint не может надежно определить, были ли объявлены vars в верхней части функции или нет.