How to use setInterval function within for loop

Question

I'm trying to run multiple timers given a variable list of items. The code looks something like this:

var list = Array(...);

for(var x in list){
    setInterval(function(){
        list[x] += 10;
        console.log(x + "=>" + list[x] + "\n");
    }, 5 * 1000);
}

The problem with the above code is that the only value being updated is the item at the end of the list, multiplied by the number of items in the list.

Can anyone offer a solution and some explanation so I know why it's behaving this way?

score 45 · Answer 1 · edited Aug 06 '18 at 07:58

45

var list = [1, 2, 3, 4, 5];

for (var i = 0, len = list.length; i < len; i += 1) {
    (function(i) {
        setInterval(function() {
            list[i] += 10;
            console.log(i + "=>" + list[i] + "\n");
        }, 5000)
    })(i);
}

Here is the working code:

var list = [1, 2, 3, 4, 5];

for (var i = 0, len = list.length; i < len; i += 1) {
    (function(i) {
        setInterval(function() {
            list[i] += 10;
            console.log(i + "=>" + list[i] + "\n");
        }, 5000)
    })(i);
}

Here the index i is stored in an anonymous function, so that it is not overwritten by consecutive loops. setInterval function in your code keeps the reference only to the last value of i.

edited Aug 06 '18 at 07:58

Akhil Aravind

5,741
16
35

answered Oct 13 '11 at 03:24

Lapple

3,385
20
20

Excellent solution! A small note: anything that can be changed in the loop can be passed to the anonymous function, not just index. – Muntashir Akon Sep 22 '17 at 15:55
Thank you very much! Using the anonymus function works perfect! – quinqui Feb 23 '22 at 23:21

canon · Accepted Answer · 2019-06-20T16:05:22.317

So, a few things:

Most importantly, the callback function you've passed to setInterval() maintains a reference to x rather than the snapshot value of x as it existed during each particular iteration. So, as x is changed in the loop, it's updated within each of the callback functions as well.
Additionally, for...in is used to enumerate object properties and can behave unexpectedly when used on arrays.
What's more, I suspect you really want setTimeout() rather than setInterval().

You can pass arguments to your callback function by supplying additional arguments to setTimout():

var timeoutID = window.setTimeout(func, delay, [param1, param2, ...]);

Numbers will be passed by value rather than reference. Here's an example:

var list = [1,2,3,4];

for (var x = 0, ln = list.length; x < ln; x++) {
  setTimeout(function(y) {    
    console.log("%d => %d", y, list[y] += 10);
  }, x * 500, x); // we're passing x
}

Yes, might I recommend reading this article: http://blog.morrisjohns.com/javascript_closures_for_dummies. On top of that though, setTimeout within a loop may not be what he wants to do anyway as all of the callbacks will be fired at the same time, they will not be staggered. — SoWeLie, Oct 13 '11 at 03:25
The OP doesn't "need a closure", quite the opposite—it has a closure to *x* that needs to be avoided (which your answer actually does). It keeps the closure to *list* though. — RobG, Oct 13 '11 at 04:00

score 6 · Answer 3 · answered Oct 26 '18 at 12:05

You can combine forEach and setTimeout to loop over the array with the interval.

let modes = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10];
let interval = 1000; //one second
modes.forEach((mode, index) => {

  setTimeout(() => {
    console.log(mode)
  }, index * interval)
})

score 4 · Answer 4 · edited Sep 12 '13 at 10:47

4

You don't have to use a for cycle with the setInterval statement. Try this:

var list = Array(...);
var x = 0;

setInterval(function() {

    if (x < list.length;) {
        list[x] += 10;
        console.log(x+"=>"+list[x]);
    }

    else return;

    x++;
}, 5000);

edited Sep 12 '13 at 10:47

marsei

7,691
3
32
41

answered Sep 12 '13 at 10:30

uncleJessie

41
1

score 2 · Answer 5 · answered Mar 29 '16 at 15:41

I don't know how to do this with a for loop but this code here will print out each element in an array at timed intervals:

function displayText(str) {
   $('.demo').append($('<div>').text(str));
}
var i = 0;

var a = [12, 3, 45, 6, 7, 10];

function timedLoop() {
setTimeout(function () {
    displayText(a[i]);
    i++;
    if(i < a.length) {
        timedLoop();
    }
}, 2000)
}

timedLoop();

Using a bit of jquery to show it in the browser.

Karandeep Virk · Answer 6 · 2022-10-13T04:05:45.150

1

Please look at this simplest solution. It does work with forloop too. It looks like an interval but it is a timeout increased for each iteration. For each iteration setTimeout doubles.

for(let i=1; i<=10;i++){
    setTimeout(function(){
        console.log(i)
    },i*1000)
}

edited Oct 13 '22 at 04:05

answered Oct 13 '22 at 04:04

Karandeep Virk

31
2

score 0 · Answer 7 · answered Dec 04 '14 at 10:25

0

If you have JSON array and jQuery included, you can use:

$.each(jsonArray, function(i, obj) {
    setInterval( function() {
        console.log(i+' '+obj);
    }, 10);
});

answered Dec 04 '14 at 10:25

atulmy

1,364
15
23

How to use setInterval function within for loop

7 Answers7

Linked

Related