简化承诺链

Question

我试图通过让函数处理一组函数来尽可能简化 Promise 链。它们需要按顺序排列，但我为它们提供正确语法的方法不起作用，我确信有更好的方法来做到这一点。链中的每个函数都调用数据库，需要在返回时触发下一个函数。

// Run each in order.
var Chain = function(chain, cb){
    chain.forEach(function(i){  i = function(r){return new Promise(i)};   });
    chain.reduce(function(cur, next) { {cur.then(next)} }, Promise.resolve()).then(function() { cb() });
}
Chain([

    r=>{ 
        UserData('elle',  i=>{ 
            console.log(i);
            r(i)
        }) 
    },
    r=>{ 
        UserData('tyler', {age:"22"}, i=>{ 
            console.log(i);
            r(i);
        }) 
    },
    r=>{ 
        UserData('nerd',  i=>{ 
            console.log(i);
            r(i) 
        }) 
    },
    r=>{ 
        UserData('jeax',  i=>{ 
            console.log(i);
            r(i) 
        }) 
    }

], function(){

    console.log("Done.");

});

下面的示例按我的需要工作，只是没有使用我想要的链功能和每个项目的简单性。

var PChain = function(cb){
    // Convert each item.
    return function(){ return new Promise(r=>{cb(r)}) };
}
// The items.
var chain = [
    PChain(r=>{ 
        UserData('elle',  i=>{ 
            console.log(i);r(i)
        }) 
    }),
    PChain(r=>{ 
        UserData('tyler', {age:"22"}, i=>{ 
            console.log(i);r(i);
        }) 
    }),
    PChain(r=>{ 
        UserData('nerd',  
            i=>{ console.log(i);r(i) 
        }) 
    }),
    PChain(r=>{ 
        UserData('jeax',  
            i=>{ console.log(i);r(i) 
        }) 
    })
];
chain.reduce(function(cur, next) { return cur.then(next) }, Promise.resolve()).then(function() {
    //all executed
    console.log("Done.");
});

Answer 1

你的错误在于Chain函数：

chain.forEach(function(i){  i = function(r){return new Promise(i)};   });
chain.reduce(function(cur, next) { {cur.then(next)} }, Promise.resolve()).then(function() { cb() });

具体来说，在forEach 中分配给i 绝对没有任何作用。你真正想要的是使用map:

chain.map(function(exec){ return function(r){ return new Promise(exec)}; }).reduce(…);

map 返回一个函数数组，然后您可以对其进行归约。

---

不管怎样，你实际上应该做的是promisifyUserData函数：

function user(name, options) {
    return new Promise((resolve, reject) => {
        if (options)
            UserData(name, options, resolve);
        else
            UserData(name, resolve);
    });
}

你甚至不需要链，但可以简单地写

user('elle').then(i => { 
    console.log(i);
    return user('tyler', {age:"22"});
}).then(i => { 
    console.log(i);
    return user('nerd');
}).then(i => {
    console.log(i);
    return user('jeax');
}).then(i => {
    console.log(i);
}).then(function() {
   //all executed
   console.log("Done.");
});

如果您有一个动态大小的用户名数组，您仍然可以使用reduce，但您不应该使用函数数组。

Answer 2

你应该使用Promise.all(iterable)：

Promise.all([r1, r2, r3]).then(function(values) { 
  // Do something
});

更新

要遵守特定顺序，您应该check this answer。不幸的是，没有 ES6 方法能够原生地做到这一点。

最常用的 JS 承诺队列系统之一（除其他外，by Angular）是 Kris Kowal 的 Q，它确实是 recommend something similar，所以这与您所写的相差不远。

注意

在您的代码中，您似乎正在尝试实现某种批处理请求。你要知道，在 JS 中真正需要一个有序的 Promise 链是非常稀缺的。

1. 如果您将其用于流（如文件内容操作），最好使用pipes。
2. 如果您将其用于批处理/过滤 API 请求：
   • 如果您控制 API：更改您的 API 以管理批处理/过滤请求
   • 如果您无法控制它：创建一个中间件应用程序来管理它