Redux中如何实现批量API调用完成后执行Promise.all并捕获异常

你的代码存在两个核心问题，直接导致了当前的异常行为：

1. then回调的错误写法 #

你在Promise.all(openAccountsPromise).then(dispatch(AccountsReducer.setOpenAccountSuccess()))中直接调用了dispatch(...)，这会让setOpenAccountSuccess对应的action立刻被派发，完全不会等待Promise.all完成。then方法需要接收一个函数作为参数，而不是直接执行函数的结果。

2. 未等待Promise.all执行完成 #

由于你没有用await等待Promise.all的结果，外层的try/catch无法捕获Promise.all抛出的错误——当任意一个API调用失败时，Promise.all会进入rejected状态，但因为没有等待它，这个错误不会流入catch块，自然也无法触发失败状态的设置。

修改后的完整代码 #

export const openAccounts = (data, userId, addressId) => async (dispatch) => {
  dispatch(AccountsReducer.setOpenAccountsLoading());

  try {
    const openAccountsPromise = data.selectedProducts.map(product => {
      const formatedData = setOpenAccountAndProductData(data, userId, addressId, product);
      // 直接返回API调用的Promise，无需额外async/await
      return AccountsService.openAccounts(formatedData);
    });

    // 等待所有API调用完成，任意调用失败都会触发catch
    await Promise.all(openAccountsPromise);
    // 所有请求成功后再派发成功action
    dispatch(AccountsReducer.setOpenAccountSuccess());

  } catch(error) {
    dispatch(AccountsReducer.setOpenAccountFailure(error.response?.data));
  }
}

关键修改说明 #

• 用await替代then写法，借助async函数的特性，让Promise.all的错误自动流入catch块，确保任一请求失败时能触发失败状态。
• 将成功action的派发移到await Promise.all之后，保证只有所有API调用都成功才会执行这一步。
• 简化map中的逻辑：AccountsService.openAccounts本身返回Promise，无需额外包裹async/await，不影响请求的并行执行。

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