Не работает clearInterval(). В чём проблема?

Question

[inscamp]

Есть функция, генерирующая случайный цвет, и функция, устанавливающая его в качестве backgroundColor для дива.

При первом клике див будет менять свой цвет (за счёт использования setInterval). При втором клике по нему же он должен перестать менять цвет (соответсвенно, за счёт clearInterval).

var ind = false;

item.onclick = function(){
    if (!ind) {
	  var cycle = setInterval(changeBG, 100);
      ind = true;
	} else {
      clearInterval(cycle);
      ind = false;
	}
};

В представленном коде имеется какая-то явная ошибка в вызове clearInterval, поскольку див не реагирует на повторный клик.
В чём же проблема?

Answer 1

[rmaksim]

var ind = false;
var cycle

item.onclick = function(){
    if (!ind) {
        cycle = setInterval(changeBG, 100);
        ind = true;
    } else {
        clearInterval(cycle);
        ind = false;
    }
};

а можно обойтись и без второй переменной
т.к. setInterval возвращает Id таймера, можно его и проверять и потом соотв. сбрасывать

var timer

item.onclick = function(){
    if (!timer) {
        timer = setInterval(changeBG, 100);
    } else {
        timer = clearInterval(timer);
    }
};

Comments

[inscamp]

Спасибо.

[Сергей Мельников]

inscamp: не забывайте отмечать ответ пользователя как решение

[StraighF]

1) clearInterval() не имеет возвращаемого значения.
2) Параметр, который передаётся clearInterval(*id*) - не обнуляется. Т.е. как хранил id, который вернул setInterval(), так и хранит.