简单承诺队列:q.all在延期承诺解决之前解决

时间:2015-04-16 15:44:58

标签: node.js queue promise q

我努力完全掌握承诺的下一部分......

我正在尝试创建一个简单的promise队列(长期目标是限制数据库上的查询),然后我可以使用Q.all()和Array.protoype.map()。

(这似乎与this question有关,但我没有看到明确的决议。)

这是我的简单框架:

var Q = require('q');

var queue = [];
var counter = 0;
var throttle = 2;  // i can do things at most two at a time

var addToQueue = function(data) {
    var deferred = Q.defer();
    queue.push({data: data, promise: deferred});
    processQueue();
    return(deferred.promise);
}

var processQueue = function() {
    if(queue.length > 0 && counter < throttle) {
       counter++;
       var item = queue.shift();
       setTimeout(function() {  // simulate long running async process
          console.log("Processed data item:" + item.data);
          item.promise.resolve();
          counter--;
          if(queue.length > 0 && counter < throttle) {
             processQueue(); // on to next item in queue
          }
       }, 1000);
    }
}

data = [1,2,3,4,5];
Q.all(data.map(addToQueue))
    .then(console.log("Why did get here before promises all fulfilled?"))
    .done(function() {
        console.log("Now we are really done with all the promises.");
    });

但是,如上所述,&#34;然后&#34;立刻被召唤,只有&#34;完成&#34;推迟到所有承诺的解决。我在api documentation中注意到,唯一的例子确实使用.done()而不是then()。那么也许这是预期的行为?问题是,我无法链接其他操作。在这种情况下,我需要创建另一个延期保证并在Q.all的done函数中解决它,如下所示

data = [1,2,3,4,5];

var deferred = Q.defer();

deferred.promise
    .then(function() {
        console.log("All data processed and chained function called.");
    })  // could chain additional actions here as needed.

Q.all(data.map(addToQueue))
    .done(function() {
        console.log("Now we are really done with all the promises.");
        deferred.resolve();
    });

这可以按照需要运行,但额外的步骤让我觉得我必须遗漏一些关于如何正确使用Q.all()的内容。

我使用Q.all()是否有问题,或者上面的额外步骤实际上是正确的方法吗?

编辑:

Tyrsius指出我的论点.then不是对函数的引用,而是一个立即求值的函数(console.log(...))。我应该怎么做:

Q.all(data.map(addToQueue))
    .then(function() { console.log("Ahhh...deferred execution as expected.")})

1 个答案:

答案 0 :(得分:4)

实际上你的问题是标准的Javascript。

Q.all(data.map(addToQueue))
    .then(console.log("Why did get here before promises all fulfilled?"))
    .done(function() {
        console.log("Now we are really done with all the promises.");
    });

仔细观察第二行。 Console.log立即被评估并作为参数发送给.then。这与promises无关,它只是javascript如何解析函数调用。你需要这个

Q.all(data.map(addToQueue))
        .then(function() { console.log("Why did get here before promises all fulfilled?")})
        .done(function() {
            console.log("Now we are really done with all the promises.");
        });

修改

如果这是你做了很多事情,你可以制作一个按照你想要的方式运作的功能返回功能

function log(data) {
    return function() { console.log(data);}
}

Q.all(data.map(addToQueue))
        .then(log("Why did get here before promises all fulfilled?"))
        .done(function() {
            console.log("Now we are really done with all the promises.");
        });