正确的方式重复出现Q.Promise

Question

我做了这样的事情。这似乎对我有用，但可以吗？有更好的方法吗？

function myPromise(options) {
  return Q.Promise(function(resolve, reject, notify) {
    doSomthingAsync(options, function(resp) {
       notify(resp);
      if (resp.nextPageToken) {
        options.pageToken = resp.nextPageToken;
        myPromise(options).then(resolve, reject, notify);
      } else {
        resolve(resp);
      }
    });
  });
}

注意：我知道可变选项是不明智的。

仅供参考：我不确定，但在ReactiveCocoa中，也有同样的功能。-[RACSignal subscribe:]https://github.com/ReactiveCocoa/ReactiveCocoa/blob/1e97af8f5681b3685770eb30faf090e64c293290/ReactiveCocoaFramework/ReactiveCocoa/RACSignal.h#L115-L131

Answer 1

有更好的方法吗？

是的。.then(resolve, reject, notify);是延迟的反模式。

更好地将可能的递归调用与显式then:链接起来

function myPromise(options) {
  return Q.Promise(function(resolve, reject, notify) {
    doSomthingAsync(options, function(resp) {
      // if (err) return reject(err);
      notify(resp);
      resolve(resp);
    });
  }).then(function(resp) {
    if (resp.nextPageToken) {
      options.pageToken = resp.nextPageToken;
      return myPromise(options);
    } else {
      return resp;
    }
  });
}

您可能可以使用其中一个回调函数helper方法来代替new Q.Promise调用。