bluebirdjspromise封装在for循环中

Question

我有一堆函数用来向我的服务提供数据。我想循环遍历它们中的每一个，并在其中一个返回所需结果时立即停止。如果第一个工作，那很好。如果它有异常或数据无效，我想移动到下一个等等。

我该如何实现这一点？我有下面的代码：

handleData: function(address) {
  var self = this;
  return new Promise(function (resolve, reject) {
    for (var i = 0; i < self.listAllAvailableProviders.length; ++i) {
      var handler = self.listAllAvailableProviders[i];
      new handler().getData(address)
        .then(function(value) {
          Logger.info(value);
          resolve(value);
        })
        .catch(function(err){
          Logger.error(err);
        })
    }
    reject("");
  });
}

如何修复它，使其在第一个获得正确数据后立即停止？我已经通读了bluebirdjs文档，但没有结果。

编辑：我把一个突破语句放在解决之后，我得到了这个：

SyntaxError: Illegal break statement
at Object.exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:513:28)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
at Module.require (module.js:468:17)
at require (internal/module.js:20:19)

Answer 1

如果promise不是硬约束，那么caolan/async#eachSeries或类似产品可能会有所帮助。类似于。。。

// var Promise = require(?)
// var async = require("async")
handleData: asyncProviderFinder

...

function asyncProviderFinder(address){
  var self = this;
  return new Promise(function(resolve, reject){
    async.eachSeries(
      self.listAllAvailableProviders, 
      function iterate(provider, next){
        var handler = provider;
        new handler().getData(address)
        .then( function(value){
          Logger.info(value);
          next("abort"); // callback any error to abort future iterations
          return resolve(value);
        })
        .catch( function (err){
          Logger.error(err);
          next();
        });
      },
      function callback(err, firstProvider){
        if ((firstProvider === undefined) && !err ){ reject(""); }
      }
    );
  });
}

Answer 2

您正在for循环中并行运行所有请求，因此当您找到一个具有您喜欢的值的请求时，其他请求已经启动，因此无法“不”运行它们。如果你不想在找到一个后再运行其他的，你就不需要同时启动它们。因此，这将引导您进入一个设计模式，在该模式中序列化请求。运行一个，如果不成功，运行下一个，依此类推。

据我所知，蓝鸟并没有一个内置的计划来满足你的要求。我能想到的最简单的事情是使用Bluebird中的一个数组处理函数，该函数将一个接一个地序列化请求，例如Promise.mapSeries（），然后在找到合适的值时使用拒绝来中止处理。

handleData: function(address) {
    return Promise.mapSeries(this.listAllAvailableProviders, function(handler) {
        return new handler().getData(address).then(function(value) {
            // the first success we get, we will throw with 
            // the returned value in order to stop the .mapSeries progression
            throw value;
        }, function(err) {
            // log the error, but don't let the rejection propagate so other handlers are called
            Logger.error(err);
        })
    }).then(function() {
        // nothing succeeded here, turn it into an overall rejection
        throw new Error("No getData() handlers succeeded");        
    }, function(val) {
        // reject here means we got a good value so turn it into a resolved value
        return val;
    })
}


// usage
obj.handleData().then(function(val) {
    // got value here
}).catch(function(err) {
    // no values here
});

奇怪的是，如果您自己迭代处理程序，它似乎代码更少，而且可能更简单：

handleData: function(address) {
    var index = 0;
    var handlers = this.listAllAvailableProviders;
    var handlerCnt = handlers.length;

    function next() {
        if (index < handlerCnt) {
            var handler = handlers[index++];
            return new handler().getData(address).catch(next);
        } else {
            return Promise.reject(new Error("No handler found for address"));
        }
    }
    return next();
}