用promise包装promise是一种反模式吗？

Question

我正在尝试使用Promise API编写重新连接到超时数据库的代码。

我最后做的是把连接DB的promise包装成一个promise，但我不确定这是否是最好的方式。我想可能有一种方法可以使用最初的promise，从尝试连接到数据库，但我无法找到它。

function connect(resolve) {
  console.log('Connecting to db...');
  MongoClient.connect(url, { promiseLibrary: Promise })
    .then((db) => resolve(db))
    .catch((err) => {
      console.log('db connection failed!:\n', err);
      if (retry++ < 3) {
        console.log('Trying again...');
        setTimeout(() => connect(resolve), 5000);
      } else {
        console.log('Retry limit reached!');
      }
    });
}

module.exports = new Promise(connect);

我认为没有setTimeout块是可能的，但我无法解决它。

Answer 1

这里有一个更通用的解决方案（非阳性测试）：

function withRetry(asyncAction, retries) {
  if (retries <= 0) {
    // Promise.resolve to convert sync throws into rejections.
    return Promise.resolve().then(asyncAction); 
  }
  return Promise.resolve()
    .then(asyncAction)
    .catch(() => withRetry(asyncAction, retries - 1));
}

此函数将接受一个返回promise和多次重试的函数，如果promise被拒绝，则重试该函数的次数与重试次数相同。

如果问题解决，重试链将停止。

就您而言：

let connectionPromise = withRetry(connect, 3); // connect with 3 retries if fails.