問(wèn)題描述

想要callback在findById之后執(zhí)行，但又不能放在它的回調(diào)中，不然會(huì)執(zhí)行多次。應(yīng)該怎么辦？

問(wèn)題解答

軟件是服務(wù)行業(yè)，要有服務(wù)意識(shí)……代碼貼圖沒(méi)問(wèn)題，可以看得很清晰，但是如果回答者想改你的代碼，還得重新敲一遍…………

我看你已經(jīng)用了 es6 的語(yǔ)法，所以正解肯定是 Promise，可以自己封裝成 Promies

exports.findList = function(findObj, callback) { Info.find(findObj, function(err, info) {const promises = info.map(f => new Promise((resolve, reject) => { f.image = f.images.split(',')[0]; User.findById(f.author_id, function(error, user) {if (error) { reject(error); return;}f.author_name = user.name;f.authro_avatar = user.avatar;resolve(f); });}));Promise.all(promises) .then(function(values) {// 成功的時(shí)候，這個(gè) values 是所有 info 對(duì)象，// 作為一個(gè)數(shù)組返回出來(lái)，而不是某一個(gè)callback(null, values); }) .catch(function(error) {// 注意這里 error 是第一個(gè)失敗 error// 不是所有的 error callback(error); }); });};

當(dāng)然也可以用 Bluebird 提供的工具函數(shù)來(lái)封裝 Promise

需要注意的是你這里運(yùn)行了好多個(gè)異步調(diào)用，所以要一起返回的話，肯定也是好多個(gè)結(jié)果。所以注意 Promise.all 后面 then() 中的回調(diào)。

如果你要用 es5，建議你寫成 es6 再 Babel。不過(guò) Node 7.6+ 已經(jīng)支持 es2017 的一些特性，所以其實(shí)不必在意這個(gè)問(wèn)題。如果確實(shí)需要，就采用在 findById 回調(diào)中記數(shù)的辦法，記滿放大招(調(diào)用 callback)。

方案一

使用 async/await。這樣就可以按照同步方式使用 User.findById。

方案二

在循環(huán)中把 User.findById 加入一個(gè)數(shù)組，然后用 Promise.all，callback 寫在 Promise.all().then() 中。

方案三

使用一個(gè)狀態(tài)變量，每次檢查一下，滿足條件再執(zhí)行 callback。丑陋的方法，不要用。

設(shè)置個(gè)參數(shù)，執(zhí)行完設(shè)置下，以后每次進(jìn)入回調(diào)判斷下 ..

對(duì)于callback地獄，請(qǐng)使用Promise 或者類似類庫(kù)。

exports.findList=function (findObj,callback) { Info.find(findObj,function (err,info) {for(let i in info){ info[i].image=info[i].images.split(’,’)[0]; User.findById(info[i].author_id,function (error,user) {info[i].author_name=user.name;info[i].author_avatar=user.avatar;if(i==info.length-1){ callback(err,info);} })} })}

我的解決方法，有點(diǎn)low。。？

二樓說(shuō)得對(duì)，用promise解決。