如何在不使用Promise.defer()的情况下结合Promise与循环并重构代码？

嘿，我来帮你把这段混乱的代码捋清楚，彻底丢掉Promise.defer()这种反模式，同时让逻辑回归简洁～

首先咱们明确核心需求：把user对象的指定属性（fieldArray里的字段）复制到新对象，其中部分字段可能需要异步获取值。原来的代码用Promise.defer()手动管理Promise状态，还得手动计数跟踪异步完成情况，这不仅容易出错，也完全不符合Promise的设计思路。

重构思路拆解 #

1. 抛弃Promise.defer()反模式：Promise本身提供了Promise.resolve()、Promise.all()等方法，完全不需要手动创建deferred对象来控制状态。
2. 统一同步/异步逻辑：把每个字段的处理逻辑封装成独立函数，不管是同步取值还是异步调用，都返回一个Promise，这样就能用统一方式处理所有字段。
3. 用Promise.all()批量处理异步：把fieldArray转换成Promise数组，用Promise.all()等待所有字段处理完成后再合并结果，替代手动计数的繁琐逻辑。

重构后的代码示例 #

function cleanFunction(user, fieldArray) {
  // 没有指定字段数组时，直接返回user对象的Promise
  if (!fieldArray) {
    return Promise.resolve(user);
  }

  // 封装单个字段的处理逻辑：同步/异步都返回Promise
  const handleField = (field) => {
    // 这里替换成你实际的字段判断逻辑
    if (/* 该字段为同步取值的情况 */) {
      // 同步逻辑用Promise.resolve包装，统一返回格式
      return Promise.resolve({ [field]: user[field] });
    } else {
      // 异步逻辑：调用返回Promise的异步函数，处理后返回键值对
      return someAsyncOperation(user[field])
        .then(asyncResult => ({ [field]: asyncResult }));
    }
  };

  // 批量处理所有字段，等待全部完成后合并结果
  return Promise.all(fieldArray.map(handleField))
    .then(fieldResults => {
      // 把每个字段的结果合并成最终对象
      return fieldResults.reduce((finalObj, fieldData) => {
        return { ...finalObj, ...fieldData };
      }, {});
    })
    .catch(error => {
      // 统一处理所有错误，避免遗漏
      console.error('处理字段时出错:', error);
      throw error; // 可根据需求决定是否抛出错误
    });
}

为什么这样更好？ #

• 代码更简洁：丢掉了手动计数、deferred状态管理的冗余代码，逻辑一目了然。
• 错误处理更可靠：Promise链式调用会自动传递错误，加上统一的.catch()，不会像原代码那样容易遗漏错误处理。
• 可维护性更高：单个字段的处理逻辑抽成独立函数，后续修改字段规则时，只需要调整handleField即可。
• 符合Promise最佳实践：完全避免了Promise.defer()这种反模式，利用原生Promise API实现异步流程控制。

如果你的异步操作需要串行执行（比如前一个字段的异步结果会影响后一个），可以把Promise.all()换成Array.prototype.reduce()来链式调用Promise，不过大部分场景下并行处理效率更高～

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