Sequelize.js定时循环中findAll仅返回单条数据问题求助

你的问题跟findAll()无关，它既没有只执行一次，也没有只返回一条数据——是你写的for循环条件错误，导致没遍历到所有查询结果。

具体问题分析 #

你当前的for循环写法：

for (let i = 0; u_sers.length; i++) {
  // 循环体
}

这里的循环条件是u_sers.length，只要数据库返回的用户数组不为空，这个条件就永远是true，循环会持续执行（直到i超出数组索引范围，访问u_sers[i]得到undefined）。你说“仅获取第一条条目就停止了”，大概率是循环体逻辑只执行一次就意外终止，或者你误写了循环条件，但核心问题都出在这个错误的循环判断上。

正确的循环写法 #

要遍历所有用户数据，循环条件需要判断索引i是否小于数组长度，也就是i < u_sers.length：

const task = cron.schedule('* * * * *', async () => {
    try {
        let u_sers = await User.findAll();
        // 修正循环条件，遍历所有用户
        for (let i = 0; i < u_sers.length; i++) {
          // 在这里处理每个用户，比如输出用户信息
          console.log('用户数据:', u_sers[i]);
        }
        console.log('running a task every minute');
    } catch (err) {
        console.log(err);
    }
});

更简洁的替代写法 #

不想手动处理索引的话，用for...of循环更省心，还能避免索引错误：

for (const user of u_sers) {
  console.log('用户数据:', user);
}

额外提醒：如果findAll()确实只返回一条数据，那你需要检查数据库中是否本身只有一条记录，或者是否给findAll()添加了未展示的过滤条件。

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