koa-compose 为什么需要 Promise.resolve(fn(context, dispatch.bind(null, i + 1)));
在 koa-compose
中 return Promise.resolve(fn(context, dispatch.bind(null, i + 1)));为什么需要 Promise.resolve()
a: 在 koa/application.js
中
fnMiddleware(ctx).then(handleResponse)
, 可知需要一个 Promise
但是 改成如下也是可以工作的
return function (context, next) {
// last called middleware #
let index = -1
return dispatch(0)
function dispatch (i) {
if (i <= index) return Promise.reject(new Error('next() called multiple times'))
index = i
let fn = middleware[i]
if (i === middleware.length) fn = next
// 添加 对 i === 0 判断是否是 fnMiddleware(ctx).then
if( i === 0) {
fn = (...args) => {
let res = null;
// 1楼提出的改善
try{
res = Promise.resolve(middleware[0](...args))
} catch(e) {
res = Promise.reject(e);
}
return res;
}
}
if (!fn) return Promise.resolve()
try {
return fn(context, dispatch.bind(null, i + 1));
} catch (err) {
return Promise.reject(err)
}
}
}
b: 如果是 middleware 中 需要 await next
的话, 一定是一个 async
函数, 所以也不需要Promise.resolve()
c: 如果 middleware 是一个同步函数, 也不需要 Promise.resolve()
还有哪些情况我没有考虑到的?
6 回复
按你这个写法, 这里得加上 try/catch
if( i === 0) {
fn = (...args) => {
let res = null;
try{
res = Promise.resolve(middleware[0](...args))
} catch(e) {
res = Promise.reject(e);
}
return res;
}
}
@xinshangshangxin 感觉很难有了,等楼下看有什么补充的
递归层级过深他不会爆栈你这里会爆栈?
@ta7sudan 不对,当我没说
Good