koa-compose 为什么需要 Promise.resolve(fn(context, dispatch.bind(null, i + 1)));
发布于 6 年前 作者 xinshangshangxin 4212 次浏览 来自 问答

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 回复

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

@yviscool

谢谢指出错误~

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

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

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

@ta7sudan 不对,当我没说

回到顶部