Question

我正在使用Promises编写我的第一段代码，并且得到了一些意想不到的结果。我有一些看起来像这样的代码（使用jQuery）：

$('.loading-spinner').show();
$('.elements').replaceWith(function() {
   // Blocking code to generate and return a replacement element
});
$('.newElements').blockingFunction();
$('.loading-spinner').hide();

为防止在运行此代码时页面被阻止，我尝试使用setTimeout和Promises使其异步，如下所示：

$('.loading-spinner').show();
var promises = [];
var promises2 = [];
$('.elements').each(function(i, el){
    promises[i] = new Promise(function(resolve, reject) {
        setTimeout(function() {
            $(el).replaceWith(function() {
                // Code to generate and return a replacement element
            });
            resolve(true);
        }, 100);
    });
});
Promise.all(promises).then(function(values) {
    $('.newElements').each(function(i, el) {
        promises2[i] = new Promise(function(resolve, reject) {
            setTimeout(function() {
                $(el).blockingFunction();
                resolve(true);
            }, 100);
        });
    });
});
Promise.all(promises2).then(function(values) {
    $('.loading-spinner').hide();
});

我想要实现的是，一旦promises中的Promise得到解决，promises2中的Promise就会被实例化。解决这些问题后，隐藏加载微调器。

我得到的效果是，虽然页面没有被阻止很长时间，但是一旦设置了所有Promise，spinner就会消失，而不是等到它们被解决。

我可以看到promises2承诺在promises中的所有内容都解决后才解决，所以我不明白为什么会发生这种情况。我想这可能是因为我没有正确理解Promise，或者没有低估使代码异步。

Answer 1

你在Promise.all上调用promises2之前填充它，实际上当你调用它时它包含一个空数组，因此它在一个空数组上调用Promise.all，从而解决它立即无需等待promises中的承诺。

快速修复：

function delay(ms){ // quick promisified delay function
    return new Promise(function(r){ setTimeout(r,ms);});
}

var promises = $('.elements').map(function(i, el){
    return delay(100).then(function(){
        $(el).replaceWith(function() {
            // Code to generate and return a replacement element
        });
});
Promises.all(promises).then(function(els){
    var ps = $('.newElements').map(function(i, el) {
        return delay(100).then(function(){ 
            $(el).blockingFunction();
        });
    });
    return Promise.all(ps);
}).then(function(){
    $('.loading-spinner').hide();
});

我们可以做得更好，没有理由为n元素触发n超时：

delay(100).then(function(){
    $(".elements").each(function(i,el){ 
        $(el).replaceWith(function(){ /* code to generate element */});
    });
}).
then(function(){ return delay(100); }).
then(function(){
    $('.newElements').each(function(i, el) { $(el).blockingFunction(); });
}).then(function(){
    $('.loading-spinner').hide();
}).catch(function(err){
   throw err;
});

Promise.all比预期更早解决

1 个答案: