1 回答

TA贡献2080条经验 获得超4个赞
乍一看,这代码一大堆的问题!
1) req.app.get('jwtTokenSecret')这里出错了,一看到这个req就觉得奇怪。但是这个解码中间件没用过,为防被打脸,还专门搜了一下。这里应该是var decoded = jwt.decode(token, app.get('jwtTokenSecret')); CtrlC CtrlV 是程序员用的最多的按键,这都搞错的话职业素养还需加强啊。
2)if语法问题。你的if(err)内部既没有return后面也没else分支,所以err为真时整个函数体的代码都执行了,而你本意是一个if else。这里我猜你认为res.end会终止整个响应函数?
3)一个响应如果已经end之后,是不能再去设置status的,否则会报错退出程序。在express里面,每次调用send、json这些响应方法的时候,都会自动附加一个end。你send之后还来一个status,程序终结。
4)try catch 不能捕捉异步里面的异常,而newPost.save这显然是一个异步过程,所以你这里的try catch只能捕捉到同步过程中的异常。话说,你是想捕捉哪一步的异常呢?我猜只是想捕捉解码那部分吧。try里面包的东西太多了,你本来就想不让解码的出错影响到其他,但是后面你还用了name:decoded.iss,那这个try catch就耐人寻味了。应该将catch放在这句之前。
接下来我就帮你还原一下这个流程是怎么挂的吧。首先是post请求发文对吧,一个请求过来,首先在var decoded = jwt.decode(token,req.app.get('jwtTokenSecret'));这里就应该挂了,第一个if,如果进去又挂了,然后new也可能挂一次,save异步直接过去,然后res.send。接着save的异步里面又有可能挂一次。或者catch 里面也会挂一次。所以我觉得应该不是在你请求的时候才挂的,应该是早挂了吧?
说实话,我还是建议你好好看看js语法,然后好好看看官方文档,然后再好好看看别人代码,才来写你的。
添加回答
举报