使用node-schedule恢复任务时出现Cannot read property of undefined错误求助

问题根源 #

你遇到的问题是变量作用域导致的延迟绑定：

• 用var声明的i属于函数作用域，在整个循环结束后，i的值会固定为plates.length（数组的长度，超出了数组的索引范围）。
• 当定时任务触发执行回调时，才会去读取plates[i]，此时i已经是plates.length，plates[i]自然是undefined，所以会抛出Cannot read properties of undefined (reading 'url')的错误。
• 而你用console.log()在循环过程中能正常读取，是因为此时i是当前迭代的有效值，还没到循环结束的最终值。

解决方案 #

以下三种方式都能解决这个问题，推荐优先使用前两种ES6+的写法：

1. 将var替换为let（推荐）

let是块级作用域，每次循环都会创建一个独立的i绑定，回调函数会捕获当前迭代的i值：

const now = new Date();
for(let i = 0; i < plates.length; i++){
    if(now >= plates[i].expires_on){
        end_auction(plates[i].url);
    }else{
        jobs.push(schedule.scheduleJob(new Date(plates[i].expires_on), () => end_auction(plates[i].url)));
    }
}

2. 使用forEach循环（更简洁）

forEach的回调函数会为每个迭代项创建独立作用域，直接捕获当前的plate对象：

const now = new Date();
plates.forEach(plate => {
    if(now >= plate.expires_on){
        end_auction(plate.url);
    }else{
        jobs.push(schedule.scheduleJob(new Date(plate.expires_on), () => end_auction(plate.url)));
    }
});

3. 立即执行函数（IIFE，ES6之前的兼容写法）

通过立即执行函数为每次循环创建独立作用域，保存当前迭代的索引或对象：

var now = new Date();
for(var i = 0; i < plates.length; i++){
    (function(currentIndex){
        const plate = plates[currentIndex];
        if(now >= plate.expires_on){
            end_auction(plate.url);
        }else{
            jobs.push(schedule.scheduleJob(new Date(plate.expires_on), () => end_auction(plate.url)));
        }
    })(i);
}

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