如何解决递归异步承诺？

我正在玩弄承诺，并且在异步递归承诺方面遇到了麻烦。

场景是一名运动员开始跑 100m，我需要定期检查他们是否完成，一旦完成，打印他们的时间。

编辑以澄清 :

在现实世界中，运动员在服务器上运行。startRunning涉及对服务器进行 ajax 调用。checkIsFinished还涉及对服务器进行 ajax 调用。下面的代码是尝试模仿这一点。代码中的时间和距离是硬编码的，以使事情尽可能简单。抱歉没有更清楚。

End edit

我希望能够写出以下内容

startRunning()
  .then(checkIsFinished)
  .then(printTime)
  .catch(handleError)

where

var intervalID;
var startRunning = function () {
  var athlete = {
    timeTaken: 0,
    distanceTravelled: 0
  };
  var updateAthlete = function () {
    athlete.distanceTravelled += 25;
    athlete.timeTaken += 2.5;
    console.log("updated athlete", athlete)
  }

  intervalID = setInterval(updateAthlete, 2500);

  return new Promise(function (resolve, reject) {
    setTimeout(resolve.bind(null, athlete), 2000);
  })
};

var checkIsFinished = function (athlete) {
  return new Promise(function (resolve, reject) {
    if (athlete.distanceTravelled >= 100) {
      clearInterval(intervalID);
      console.log("finished");
      resolve(athlete);

    } else {
      console.log("not finished yet, check again in a bit");
      setTimeout(checkIsFinished.bind(null, athlete), 1000);
    }    
  });
};

var printTime = function (athlete) {
  console.log('printing time', athlete.timeTaken);
};

var handleError = function (e) { console.log(e); };

我可以看到第一次创建的承诺checkIsFinished从未得到解决。我怎样才能确保这个承诺得到履行，以便printTime叫做？

代替

resolve(athlete);

我可以

Promise.resolve(athlete).then(printTime);

但如果可能的话，我想避免这种情况，我真的很希望能够写

startRunning()
  .then(checkIsFinished)
  .then(printTime)
  .catch(handleError)

错误在于您正在传递一个返回承诺的函数setTimeout。这个承诺已经消失在空中。一个创可贴修复可能是在执行器函数上递归：

var checkIsFinished = function (athlete) {
  return new Promise(function executor(resolve) {
    if (athlete.distanceTravelled >= 100) {
      clearInterval(intervalID);
      console.log("finished");
      resolve(athlete);
    } else {
      console.log("not finished yet, check again in a bit");
      setTimeout(executor.bind(null, resolve), 1000);
    }    
  });
};

但是嗯。我认为这是一个很好的例子，说明为什么人们应该避免承诺构造函数反模式 https://stackoverflow.com/q/23803743/1048572（因为混合承诺代码和非承诺代码不可避免地会导致这样的错误）。

我遵循的避免此类错误的最佳实践：

1. 只处理返回 Promise 的异步函数。
2. 当不返回 Promise 时，用 Promise 构造函数包装它。
3. 尽可能窄地包装它（使用尽可能少的代码）。
4. 不要将 Promise 构造函数用于其他用途。

在此之后，我发现代码更容易推理，更难出错，因为一切都遵循相同的模式。

将其应用到您的示例中（为了简洁起见，我使用 es6 箭头函数。它们在 Firefox 和 Chrome 45 中工作）：

var console = { log: msg => div.innerHTML += msg + "<br>",
                error: e => console.log(e +", "+ e.lineNumber) };

var wait = ms => new Promise(resolve => setTimeout(resolve, ms));

var startRunning = () => {
  var athlete = {
    timeTaken: 0,
    distanceTravelled: 0,
    intervalID: setInterval(() => {
      athlete.distanceTravelled += 25;
      athlete.timeTaken += 2.5;
      console.log("updated athlete ");
    }, 2500)
  };
  return wait(2000).then(() => athlete);
};

var checkIsFinished = athlete => {
  if (athlete.distanceTravelled < 100) {
    console.log("not finished yet, check again in a bit");
    return wait(1000).then(() => checkIsFinished(athlete));
  }
  clearInterval(athlete.intervalID);
  console.log("finished");
  return athlete;
};

startRunning()
  .then(checkIsFinished)
  .then(athlete => console.log('printing time: ' + athlete.timeTaken))
  .catch(console.error);

<div id="div"></div>

注意checkIsFinished返回运动员或承诺。这在这里很好，因为.then函数会自动提升传递给 Promise 的函数的返回值。如果你会打电话checkIsFinished在其他情况下，您可能想自己进行促销，使用return Promise.resolve(athlete);代替return athlete;.

编辑回应阿米特的评论:

对于非递归答案，替换整个checkIsFinished使用这个助手的功能：

var waitUntil = (func, ms) => new Promise((resolve, reject) => {
  var interval = setInterval(() => {
    try { func() && resolve(clearInterval(interval)); } catch (e) { reject(e); }
  }, ms);
});

然后这样做：

var athlete;
startRunning()
  .then(result => (athlete = result))
  .then(() => waitUntil(() => athlete.distanceTravelled >= 100, 1000))
  .then(() => {
    console.log('finished. printing time: ' + athlete.timeTaken);
    clearInterval(athlete.intervalID);
  })
  .catch(console.error);

var console = { log: msg => div.innerHTML += msg + "<br>",
                error: e => console.log(e +", "+ e.lineNumber) };

var wait = ms => new Promise(resolve => setTimeout(resolve, ms));

var waitUntil = (func, ms) => new Promise((resolve, reject) => {
  var interval = setInterval(() => {
    try { func() && resolve(clearInterval(interval)); } catch (e) { reject(e); }
  }, ms);
});

var startRunning = () => {
  var athlete = {
    timeTaken: 0,
    distanceTravelled: 0,
    intervalID: setInterval(() => {
      athlete.distanceTravelled += 25;
      athlete.timeTaken += 2.5;
      console.log("updated athlete ");
    }, 2500)
  };
  return wait(2000).then(() => athlete);
};

var athlete;
startRunning()
  .then(result => (athlete = result))
  .then(() => waitUntil(() => athlete.distanceTravelled >= 100, 1000))
  .then(() => {
    console.log('finished. printing time: ' + athlete.timeTaken);
    clearInterval(athlete.intervalID);
  })
  .catch(console.error);

<div id="div"></div>