第一:你的代码从根本上来说是有缺陷的
为什么我说根本上有缺陷是因为你似乎误解了如何functions
work
return Promise.map(nameList, function(personName){ <-- this function
for (var j=1; j<=10; j++){
return promises.push(new Promise(function(resolve, reject) { <-- is returning HERE
Here Array.prototype.push
函数返回index
新添加的项目。 (我查了一下,直到)
这里最大的问题是你回归 it.
只要你return
the mapper
的函数Promise.map http://bluebirdjs.com/docs/api/promise.map.html你告诉它你完成了!所以在上面的代码中,它甚至不会进入下一次迭代for loop
在 MDN 中查找规范return https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/return它说
return 语句结束函数执行并指定要返回给函数调用者的值。
希望这能回答您关于提前返回哪里的问题。
第二:我不会指望你的承诺
我自己也很难兑现诺言。让我引导您前往关于承诺的精彩文章 https://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html我最近读过。现在让我应用我从中学到的东西。谢谢你提出这个问题。
我们要兑现的承诺回调地狱 http://callbackhell.com/但你却把它的用途全部串起来了。
我会尝试从最嵌套的部分开始压平承诺
SOME_SEARCH_FX_THAT_RETURNS_A_PROMISE(params).then(function(data){
return Promise.map(data, function(item, index){
if (!_.has(jsonDict, item.key)){
jsonDict[item.key] = {
name: personName
}
}
return
}).then(function(){
console.log(jsonDict)
return resolve(true)
})
}).catch(function(err){
console.log(err)
return reject(false)
})
这里的内部承诺图是不必要的,你可以只使用标准map
即使是一个for loop
因为你实际上并没有在这里映射任何东西......
[重构1]
// Create subroutine, careful with the globals...
let populateJsonDict = singleData => {
if (!_.has(jsonDict, singleData.key)) jsonDict[singleData.key] = { name: personName }
}
SOME_SEARCH_FX_THAT_RETURNS_A_PROMISE(params).then(data => {
data.forEach(populateJsonDict);
resolve(true); // this will be removed later
}).catch(function(err){
console.log(err);
reject(false); // this will be removed later
})
好吧,让我们向上移动回调金字塔(或向下移动?)。列表中的下一个
for (var j=1; j<=10; j++){
return promises.push(new Promise(function(resolve, reject) {
params['page'] = j;
console.log('PAGE ' + j + ' ' + personName)
//[Refactored 1]
}))
}
这里又是一个不必要的承诺,是时候摆脱它了。这里问题的基石是return
[重构2]
for (var j=1; j<=10; j++){
//[from Refactored 1]
let populateJsonDict = singleData => {
if (!_.has(jsonDict, singleData.key)) jsonDict[singleData.key] = { name: personName }
}
params['page'] = j; // I assume this a global somewhere
let p = SOME_SEARCH_FX_THAT_RETURNS_A_PROMISE(params).then(data => {
data.forEach(populateJsonDict);
// Removed because we've removed the parent promise
}).catch(function(err){
console.log(err);
// Removed because we've removed the parent promise
})
promises.push(p)
}
我会快进,因为我注意到这已经变得很长了。下一个要重构的部分
return new Promise((res, rej) => {
var promises = [];
return Promise.map(nameList, function(personName){
//[Refactored 2]
}).then(function(){
return res(true)
})
}).then(function(){
console.log('Done with everything.')
})
我真的不知道如何挽救这个,所以我会写一些集体的东西。
[重构3:最终]
var promises = [];
nameList.forEach(personName => { // Like earlier, redundant Promise.map
//[from Refactored 2]
for (var j=1; j<=10; j++){
let populateJsonDict = singleData => {
if (!_.has(jsonDict, singleData.key)) jsonDict[singleData.key] = { name: personName }
}
params['page'] = j;
let p = SOME_SEARCH_FX_THAT_RETURNS_A_PROMISE(params).then(data => {
data.forEach(populateJsonDict);
}).catch(function(err){
console.log(err);
})
promises.push(p)
}
});
// At this point you have an array of Promises, for this we can utilize Promise.all
Promise.all(promises)
.then(() => console.log('Done with Everything'));
我想这本来可以做得更好。让我做最后一个版本。
[重构3.1:最终版]
let populateJsonDict = name => key => !_.has(jsonDict, key) && Object.assign(jsonDict, {[key]:name};
let promises = _.times(10,
index => {
params['page'] = index+1;
return Promise.map(nameList, name => {
let populateWithName = populateJsonDict(name);
let iterate = data => data.forEach(populateWithName);
return SOME_SEARCH_FX_THAT_RETURNS_A_PROMISE(params)
.then(iterate)
.catch(err => console.log(err));
});
});
Promise.all(promises)
.then(() => console.log('Everything done'));
好吧,仍然有这种不满意的感觉,但这就是我现在所拥有的……这对我来说确实比对你更重要。所以再次感谢你的朋友。我希望我们双方都能在这个不断变化的工作领域继续前进。如果这一切听起来有点居高临下,我深表歉意。干杯!