async函数中forEach用await导致errorText始终为空的问题排查

你的代码核心问题出在**forEach无法正确处理异步回调**，具体原因和修复方案如下：

为什么errorText始终为空？ #

• forEach会遍历数组并执行回调，但它完全忽略回调函数的返回值（包括Promise），所以外层的await licenseList.forEach(...)根本不会等待所有异步操作完成。函数会直接跳到return errorText这一行，此时异步回调里的代码还没执行，自然errorText是空的。
• 另外还有两个冗余问题：
  • await licenseFile.then(...)完全没必要，因为licenseFile本身就是Promise，直接用await licenseFile获取值更简洁。
  • await Object.keys(...)是多余的，Object.keys是同步方法，不需要加await。

修复后的代码 #

推荐用for...of遍历，它能正确等待每个异步操作完成，还能在发现无效项时提前返回：

const isLicensesValid = async (
  licenseList: Promise<string>[]
): Promise<string> => {
  for (const licenseFile of licenseList) {
    const licenseData = await licenseFile;
    const parsedData = JSON.parse(licenseData);
    const licenseType = Object.keys(parsedData.Features)[0];
    if (!isLicenseValid(licenseData, licenseType.toLowerCase())) {
      return "One or more invalid license file in zip";
    }
  }
  return "";
};

如果需要等待所有许可证验证完成后再统一返回结果（比如收集多类错误），可以用Promise.all搭配map：

const isLicensesValid = async (
  licenseList: Promise<string>[]
): Promise<string> => {
  const validationResults = await Promise.all(licenseList.map(async (licenseFile) => {
    const licenseData = await licenseFile;
    const parsedData = JSON.parse(licenseData);
    const licenseType = Object.keys(parsedData.Features)[0];
    return isLicenseValid(licenseData, licenseType.toLowerCase());
  }));

  if (validationResults.some(result => !result)) {
    return "One or more invalid license file in zip";
  }
  return "";
};

关键总结 #

• 永远不要用forEach处理异步操作，改用for...of、Promise.all搭配map这类支持异步的遍历方式。
• 避免await和then嵌套混用，用await让异步代码逻辑更清晰易读。

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