Question

Possible Duplicate:
Javascript infamous Loop problem?

With the for loop, it does nothing. Without the for loop, it works perfect.

Without the for loop:

var refreshId = setInterval(function(){
  $("#k1").fadeTo(1000,0.01,callfunc).fadeTo(1000,1);
}, 5000);
var refreshId = setInterval(function(){
  $("#k2").fadeTo(1000,0.01,callfunc).fadeTo(1000,1);
}, 5000);

and so on…

With the for loop:

for(var i=1;i<7;i++){
  var refreshId = setInterval(function(){
    $("#k"+i).fadeTo(1000,0.01,callfunc).fadeTo(1000,1);
  }, 5000);
}

What am I doing wrong here?

Answer 1

As @pimvdb pointed out you have closure (setTimeout creates one) inside loop, it won’t work, you should have something like:

for(var i = 1; i < 7; i++) {
    (function(e) {
        var refreshId = setTimeout(function() {
            $("#k"+e).fadeTo(1000,0.01,callfunc).fadeTo(1000,1);
        }, 5000);
    })(i);
}

You may also want to take a look at this question for more information.