如何避免在if-else语句中嵌套Promise？代码重构求助

嘿，我完全懂你的困扰——你觉得自己没做“嵌套Promise”，但部署工具还是揪着这个警告不放。其实问题出在你第一个then的回调里，又嵌套了另一个then链（staffRef.delete().then(...)），工具会把这种在回调里创建新Promise链的情况判定为嵌套问题。

下面给你两种重构方案，都能彻底消除这个警告，同时让代码更易维护：

方案一：使用async/await（推荐） #

这是最直观的方式，把异步代码写成同步风格，完全摆脱嵌套的then回调：

async function deleteStaffProfile(uid) {
  const staffRef = db.collection("staff").doc(uid);
  const doc = await staffRef.get();

  if (doc.exists) {
    await staffRef.delete();
    console.log("Employee ", uid, " profile has been deleted in staff collection");
  } else {
    console.log("Employee ", uid, " had no dependencies");
  }

  return null;
}

为什么这能解决问题？ #

async/await让我们可以直接等待异步操作完成，不用嵌套回调。整个逻辑是线性的，没有嵌套的Promise链，完美符合工具对“无嵌套Promise”的要求。

方案二：拉平Promise链（兼容旧环境） #

如果你因为环境限制不能用async/await，可以把嵌套的then拉成链式调用，避免在回调里创建新的Promise链：

function deleteStaffProfile(uid) {
  const staffRef = db.collection("staff").doc(uid);
  return staffRef.get()
    .then((doc) => {
      if (doc.exists) {
        // 直接返回delete的Promise，让后续then处理结果
        return staffRef.delete();
      } else {
        console.log("Employee ", uid, " had no dependencies");
        // 返回一个resolved的Promise，保持链的一致性
        return Promise.resolve(null);
      }
    })
    .then((deleteResult) => {
      // 这里判断是否是delete操作的结果
      if (deleteResult !== null) {
        console.log("Employee ", uid, " profile has been deleted in staff collection");
      }
      return null;
    });
}

关键改动点 #

把原来嵌套在if里的then回调，移到了主Promise链的外部。这样整个Promise链是扁平的，没有回调嵌套，工具就不会触发警告了。

两种方案都能解决你的问题，优先选async/await，代码可读性更高~

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