Надо ли бояться глобальной переменной в JS?

Question

[semki096]

Ситуация. Мне надо по клику клонировать див с id. Каждому новому клону автоматом прибавляю единицу.

var a = 0;
$("body").on('click', '#add', function() {
                            a = a + 1;
                           $("#m").clone().attr('id', 'm_' + a)
});

Но как видите здесь a обьявлена глобальной переменной. Правильно ли это, если учесть что некошерно вроде как обьявлять глобальные переменные.

Answer 1

[Андрей]

Ну сделайте так хотябы:

(function(){
var a = 0;
$("body").on('click', '#add', function() {
                            a = a + 1;
                           $("#m").clone().attr('id', 'm_' + a)
});
})()

и никто не узнает :)

Comments

[semki096]

я пока тока вникаю. можете подсказать ещё один момент . У меня один файл со всеми функциями. Начинается он с $(document).ready(function() и в нем находятся все функции. Как в таком случае обычно поступают, у меня тогда получится русская матрёшка ))

[semki096]

и ещё если не сложно - впервые столкнулся с такой концовкой "})()" не подскажете что это означает?

[Антон]

semki096:

var func = function() { }  // создание функции
func(); // вызов функции
(function() { }) (); // создание и одновременный вызов

[Андрей]

$(document).ready(function() прекрасно изолирует все переменные. Матрешка, это да, получится. Если уже задумываетесь над этим, попробуйте применить backbonejs для придания этому аду структуры. На сегодняшний день не самое лучшее решение на мой взгляд, но сам подход важно понять, потом можно и свое написать и другое поискать, сформировав требования.

Это "шаблон немедленно вызываемой функции". Функция отрабатывает сразу же, ее не надо специально вызывать.
getinstance.info/articles/javascript/immediately-i...

[semki096]

спасибо