koa-compose 为什么需要 Promise.resolve(fn(context, dispatch.bind(null, i + 1)));
 发布于 6 年前  作者 xinshangshangxin  4987 次预览  最后一次回复是 6 年前  来自 问答 

koa-composereturn 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 回复
yviscool

按你这个写法, 这里得加上 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

@yviscool

谢谢指出错误~

请问 还有其它情况 是需要原来的 Promise.resolve(fn(context, dispatch.bind(null, i + 1))); 吗?

yviscool

@xinshangshangxin 感觉很难有了,等楼下看有什么补充的

ta7sudan

递归层级过深他不会爆栈你这里会爆栈?

ta7sudan

@ta7sudan 不对,当我没说