在Promise的.catch中使用async回调处理错误是否属于JS反模式？

结论 #

这种给.catch()传入async回调的写法不属于严格意义上的反模式，语法本身是支持的，你当前需求下的基础逻辑也能跑通，但存在几个非常容易踩的隐患，且可读性不如标准的try/catch结构。

首先纠正笔误 #

你代码中的aysnc是拼写错误，实际应该是async，否则代码会直接抛出语法错误。

可行的底层逻辑 #

.catch()的回调如果是async函数，会自动返回Promise，外层await会等待这个Promise状态变更后才会继续往下执行，所以你想要的「等rollback执行完再重置isExecuting」的需求，在没有额外异常的场景下是可以正常满足的。

潜在问题 #

1. rollback异常会导致逻辑中断、状态卡死
   如果rollback()本身执行报错，async回调会返回一个rejected状态的Promise，外层await会直接抛出这个异常，导致后续的isExecuting.current = false完全不会执行，isExecuting会永久为true，之后用户再也无法触发该方法。
2. 异常遗漏风险
   catch回调内部抛出的错误没有被捕获，会直接抛出全局的unhandledRejection异常，可能导致程序崩溃。
3. 可读性略低
   绝大多数JavaScript开发者更习惯用try/catch处理async/await的同步化错误，把异步错误处理逻辑塞到.catch()回调里，更容易被其他开发者漏看内部的异步逻辑。

更推荐的改写方案 #

用try/catch/finally结构，逻辑更清晰，也能保证状态无论如何都会被重置：

const method = async () => {
  if(isExecuting.current) return;
  isExecuting.current = true;

  try {
    await something();
  } catch (err) {
    console.log(err);
    try {
      await rollback();
    } catch (rollbackErr) {
      // 这里自行处理回滚失败的逻辑，比如上报错误、提示用户
      console.error('回滚操作失败', rollbackErr);
    }
  } finally {
    // 无论请求成功、请求失败、回滚失败，都会走到这里重置状态，避免卡死
    isExecuting.current = false;
  }
}

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