Прав ли JSlint и почему?

Question

[Иван]

Решил проверить свой проект с помощью jslint (до этого не пользовался линтерами).
Скачал addin jslint в VScode. Все настройки по умолчанию.

Имею такой код (упрощенно):

if(true){
var newItem = 10;
}
else{
var  newItem = 20;
}

линтер говорит что надо делать так:

var newItem;
if(true){
newItem = 10;
}
else{
newItem = 20;
}

Поясните пожалуйста почему второй вариант считается правильным?

Comments

[SagePtr]

Потому что дважды объявлять переменную - глупость.

[Иван]

SagePtr, вроде понял. Объявление переменной всегда обрабатывается до выполнения кода...

Answer 1

[Антон Швец]

Замени на let и посмотри, работает ли твоя конструкция.
А вообще по хорошему переменные надо объявлять в начале функции.
А еще почитай про поднятие var. Два var попросту бессмысленны.

Comments

[Иван]

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

[Антон Швец]

Иван, Где используются будешь объявлять const, потому что по другому не получится.

[Иван]

Объявление переменной всегда обрабатывается до выполнения кода - вот разумный ответ. спасибо.

[Антон Швец]

Иван, да, это и есть поднятие или хоистинг.