如何让这段使用await-to-js的Node.js异步请求代码更简洁?
解决方案
完全可以合并,以下两种方案均和原有逻辑完全一致,满足任意请求出错立即终止后续操作、返回对应错误信息的要求:
方案1:最简改造(仅合并错误返回逻辑)
把所有请求放入try块,出错直接抛出,最外层统一捕获返回即可:
try { let [loginErr, loginRes] = await to(sa_login(req)) if (loginErr) throw loginErr let [infoErr,infoRes] = await to(sa_getUserInfo(loginRes)) if(infoErr) throw infoErr let [courseInfoErr,courseInfoRes] = await to(sa_getCourseInfo(loginRes)); if (courseInfoErr) throw courseInfoErr // 原有正常业务逻辑写在此处 // ... } catch (err) { // 三个请求的错误判断合并为这一处处理 return ctx.sendError('400', err) }
方案2:彻底消除重复if判断
封装自动抛出错误的请求工具,后续新增请求也不需要再写重复的错误判断代码:
// 封装请求工具:请求出错自动抛出 const toThrow = async (promise) => { const [err, res] = await to(promise) if (err) throw err return res } // 业务代码 try { const loginRes = await toThrow(sa_login(req)) const infoRes = await toThrow(sa_getUserInfo(loginRes)) const courseInfoRes = await toThrow(sa_getCourseInfo(loginRes)) // 原有正常业务逻辑写在此处 // ... } catch (err) { return ctx.sendError('400', err) }
逻辑说明
- 完全保留原有串行执行逻辑:先执行登录、再获取用户信息、最后获取课程信息,不会改变请求顺序
- 任意请求出错都会直接终止后续操作,返回的错误信息和原有代码完全一致
内容的提问来源于stack exchange,提问作者Marshall
相关产品推荐
相关产品推荐

