Promise.all的catch意外触发？已捕获错误仍触发的问题排查

我的问题 #

我有一个异步方法populate()，尝试执行某些操作；若失败则捕获错误，并调用另一个异步方法populateFromServer()（该方法也可能失败）。我在Promise.all()中调用多个该方法，并在后续添加catch处理所有尝试都失败的情况。但不知为何，尽管错误应该已被捕获，.all()的catch始终因populate()触发。

以下是我的简化TypeScript代码：

abstract class Database {
  public static async populate() {
    try {
      await populateFromLocalFile();
    } catch (err) {
      await this.populateFromServer();
    }
  }

  private static async populateFromServer() {
    await fetchFromServer();  // 可能失败，比如无网络连接
  }
}

调用代码：

Promise.all([
  ChildDB1.populate(),
  ChildDB2.populate(),
  ChildDB3.populate(),
])
.then(() => /* 填充成功（本地或远程任一方式） */)
.catch(() => /* 所有填充尝试均失败 */)

我的代码中，即便populate()的错误已被捕获且populateFromServer()执行正常（我已通过日志确认），Promise.all的catch仍会触发。这是为什么？该如何修复？

我已尝试的方案 #

返回Promise.resolve() #

有人建议在catch块末尾返回Promise.resolve()，但问题仍未解决：

public static async populate() {
    try {
      await populateFromLocalFile();
    } catch (err) {
      await this.populateFromServer();
      return Promise.resolve()
    }
    return Promise.resolve()
  }

使用.then #

我还尝试改用.then的写法，但问题依旧：

abstract class Database {
  public static async populate() {
    await populateFromLocalFile()
      .then(() => doStuff())
      .catch(async () => {
          console.log('错误在此处发生');
          await this.populateFromServer();
      });
  }

  private static async populateFromServer() {
    await fetchFromServer();
  }
}

问题原因 #

你的populate()方法仅捕获了populateFromLocalFile()的错误，但调用populateFromServer()时如果抛出错误，这个错误会直接冒泡到方法外部，导致populate()返回一个rejected状态的Promise。

另外，你提到即使populateFromServer()执行正常，Promise.all()的catch仍触发，这说明至少有一个ChildDB实例的populate()方法最终返回了rejected状态的Promise——要么是该实例的本地和服务器填充都失败了，要么代码中存在其他未被捕获的错误。

修复方案 #

要实现“本地或远程任一方式成功则populate() resolve，只有两次都失败才reject”的逻辑，需要完整捕获两次尝试的错误：

方案1：嵌套try/catch（async/await风格） #

abstract class Database {
  public static async populate() {
    try {
      await populateFromLocalFile();
    } catch (localErr) {
      try {
        await this.populateFromServer();
      } catch (serverErr) {
        // 两次填充都失败，抛出错误让Promise.all的catch捕获
        throw new Error('本地和服务器填充均失败');
      }
    }
    // 任一方式成功，async函数自动返回resolved Promise
  }

  private static async populateFromServer() {
    await fetchFromServer();
  }
}

方案2：链式Promise处理 #

如果偏好.then/.catch风格，可以简化为：

abstract class Database {
  public static populate() {
    return populateFromLocalFile()
      .catch(() => this.populateFromServer());
    // 若populateFromServer也失败，会自动返回rejected Promise
  }

  private static async populateFromServer() {
    await fetchFromServer();
  }
}

关键说明 #

• 你之前手动返回Promise.resolve()无效，是因为如果populateFromServer()抛出错误，这个错误会先于return语句触发，导致方法返回rejected Promise。
• async函数中，只要没有未被捕获的错误，会自动返回resolved状态的Promise，无需手动return Promise.resolve()。

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