intervals

[iperez_genius]

I am trying to display 4 divs one after eachother using setInterval

function switchIcon(current, next)
{

	document.getElementById(next).innerHTML = "<div class=\"on\"></div>";
		document.getElementById(current).innerHTML = "<div class=\"off\"></div>";

	   
	       
}

function timedMsg()
{
	var IntervalId1 = 0;
		var IntervalId2 = 1;
			var IntervalId3 = 2;
		var IntervalId4 = 3;


IntervalId1 =setInterval(switchIcon("stepImage1", "stepImage2"),2000);
    clearInterval ( IntervalId1);

IntervalId2 =setInterval(switchIcon("stepImage2", "stepImage3"),2000);
    clearInterval ( IntervalId2);

IntervalId3 =setInterval(switchIcon("stepImage3", "stepImage4"),2000);
    clearInterval ( IntervalId3);

IntervalId4 =setInterval(switchIcon("stepImage4", "stepImage1"),2000);
    clearInterval ( IntervalId4);

}

the code is simple,
onLoad timeMsg() is called
then after 2000 milliseconds each setInterval should be called. What i am finding is that only the first setInterval call is being fired and then the scripts stops...

anyone know why...

could it be that all 4 statements are being fired at the same time?

thanks in advance

Ilan

[tirna]

Maybe you should be using setTimeout() instead.

[iperez_genius]

setTimeout didn't work either...

i will give it another go though...

thanks for the reply...

[Kor]

setInterval and setTimeout need a reference of a function as argument. use an anonymous:

IntervalId1 =setInterval(function(){switchIcon('stepImage1','stepImage2')},2000);
// and so on

[Declan1991]

And you will have to use 2000 for the first setTimeout, 4000 for the second etc., because the time is being set from now, not from when the next finishes.

[Kor]

And you will have to use 2000 for the first setTimeout, 4000 for the second etc., because the time is being set from now, not from when the next finishes.

In fact setTimeout() would be a better choice, I presume.

[Declan1991]

In fact setTimeout() would be a better choice, I presume.

That reminds me, do you only want this to loop once? If you want it to loop multiple times, there's better ways of doing this.

[criterion9]

Another thing to consider:

IntervalId1 =setInterval(switchIcon("stepImage1", "stepImage2"),2000);
clearInterval ( IntervalId1);

Why are you clearing the intervals immediately after setting them?

[tirna]

setTimeout didn't work either...

i will give it another go though...

thanks for the reply...

What you want to do can be done using setTimeout().

If you post your code using setTimeout() it will be easier to see what is going on.