Node.js Async/Await执行顺序异常及Promise未正确等待问题排查求助

嘿，我看你这代码的问题主要出在回调API和async/await混用，还有几个细节错误打乱了执行顺序，咱们一步步捋清楚：

核心问题拆解 #

1. connection.query没被Promise化，导致checkfornewdata提前返回 #

你在checkfornewdata里用了connection.query的回调写法，虽然给回调加了async，但这个回调的执行完全不会阻塞checkfornewdata本身——函数会直接走到末尾，默认返回一个值为undefined的resolved Promise，这就是为什么test里的console.log(db)会先输出undefined。

2. getusers函数的await和then混用，变量赋值出错 #

你在getusers里既用了await又套了.then()，不仅没必要，还出现了变量名拼写错误（joindate和joinDate大小写不一致），导致最终返回的joinDate还是初始的undefined。

3. 其他细节bug #

• newusers没初始化，直接push会报错；
• connection.end()放在循环内部，第一次循环就会关闭数据库连接，后面的遍历直接失效；
• 用for...in遍历数组不太稳妥，容易遍历到数组原型上的额外属性。

修正后的完整代码 #

async function test() { 
  var db = await checkfornewdata();
  console.log(db); // 现在会正确输出筛选后的新用户列表
}

async function checkfornewdata() { 
  const state = `SELECT * FROM users`;
  let newusers = []; // 初始化空数组，避免push报错

  // 把回调式的connection.query包装成Promise，才能用await等待
  const queryResult = await new Promise((resolve, reject) => {
    console.log('Get connection ...');
    connection.query(state, (err, result) => {
      if (err) reject(err);
      else resolve(result);
    });
  });

  console.log('Query Result');
  console.log(queryResult);

  // 用for...of遍历数组，比for...in更安全
  for (const user of queryResult) {
    const checknew = await getusers(user.id);
    if (user.joinDate > checknew.joinDate) {
      newusers.push(user);
    }
    console.log('New entries:');
    console.log(newusers);
  }

  connection.end(); // 所有数据处理完再关闭连接
  return newusers;
}

async function getusers(id) { 
  if (typeof id !== 'undefined') {
    console.log("getting joinDate of " + id);
    try {
      // 直接await第三方API，不用套then链
      const User = await 3rdpartyapi.getUser(id);
      const joinDate = User.joinDate;
      console.log(joinDate);
      return joinDate;
    } catch (err) {
      console.log(err);
      throw err; // 抛出错误让上层处理，避免把错误当成正常返回值
    }
  }
  return null; // 处理id为undefined的边界情况
}

修复后执行顺序说明 #

现在代码的执行流程会变成你期望的样子：

1. 先完成数据库查询，输出Query Result和数据库条目；
2. 逐个调用第三方API，输出每个用户的joinDate；
3. 筛选出符合条件的新用户，输出New entries；
4. 最后test函数里输出最终的筛选结果。

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