Всегда ли следует объявлять переменные?

Question

[Сергей]

Какая распространённая практика при работе с различными элементами в Javascript? Следует ли каждый раз сохранять значения элементов в переменных, даже если они используются один раз? Приведу пример.

document.getElementById('myElement').addEventListener('click', myFunction);

vs

var myElement = document.getElementById('myElement');
myElement.addEventListener('click', myFunction);

Следует ли мне выбирать первый вариант, если нигде более в коде я не обращаюсь к данному элементу? Или второй вариант оптимальнее? За счёт чего?

Answer 1

[Владимир Проскурин]

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

Comments

[Сергей]

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

[Виталий]

Сергей, можно вообще не использовать переменные, зачем читабельность и понятность кода? откуда вообще такие сложные мысли, что это надо обсуждать на тостере?

[Владимир Проскурин]

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

[Сергей]

Владимир Проскурин, спасибо за ответ. Виталий, я обеими руками за за чистый и понятный код (хоть первый вариант вполне приемлемый с точки зрения читабельности), но тут вопрос больше о производительности.

[Виталий]

Сергей, для полноты понятности ответа может быть ситуация некрасивая

let selector = document.querySelector('blablabla');
if(selector){
selector.click();
} else {
selector.click() // будет выброшено исключение ибо null.click()
}

Answer 2

[0xD34F]

Да делайте как вам удобнее.

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

Comments

[Сергей]

Насчёт стрёмного селектора - хороший аргумент :)

Answer 3

[Alexander Pushkarev]

Если один раз, то совсем не обязательно выносить в переменные