Ответ уже дали, и тут очевидно, что переменная animation
определена локально в функции, поэтому после повтороного попадиния в функцию - значение теряется по причине того, что она локальна и видна только на протяжении работы функции и в момент запуска функции всегда создается заново. Я все же дам ответ, содержащий исправленный код:
var animation;
document.onkeydown = function(event) {
if (event.key === 'ArrowDown') {
var count = 0;
animation = setInterval(function() {
count++;
console.log(count);
}, 1000);
} else if (event.key === 'ArrowUp') {
clearInterval(animation);
}
};
А также, ссылки для изучения:
Функция setInterval
после своего выполнения возвращает intervalID
, остановить функцию setInterval
можно при помощи функции clearInterval
. В функцию clearInterval
передается идентификатор действия, которе нужно остановить. При передаче некорректного intervalID
в метод clearInterval
- не инициируется ошибка, поэтому код продолжает исправно работать без каких-либо ошибок в консоли.
animation
. (Переменнаяcount
попадает в замыкание функции прописанной внутри setInterval и потому остаётся существовать)