2013-08-22 6 views
8

У меня с трудом выполняются обещания последовательно.Как последовательно выполнять обещания с Q в Javascript?

var getDelayedString = function(string) { 
    var deferred = Q.defer(); 

    setTimeout(function() { 
     document.write(string+" "); 
     deferred.resolve(); 
    }, 500); 

    return deferred.promise; 
}; 

var onceUponATime = function() { 
    var strings = ["Once", "upon", "a", "time"]; 

    var promiseFuncs = []; 

    strings.forEach(function(str) { 
     promiseFuncs.push(getDelayedString(str)); 
    }); 

    //return promiseFuncs.reduce(Q.when, Q()); 
    return promiseFuncs.reduce(function (soFar, f) { 
     return soFar.then(f); 
    }, Q());  
}; 

getDelayedString("Hello") 
.then(function() { 
    return getDelayedString("world!") 
}) 
.then(function() { 
    return onceUponATime(); 
}) 
.then(function() { 
    return getDelayedString("there was a guy and then he fell.") 
}) 
.then(function() { 
    return getDelayedString("The End!") 
}) 

onceUponATime() должны последовательно вывод [ «Когда», «на», «а», «время»], но вместо этого они выводятся сразу же по какой-то причине.

jsFiddle здесь: http://jsfiddle.net/6Du42/2/

Любая идея, что я делаю неправильно?

ответ

15

, но вместо этого они выводятся немедленно по какой-либо причине.

Вы вызываете их уже здесь:

promiseFuncs.push(getDelayedString(str)); 
//        ^^^^^ 

Вам нужно будет нажать function(){ return getDelayedString(str); }. Btw, вместо использования нажатия на массив в цикле each вы скорее должны использовать map. А на самом деле вы на самом деле не нужно, что в любом случае, но может reduce над strings массив непосредственно:

function onceUponATime() { 
    var strings = ["Once", "upon", "a", "time"]; 

    return strings.reduce(function (soFar, s) { 
     return soFar.then(function() { 
      return getDelayedString(s); 
     }); 
    }, Q());  
} 

О, и don't use document.write.

+0

Спасибо за ответ и дополнительные подсказки! – Nick

+1

Да, это должна быть функция * factory * для работы, иначе она выполняется немедленно. – FlavorScape