Granted the following piece of code:

function updateOdometers(odometers) {
    setTimeout(function(){
        odometers[1].update(odometers[1].value + 10);
    }, 500);

}

setInterval(updateOdometers(odometers), 2000);

For whatever reason, this code updates the value of odometer only once, rather than every 2000ms with a delay inside. Googling/SO-ing around didn't get me much of a result. Any ideas?

Granted the following piece of code:

function updateOdometers(odometers) {
    setTimeout(function(){
        odometers[1].update(odometers[1].value + 10);
    }, 500);

}

setInterval(updateOdometers(odometers), 2000);

For whatever reason, this code updates the value of odometer only once, rather than every 2000ms with a delay inside. Googling/SO-ing around didn't get me much of a result. Any ideas?

• javascript
• settimeout
• setinterval

• Why are you trying to do this? – Bergi Commented Mar 3, 2015 at 20:39
• 2 Duplicate of stackoverflow./questions/28837247/… – Telokis Commented Mar 3, 2015 at 20:40
• Blah, didn't find the original question you're referring to. – favoretti Commented Mar 3, 2015 at 20:55
• @Bergi: this is a not very sane sanitized piece of code. I have a number of odometers I need to update. The update has to happen every 6 seconds and inner odometers need to update one by one. Makes sense? – favoretti Commented Mar 3, 2015 at 21:03

1 Answer 1

This line :

setInterval(updateOdometers(odometers), 2000);

should be

setInterval(function () {updateOdometers(odometers);}, 2000);

Otherwise you will be calling updateOdometers(odometers) and passing its result to setInterval.