我有一些JavaScript代码，看起来像:
function statechangedPostQuestion()
{
//alert("statechangedPostQuestion");
if (xmlhttp.readyState==4)
{
var topicId = xmlhttp.responseText;
setTimeout("postinsql(topicId)",4000);
}
}
function postinsql(topicId)
{
//alert(topicId);
}
我得到一个错误，topicId没有定义 在我使用setTimeout()函数之前，一切都在工作。
我希望我的postinsql(topicId)函数在一段时间后被调用。 我该怎么办?
当前回答
由于IE中的第三个可选参数有问题，并且使用闭包阻止我们改变变量(例如在循环中)，但仍然达到预期的结果，我建议以下解决方案。
我们可以尝试像这样使用递归:
var i = 0;
var hellos = ["Hello World1!", "Hello World2!", "Hello World3!", "Hello World4!", "Hello World5!"];
if(hellos.length > 0) timeout();
function timeout() {
document.write('<p>' + hellos[i] + '<p>');
i++;
if (i < hellos.length)
setTimeout(timeout, 500);
}
我们需要确保没有其他东西改变这些变量，并且我们写了一个适当的递归条件来避免无限递归。
其他回答
有些答案是正确的，但令人费解。
四年后，我再次回答这个问题，因为我仍然会遇到过于复杂的代码来解决这个问题。有一个优雅的解决方案。
首先，在调用setTimeout时不要传入一个字符串作为第一个参数，因为它有效地调用了对缓慢的“eval”函数的调用。
那么如何将参数传递给超时函数呢?通过使用闭包:
settopic=function(topicid){
setTimeout(function(){
//thanks to closure, topicid is visible here
postinsql(topicid);
},4000);
}
...
if (xhr.readyState==4){
settopic(xhr.responseText);
}
一些人建议在调用timeout函数时使用匿名函数:
if (xhr.readyState==4){
setTimeout(function(){
settopic(xhr.responseText);
},4000);
}
语法是正确的。但是当settopic被调用时，也就是4秒之后，XHR对象可能就不一样了。因此，预绑定变量非常重要。
取代
setTimeout("postinsql(topicId)", 4000);
with
setTimeout("postinsql(" + topicId + ")", 4000);
或者更好的是，用匿名函数替换字符串表达式
setTimeout(function () { postinsql(topicId); }, 4000);
编辑:
Brownstone的评论是不正确的，这将按照预期工作，正如在Firebug控制台中运行它所演示的那样
(function() {
function postinsql(id) {
console.log(id);
}
var topicId = 3
window.setTimeout("postinsql(" + topicId + ")",4000); // outputs 3 after 4 seconds
})();
请注意，我同意其他人的观点，即您应该避免将字符串传递给setTimeout，因为这将在字符串上调用eval()，而不是传递一个函数。
由于IE中的第三个可选参数有问题，并且使用闭包阻止我们改变变量(例如在循环中)，但仍然达到预期的结果，我建议以下解决方案。
我们可以尝试像这样使用递归:
var i = 0;
var hellos = ["Hello World1!", "Hello World2!", "Hello World3!", "Hello World4!", "Hello World5!"];
if(hellos.length > 0) timeout();
function timeout() {
document.write('<p>' + hellos[i] + '<p>');
i++;
if (i < hellos.length)
setTimeout(timeout, 500);
}
我们需要确保没有其他东西改变这些变量，并且我们写了一个适当的递归条件来避免无限递归。
setTimeout(function() {
postinsql(topicId);
}, 4000)
您需要将匿名函数作为参数而不是字符串提供，根据ECMAScript规范，后一种方法甚至不应该工作，但浏览器只是宽容。这是正确的解决方案，当使用setTimeout()或setInterval()时，不要依赖于传递字符串作为“函数”，它更慢，因为它必须被计算，这是不正确的。
更新:
正如Hobblin在他对问题的评论中所说，现在你可以使用function .prototype.bind()将参数传递给setTimeout内的函数。
例子:
setTimeout(postinsql.bind(null, topicId), 4000);
在做了一些研究和测试之后，唯一正确的实现是:
setTimeout(yourFunctionReference, 4000, param1, param2, paramN);
setTimeout将所有额外的参数传递给你的函数，以便在那里处理它们。
匿名函数可以用于非常基本的东西，但在一个对象的实例中，你必须使用“this”，没有办法使它工作。 任何匿名函数都会将“this”更改为指向窗口，因此您将失去对象引用。
