Почему обработчик событий возвращает одно и тоже значение?

Question

[Artur Aralin]

Всем привет! Есть у меня примерно такой код

cDay = angular.element('<span>' + day.getDate().toString() + '</span>').bind('click', function() {
            this.week = weeksCounter;
            console.log(this.week)
            return false;
});
element.append(cDay)

Тут весь код: pastebin.com/UNwLBR4v

В результате получается календарь, только по клике на какой-либо из дней, он выдает номер последней недели, а не той в которой он должен находиться. При этом, когда я вывожу console.log(this) devTools показывает на нужный элемент.

Answer 1

[IceJOKER]

Если что-то делается в цикле, а потом результат получаете только последнего элемента, то в 99% случаев из-за перезаписи .
Не хочу вникаться в код, но рекомендую объявить функцию к примеру function handler(){} - и в этой функции делать обработку и эту функцию передать вторым аргументом bind.

Comments

[Artur Aralin]

Вы почти были правы! Только сделать нужно было handler maker! Но все равно спасибо)

[IceJOKER]

Artur Aralin: you're welcome )

[gleendo]

For Web:)

Answer 2

[Виктор]

Если я правильно понял, Вы столкнулись с замыканиями.
Тема важная и лучше про неё почитать, а если коротко, то при срабатывании обработчика значение weeksCounter берётся не то, которое было на момент создания обработчика, а то, которое оно имеет на момент срабатывания обработчика, то есть последнее значение. Поэтому для всех обработчиков выводится одно и то же.
Обычно делают функцию, в которую передают текущее значение и там устанавливают обработчик, тем самым фиксируя значение, которое нужно для его работы.

Comments

[Artur Aralin]

Я знаю, что это замыкания) Проблема была в том, что его не правильно создал)

[Виктор]

Ну раз Вы с ними знакомы, то непонятно, к чему задан вопрос)

[Artur Aralin]

Виктор: сглупил несколько