I declared that the code posted *didn’t* work (in response to Rick - comment 22), but that using my commented-out line *does* work.

That code is never going to work because you have not written it correctly.

Look at setTimeout(”test(”+p1+”)”, 400);
expand it yourself and you get

setTimeout( test(one), 400);

test(one) is not valid syntax - unless one is defined as a variable - which it is not.

You probably meant setTimeout(”test(’ “+p1+” ‘)”, 400);
Where there are some extra quotes to yield test(’one’).

Even then, it probably doesn’t do what you expect.

javascript pass parameters to setTimeout

… THANK YOU for being top result!

Funny enough I’d done what you did but wasnt sure why it was working, and was looking for the “proper” way to do it… I guess this was the proper way!

Maybe I’ve misread something, but the following code doesn’t work, while the commented-out line, which uses the closure, is fine:

$(document).ready(function() {
var p1 = 'one';
var p2 = 'two';
//setTimeout(function(){test(p1,p2);},400);
setTimeout("test("+p1+")", 400);
});

function test(param,param2) {
alert(param);
alert(param2);
}


I’d suggest that, any other considerations aside, the closure approach is more manageable syntax, particularly where several parameters are needed.

Bryan