i have this example:

var myApp = (function() {

    var inputClick = function() {
        console.log('inputClick');
    };

    var loadRecentTimeout = function()
    {
    window.setTimeout("inputClick()",3000);
    };

    return {
      loadRecentTimeout:loadRecentTimeout,
      inputClick:inputClick
    };


})();

myApp.loadRecentTimeout(); // this returns inputClick() undefined

window.setTimeout("myApp.inputClick();",3000); // this one seems to work , but it calls that method only one time and not every 3 seconds

can anyone explain how can i make this code call the inputClick() method every 3 seconds?

thanks

i have this example:

var myApp = (function() {

    var inputClick = function() {
        console.log('inputClick');
    };

    var loadRecentTimeout = function()
    {
    window.setTimeout("inputClick()",3000);
    };

    return {
      loadRecentTimeout:loadRecentTimeout,
      inputClick:inputClick
    };


})();

myApp.loadRecentTimeout(); // this returns inputClick() undefined

window.setTimeout("myApp.inputClick();",3000); // this one seems to work , but it calls that method only one time and not every 3 seconds

can anyone explain how can i make this code call the inputClick() method every 3 seconds?

thanks

• javascript
• jquery
• settimeout
• module-pattern

Share Improve this question Follow asked Aug 20, 2012 at 18:49 PatrioticcowPatrioticcow 27.1k7676 gold badges221221 silver badges340340 bronze badges 1

• 2 you should never pass a string to setInterval or setTimeout. – jbabey Commented Aug 20, 2012 at 18:52

Add a ment  | 

2 Answers 2

Sorted by: Reset to default 4

You want to call setInterval instead of setTimeout

var eventInterval = window.setInterval(function () { 
                        myApp.inputClick(); 
                    },3000);

You also should pass your function as a function instead of a string.

If you need to cancel your repeating event you can call clearInterval

clearInterval(eventInterval)

When you use a string "functionName()" it evals it in window scope. Instead, assign a reference to the function with just using the name. setTimeout only fires once, you want to use a setInterval.

var myApp = (function() {

    var inputClick = function() {
        console.log('inputClick');
    };

    var loadRecentTimeout = function()
    {
        window.setInterval(inputClick,3000); 
    };

    return {
      loadRecentTimeout:loadRecentTimeout,
      inputClick:inputClick
    };


})();