Почему не работает setInterval?

Question

[Ivan-P]

Есть такой код. Почему-то фон li меняется не с задержкой в 1с, а мгновенно.

var ul = document.getElementById("blink");
var li = ul.getElementsByTagName('li');

for(var i = 0; i < li.length; i++) {
setInterval(blink(), 1000);
}

function blink() {
li[i].classList.toggle('cls');
}

Answer 1

[KorsaR-ZN]

А потому-что Вы функцию blink вызываете в цикле (вообщем нужно без скобок), и вообще такая конструкция работать правильно не будет.

Сделайте так:

for (var i = 0; i < li.length; i++) {
   setInterval(blink, 1000); 
}

UPD
Чтоб правильно работало, скажите что вы хотите добится? и зачем Вам тут цикл.
Просто из-за того, что функция выполняется с задержкой, вы всегда будете иметь n = li.length - 1
Да и к тому-же blink будет вызван больше раз, чем вы задумали :)

Comments

[Ivan-P]

Конечная цель - сделать бегущим красный фон элементов списка. Но вот на начальном этапе все застопорилось. Всем пунктам мгновенно присваивается класс cls задающий красный фон, а не с задержкой в 1 секунду.

[KorsaR-ZN]

Ivan-P: Т.е зажигать каждый LI с интервалом с 1 сек? или все сразу через 1 сек.

[Ivan-P]

Если убираю скобки возле blink - ругается TypeError: li[i] is undefined

[Ivan-P]

KorsaR-ZN, менять фон по очереди. Сначала первый красный, потом через секунду - второй, и так зациклить навсегда.

[KorsaR-ZN]

Ivan-P: Вот то, что Вам нужно jsfiddle.net/d8ev5wyt

[Ivan-P]

Вау, спасибо.

[Ivan-P]

KorsaR-ZN, немного подпилил ваш вариант и... jsfiddle.net/IvanP/k3fna7uc

[KorsaR-ZN]

Ivan-P: У Вас там с ошибкой, при старте, т.к элемент -1 не существует.
Вот сделайте лучше так... jsfiddle.net/k3fna7uc/3

[Artem627]

А если мне надо передавать параметр в эту функцию, то как писать?

Answer 2

[Mick Coder]

Неправильно записан первый аргумент функции setInterval() ! Как правильно

Answer 3

[GreatRash]

Логичней не создавать кучу интервалов, а менять класс у родителя, т.е. у ul. А уж в зависимости от того какой класс у родителя дети будут мигать. Таким образом вам понадобится всего один интервал, а не куча на каждого потомка.

Answer 4

[Дамир Рысаев]

Выложил код на jsfiddle.net

var ul = document.getElementById('blink'),
    li = ul.getElementsByTagName('li');

for(var i = 0; i < li.length; i++) {
	(function(el, i) {
        setInterval(function() {
            blink(el, i) 
        }, 1000);
    })(li, i);

}

function blink(el, i) {
	el[i].classList.toggle('cls');
}

1. У вас не правильно написан setInterval(). Функция blink() сразу вызывается походу исполнения цикла.
2. Чтобы понять код прочитайте про Замыкания и ссылки в руководстве JavaScript Гарден