JavaScript循环内清空数组失效 CronJob重复发短信问题如何修复？

根因说明 #

你遇到的重复短信问题和清空finalSessionString数组没有关联，核心由三个问题导致：

• 重复创建Cron任务：你每45秒就会执行一次这段循环代码，每次执行都会创建新的cronJob实例，且从未停止旧的实例，运行5轮后就有5个任务同时运行，触发时自然会发送5条重复短信。
• 闭包变量引用错误：循环内创建cronJob的回调函数直接引用了循环作用域中的变量，回调触发时拿到的是循环结束后最后一次迭代的变量值，和你预期的单次迭代数据不一致。
• 你当前清空的finalSessionString是每次循环内部通过split新生成的临时数组，完全不需要清空，你提到的要插入新值的数组应该是全局存储数据的rows数组，清空临时数组不会产生任何效果。

修复步骤 #

1. 全局维护一个已创建Cron任务的列表，每次重新执行定时逻辑前，先停止所有旧任务并清空列表，避免任务堆积。
2. 用立即执行函数（IIFE）包裹cronJob创建逻辑，将当前迭代的参数传入封闭作用域，保证回调触发时拿到正确的参数。
3. 如果你需要每次插入新记录前清空全局存储数据的数组，直接清空对应的全局数组即可，不需要处理循环内的临时数组。

修复后代码示例 #

// 全局维护Cron任务列表
let cronJobs = [];

// 你每45秒执行一次的逻辑函数
function runTask() {
  // 先停止所有旧的Cron任务
  cronJobs.forEach(job => job.stop());
  cronJobs = [];

  // 这里是你从数据源获取rows的逻辑，如果你需要清空旧的全局数据数组，在这里清空对应的数组即可

  for (let i = 0; i < rows.length; i++) {
    let sessionString = rows[i][weekday];
    const finalSessionString = sessionString.split('|');
    const cronExpression = finalSessionString[0];
    const phoneNumber = finalSessionString[1];
    const userName = finalSessionString[2];
    const trainerName = finalSessionString[3];

    // 用IIFE包裹，创建独立作用域保存当前迭代的参数
    (function(cronExp, phone, user, trainer) {
      const textJob = new cronJob(cronExp, function() {
        client.messages.create({
          to: phone,
          from: '12055578708',
          body: `hello ${user}, Your training session just finished with ${trainer}`
        }, function(err, data) {
          if (err) {
            console.log("err: " + err);
          }
          console.log(data);
        });
      }, null, true);
      // 把新任务加入全局列表
      cronJobs.push(textJob);
    })(cronExpression, phoneNumber, userName, trainerName);
  }
}

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