在不调用“then”的情况下解决 Promise

Question

我有这段代码，它是我正在为一个名为 Poolio 的 NPM 模块编写的小 API 的一部分。对于那些支持错误优先回调和承诺的人来说，我的问题似乎是一个常见问题——我们如何在保持一致的 API 和一致的 API 返回值的同时支持这两者？例如，如果我有条件地从我的 API 返回一个 Promise，这取决于我的 lib 的使用者是否提供回调，这在我看来有点尴尬。

lib 的使用者可以提供回调或使用 Promise then 函数，但不能同时提供。

这是我的库导出的一个函数，我想承诺：

Pool.prototype.any = function (msg, cb) {

    var workId = this.counter++;
    var self = this;

    return new Promise(function (resolve, reject) {

        if (typeof cb === 'function') {
            self.resolutions.push({
                workId: workId,
                cb: cb
            });
        }
        else {
            self.resolutions.push({
                workId: workId,
                resolve: resolve,
                reject: reject
            });
        }

        if (this.available.length > 0) {
            var cp = this.available.shift();
            cp.workId = workId;
            cp.send(msg);
        }
        else {
            self.msgQueue.push({
                workId: workId,
                msg: msg
            });
        }
    });

};

我的问题是 - 如果用户在原始函数参数中提供回调函数，我如何在不调用“then”的情况下解决承诺？ 抱歉很难解释，但希望你能理解。

还有一个有趣的问题： Do never resolved promises cause memory leak?

Answer 1

实际上非常简单。只有你可能错过了它，因为它隐藏在一堆乱七八糟的代码中。

基本上你这样做：

var promise = new Promise(function (resolve, reject) { /*....*/});

if (typeof cb === 'function') {
    promise.then(cb);
} else {
    return promise;
}

Answer 2

实际上，这是 API 做的很常见的事情 (mongodb-driver example)。基本上，编写一个接受回调的私有函数，编写一个公共函数检查 cb 并在必要时编写它。使用 github 中的代码（_any 可能需要重构，例如，您不需要检查 cb 是否是一个函数，也许还有其他东西）：

 // private function
var _any = function(msg, cb) {
  if (this.kill) {
    console.log('warning: pool.any called on pool of dead/dying workers');
    return;
  }

  debug('current available pool size for pool_id ' + this.pool_id + ' is: ' + this.available.length);
  var workId = this.counter++;

  if (typeof cb === 'function') {
    this.resolutions.push({
      workId: workId,
      cb: cb
    });
  } else {
    workId = -1;
  }

  if (this.available.length > 0) {
    var cp = this.available.shift();
    cp.workId = workId;
    cp.send(msg);
  } else {
    this.msgQueue.push({
      workId: workId,
      msg: msg
    });
  }
};

 // public exposed function
Pool.prototype.any = function(msg, cb) {
  if (typeof cb === 'function') {
    // cb is provided, no action is required here
    return _any(msg, cb);
  } 

  // no cb, wrap the call inside a Promise and provide a cb
  return new Promise(function(resolve, reject) {
    _any(msg, function(err, data) {
      if (err) reject(err);
      else resolve(data);
    });
  });
}