JavaScript异步函数按固定顺序执行问题求助

我看了你的代码，核心问题是你用了回调式的异步操作，但没有把它们包装成Promise，导致await完全起不到等待的作用，所以三个函数的执行顺序完全乱了，而且数据也没正确收集到。我们一步步来修复：

主要问题分析 #

1. checker函数的异步未被等待：sqldb.query是回调风格的API，你的async function checker()里直接调用它，并没有等待它执行完成就结束了。所以await checker()会立刻“完成”，直接跳到jsonreader，这时候jsons数组还是空的。
2. jsonreader的拼写错误与异步问题：
   • 你写了jsons.lenght，正确拼写是jsons.length，这会导致循环根本不执行；
   • fs.access和fs.readFile都是回调式API，同样没有被等待，文件还没读完，jsonreader就结束了，profile数组自然是空的；
   • 循环里的i没有用let声明，会导致闭包问题，所有循环迭代共用同一个i变量。
3. 异步操作未用Promise化版本：Node.js的fs模块有Promise化的API（fs.promises），数据库查询也可以包装成Promise，这样才能用await正确等待异步完成。

修复后的完整代码 #

function visualroute() {
  router.get('/user', verify, async (req, res) => {
    const jsons = [];
    const profile = [];

    // 1. 将数据库查询包装为Promise，让await能等待
    const checker = async function () {
      return new Promise((resolve, reject) => {
        sqldb.query(`SELECT * FROM ${username}`, (error, result) => {
          console.log('1');
          if (error) {
            console.log(error);
            return reject(error);
          }
          if (result.length === 0) {
            console.log('User does not have initiated a db yet');
            return res.render('user/index', { message: 'No data added to db yet' });
          } else {
            try {
              for (let i = 0; i < result.length; i++) { // 用let声明i，避免闭包问题
                jsons.push(`../public/jsonsheets/${result[i].infoticker}.json`);
              }
              console.log(jsons);
              resolve(); // 操作完成，resolve Promise
            } catch (err) {
              reject(err);
            }
          }
        });
      });
    };

    // 2. 用fs.promises的Promise化API，await每个异步操作
    const jsonreader = async function () {
      console.log('2');
      // 修正拼写错误：jsons.length
      for (let i = 0; i < jsons.length; i++) {
        try {
          // 用Promise化的access，await等待结果
          await fs.promises.access(`public/jsonsheets/${jsons[i]}`, fs.F_OK);
          console.log('Exists');
          // 用Promise化的readFile，await读取完成
          const data = await fs.promises.readFile(`public/jsonsheets/${jsons[i]}`);
          let userdata = JSON.parse(data);
          profile.push(userdata);
        } catch (err) {
          console.log('Does not exist or read error:', err);
        }
      }
      console.log(profile);
    };

    // 3. 发送数据
    const finalsender = async function () {
      console.log('3');
      res.send(profile);
    };

    // 按顺序执行三个函数
    try {
      await checker();
      await jsonreader();
      await finalsender();
    } catch (err) {
      // 处理全局错误，避免请求挂起
      console.error('流程出错:', err);
      res.status(500).send('Internal Server Error');
    }
  });
}
visualroute();

关键修改点说明 #

• 数据库查询Promise化：把sqldb.query包装进Promise，在查询完成后调用resolve()，这样await checker()会真正等待数据库操作完成，jsons数组才会被正确填充。
• 使用fs.promises：替换回调式的fs.access和fs.readFile为Promise版本，用await等待每个文件操作完成，确保profile数组能收集到所有文件数据。
• 修正拼写错误与变量声明：修复jsons.lenght为jsons.length，循环里用let i避免闭包问题。
• 全局错误捕获：在主流程外层加try/catch，避免某个步骤出错导致请求一直挂起。

这样修改后，三个函数会严格按照checker→jsonreader→finalsender的顺序执行，输出顺序会变成1→2→3，接收URL端也能拿到正确的profile数组了。

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