在 Promise 内完成 for 循环之前返回 Resolve

Question

我很困惑为什么return resolve({ status: true, data: newIDs }); 在for循环完成之前被调用。

代码

  createPallet: (data) => {
    return new Promise(async (resolve, reject) => {
      const newIDs = [];
      try {
        for (const record of data) {
          mysqlConnection.beginTransaction();
          mysqlConnection.query(
            "INSERT INTO ...",
            [record.BatchId, record.PalletNumber, record.PrimaryWeightId],
            async (error, results) => {
              if (error) {
                return reject(error);
              }

              // Create pallet sku record
              await PalletSkusService.createPalletSku(
                ...
              );

              console.log(results.insertId);
              newIDs.push(results.insertId);
            }
          );
        }
        return resolve({ status: true, data: newIDs });
      } catch (error) {
        mysqlConnection.rollback();
        return reject(error);
      }
    });
  },

期待

我希望 for 循环获取所有新插入的 id，并将它们存储到newIDs数组中。 完成后，我可以返回包含数据的解决方案。

然而，这部分我不太明白的是 - 为什么在 for 循环完成之前运行 resolve？

在这种情况下正确的做法是什么？

Answer 1

mysqlConnection.query 立即返回，而不是等待数据返回。 我不知道您为此使用的是什么库，但您想找到基于 promise 的方法来完成此操作。 然后，修改循环，使其在解析之前等待一系列承诺完成。

async createPallet(data) {
  try {
    // modified loop
    const newIDs = await Promise.all(data.map((record) => {
      mysqlConnection.beginTransaction();
      // this is a best guess for how the a promise-style query would look
      const await result = await mysqlConnection.query(
        "INSERT INTO ...",
        [record.BatchId, record.PalletNumber, record.PrimaryWeightId]
      );

      // Create pallet sku record
      await PalletSkusService.createPalletSku(
        ...
      );

      console.log(results.insertId);
      return results.insertId;
    }));

    return { status: true, data: newIDs };
  } catch (error) {
    mysqlConnection.rollback();
    throw error;
  }
},