给定以下函数,我得到警告:
警告 避免嵌套承诺承诺/不嵌套(第 6 行(
我应该如何重组函数以修复警告?
function FindNearbyJobs(uid, lat, lng){
return admin.database().ref(`users/${uid}/nearbyjobs`).remove().then(data => {
return new Promise((resolve, reject) => {
const geoQueryJobs = geoFireJobs.query({center: [lat, lng], radius: 3 });
geoQueryJobs.on("key_entered", (key, location, distance) => {
return Promise.all([admin.database().ref(`jobs/${key}/category`).once('value'), admin.database().ref(`users/${uid}/account/c`).once('value')]).then(r => {
const cP = r[0];
const cO = r[1];
if (cO.val().includes(cP.val())){
return admin.database().ref(`users/${uid}/nearbyjobs/${key}`).set({ d: distance });
}else{
return null;
}
});
});
geoQueryJobs.on("ready", () => {
resolve();
});
});
});
}
你有一个承诺then()
调用嵌套在另一个承诺的then()
中。 这被认为是糟糕的风格,并且使您的代码难以阅读。 如果你有一个工作序列要执行,最好一个接一个地链接你的工作,而不是嵌套在另一个里面。 因此,与其像这样嵌套:
doSomeWork()
.then(results1 => {
return doMoreWork()
.then(results2 => {
return doFinalWork()
})
})
按如下方式对工作进行排序:
doSomeWork()
.then(results => {
return doMoreWork()
})
.then(results => {
return doFinalWork()
})
搜索该错误消息也会产生此有用的讨论。