使用setInterval循环调用带不同参数的showCustomer函数问题求助

嘿，我立刻就发现你代码里的问题啦！你的定时器回调函数每次执行时，都会重新声明并初始化var i = 1——这意味着每次调用showCustomer传的都是1，i的自增根本起不到作用，因为下次回调执行时i又被重置成1了。

为什么会这样？ #

var声明的变量没有块级作用域，而且你把i放在了setInterval的回调函数内部，所以每过5秒，回调执行时都会重新创建一个i变量，初始值是1，调用完函数后i自增，但这个变量在回调结束后就被销毁了，完全起不到累加的作用。

修复方案来了，给你几种选择： #

方案1：把计数器放在回调外部（最直观）

把i的声明移到定时器外面，让它在每次回调执行时都能保留上一次的值，还能加上边界判断让数字在1-5之间循环：

let i = 1;
setInterval(function () {
  showCustomer(i);
  // 让i在1-5之间循环
  i = i >= 5 ? 1 : i + 1;
}, 5000);

方案2：用闭包封装计数器（避免全局变量）

如果你不想让i暴露在全局作用域里，可以用立即执行函数创建一个闭包，把变量封装起来：

(function() {
  let i = 1;
  setInterval(function () {
    showCustomer(i);
    i = i > 4 ? 1 : i + 1;
  }, 5000);
})();

方案3：递归式setTimeout（更灵活的循环控制）

如果需要更精细的控制（比如某次调用出错时可以中断循环），递归调用setTimeout会比setInterval更可靠：

function startCustomerLoop() {
  let i = 1;
  function callNext() {
    showCustomer(i);
    i = i >=5 ? 1 : i + 1;
    setTimeout(callNext, 5000);
  }
  callNext();
}
// 启动循环
startCustomerLoop();

小提醒 #

别忘了加上边界判断哦！不然i会一直递增，超过你函数要求的1-5范围，导致showCustomer可能无法正常工作。

内容的提问来源于stack exchange，提问作者Maxim Baran