一个承诺可以解决两次吗？

Question

我有这样的功能：

async howLongToMyBirthdayDate(date) {
    return await new Promise((resolve, reject) => {
      let bday;
      if (!(date instanceof Date)) {
        if (Number.isInteger(date) && date > 0) {
          bday = new Date(date);
        } else {
          setTimeout(() => reject(new Error('Wrong argument!')), 1000);
          return;
        }
      } else {
        bday = date;
      }

      const today = new Date();
      today.setHours(0, 0, 0, 0);

      bday.setFullYear(today.getFullYear());
      bday.setHours(0, 0, 0, 0);

      if (bday - today === 0) {
        setTimeout(() => resolve(this.congratulateWithBirthday()), 1000);
        
      }

      setTimeout(
        () =>
          resolve(this.notifyWaitingTime(Math.ceil((bday - today) / 86400000))),
        1000
      );
    });
  }

承诺解决了两次。我看到函数 congratulateWithBirthday 和 notifyWaitingTime 的结果。正常吗？我认为承诺只能被解决或拒绝一次 - 通过第一次调用解决或拒绝回调。也许 setTimeout 会改变它的行为？谁能给我解释一下？

Answer 1

您的承诺不需要解决两次即可获得您所描述的行为。您正在直接调用这两个函数，所以它们当然会被调用。对resolve 的第二次调用是否执行任何操作无关。

这一行

resolve(this.notifyWaitingTime(Math.ceil((bday - today) / 86400000))),

不是说“解析然后运行这个函数”，而是说“调用这个函数，然后用它的返回值调用resolve”。

编写此代码的更合适的方式可能是这样的：

async howLongToMyBirthdayDate(date) {
    const result = await new Promise((resolve, reject) => {
      let bday;
      if (!(date instanceof Date)) {
        if (Number.isInteger(date) && date > 0) {
          bday = new Date(date);
        } else {
          setTimeout(() => reject(new Error('Wrong argument!')), 1000);
          return;
        }
      } else {
        bday = date;
      }

      const today = new Date();
      today.setHours(0, 0, 0, 0);

      bday.setFullYear(today.getFullYear());
      bday.setHours(0, 0, 0, 0);

      // resolve with how long until bday
      if (bday - today === 0) {
        setTimeout(() => resolve(0), 1000);
        
      }

      // we just resolve with how long until bday
      setTimeout(
        () =>
          resolve(Math.ceil((bday - today) / 86400000)),
        1000
      );
    });

  if (result === 0) { this.congratulateWithBirthday(); }
  else if (result > 0) { this.notifyWaitingTime(result); }
  }

不是 100% 肯定这解决了您的问题，因为我看不到您调用的函数是做什么的，但我从您 (ab) 使用 resolve 的方式的猜测是，您基本上想调用两个函数之一计算后的函数取决于 bday 是否现在。

Answer 2

简短的回答，是的，鉴于您的代码，它将解析两次。这样做的原因是因为congratulateWithBirthday 和notifyWaitingTime 没有相互制约，所以它们都会在同一次运行中被调用。

如果你想避免这种情况，你应该有类似的东西

if (bday - today === 0) {
  setTimeout(() => resolve(this.congratulateWithBirthday()), 1000);
} else {
  setTimeout(() => resolve(this.notifyWaitingTime(Math.ceil((bday - today) / 86400000))), 1000);
}

另外，作为旁注，请尝试在使用async/await 或Promise 之间做出决定。没有理由同时使用这两种方法，这会使代码看起来很奇怪。