Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How to preserve setTimeout parameter value until execution?

I have a bit of code that executes on keypress and saves data to a database as the user types.

I added a setTimeout function with a clearTimeout infront of it so not EVERY single character the user enters is sending out an Ajax request to save data.

While the setTimeout works great for one input field , if the user decides to switch input fields quickly (before the setTimeout delay is up) the parameter passed to the callSomeAjax changes before the function is executed.

Simplified version of what's happening...

var a = 1; //some user data

//function to call ajax script to save data to database
function callSomeAjax(a)
{
   console.log(a);
}

setTimeout(function(){callSomeAjax(a)},1000); //executes callSomeAjax after 1 second delay

a=3; //change user data before callSomeAjax executes

// Result of console.log is 3, not 1 like I want it to be...

Code on fiddle

Any ideas?

like image 753
payling Avatar asked Dec 03 '13 17:12

payling


2 Answers

Newer browsers will let you pass the argument to setTimeout, and retrieve it in the callback.

setTimeout(function(a){
    return callSomeAjax(a);
}, 1000, a);

Or you could bind it to the function

setTimeout(function(a){
    return callSomeAjax(a);
}.bind(null, a), 1000);

Aside from those, you'd need to create the setTimeout callback in a new scope with the value.

function callbackWithValue(a) {
    return function() {
        return callSomeAjax(a);
    };
}

setTimeout(callbackWithValue(a), 1000);

Or if you already have the function as you do, and there's nothing else to pass, then you can make it more generic...

function callbackWithValue(fn) {
    var args = [].slice.call(arguments, 1);
    return function() {
        return fn.apply(null, args);
    };
}

setTimeout(callbackWithValue(callSomeAjax, a), 1000);

This last one is shows the beginnings of a .bind() function. It can be further expanded to allow the binding of this and the concatenation of additional arguments passed after the bound one.

like image 186
Blue Skies Avatar answered Oct 04 '22 08:10

Blue Skies


The problem is that setTimeout doesn't run the function passed in until 1 second has passed. What that does happen a has changed, so when callSomeAjax(a) is ran, a is at its new value.

You need to capture (in a closure) the value of a before calling setTimeout.

var a = 1;

function show(a) {
    console.log(a);
}

(function(a){
    setTimeout(function () {
        show(a)
    }, 1000);
}(a));

a = 3;

DEMO: http://jsfiddle.net/U59Hv/2/

like image 41
Rocket Hazmat Avatar answered Oct 04 '22 07:10

Rocket Hazmat