ansaurus

Question

Answer 1

+1 A:

setInterval will work.

Also, your function name is moveItem not moverItem.

Your code should look like this:

function moveItem() {
   //... your code.
   setInterval( moveItem, 2000 );
}

Jacob Relkin 2010-01-25 15:16:42

Answer 2

+2 A:

setInterval(moveItem, 2000);

is the way to execute the function moveItem every 2 seconds. The main problem in your code is that you're calling setInterval inside of, rather than outside of, the callback. If I understand what you're trying to do, you can use this:

function moveItem() {
    jQuery('.stripTransmitter ul li a').trigger('click');
}

setInterval(moveItem, 2000);

N.B.:Don't pass strings to setTimeout or setInterval - best practice is to pass an anonymous function or a function identifier (as I did above). Also, be careful to not mix single and double quotes. Pick one and stick with it.

Matt Ball 2010-01-25 15:17:03

Answer 3

+1 A:

I believe you are looking for setInterval()

Derek Adair 2010-01-25 15:17:05

Answer 4

A:

You should try something like this:

 function update(){
    i++;
    document.getElementById('tekst').innerHTML = i;
    setInterval(update(),1000);
    }

This means that you have to create a function in which you do the stuff you need to do, and make sure it will call itself with an interval you like. In your body onload call the function for the first time like this:

<body onload="update()">

Younes 2010-01-25 15:17:18

Code is broken. You're passing `undefined` to setInterval and recursively calling `update()` forever without any pause. (hint: the return value of update() is what you are passing to setInterval here)

slebetman 2010-01-25 15:22:46

-1 you create a new repeating interval every time you call this function .. its plain wrong .. and has the same problem the OP is asking help for ..

Gaby 2010-01-25 17:25:33

Answer 5

+1 A:

It should be:

function moveItem() {
  jQuery(".stripTransmitter ul li a").trigger('click');
}
setInterval(moveItem,2000);

setInterval(f, t) calls the the argument function, f, once every t milliseconds.

ntownsend 2010-01-25 15:17:41

Answer 6

+8 A:

Note that setTimeout and setInterval are very different functions:

setTimeout will execute the code once, after the timeout.
setInterval will execute the code forever, in intervals of the provided timeout.

Both functions return a timer ID which you can use to abort the timeout. All you have to do is store that value in a variable and use it as argument to clearTimeout(tid) or clearInterval(tid) respectively.

So, depending on what you want to do, you have two valid choices:

// set timeout
var tid = setTimeout(mycode, 2000);
function mycode() {
  // do some stuff...
  tid = setTimeout(mycode, 2000); // repeat myself
}
function abortTimer() { // to be called when you want to stop the timer
  clearTimeout(tid);
}

or

// set interval
var tid = setInterval(mycode, 2000);
function mycode() {
  // do some stuff...
  // no need to recall the function (it's an interval, it'll loop forever)
}
function abortTimer() { // to be called when you want to stop the timer
  clearInterval(tid);
}

Both are very common ways of achieving the same.

Miguel Ventura 2010-01-25 15:18:14

Go with the first one. Its better. Just have the timeout function create a new timeout call to itself.

a432511 2010-01-25 15:20:18

Hey, Miguel, thanks, that solved the problem, and it was so simple, I feel kind of ridiculous...

Marcos Buarque 2010-01-25 16:01:56

Edited for `vat` typos.

Matt Ball 2010-01-25 17:19:53

ansaurus

tags:

views:

answers:

Loop timer in javascript

related questions