Node.js请求验证是否需try...catch？与typeof检查是否冗余？

嗨，我来帮你拆解这个问题～

首先明确：在你当前的代码场景里，获取req.headers['authorization']时完全不需要try...catch。原因很简单：在Express这类Node.js web框架的中间件中，req（请求对象）一定是存在的，而req.headers是一个普通的JavaScript对象。当你访问对象上不存在的属性时，JavaScript只会返回undefined，不会抛出任何异常——所以这个try...catch块永远不会被触发，完全是多余的。

答案是肯定的，完全冗余。刚才说了，try...catch在这里根本起不到任何作用，而typeof检查才是真正用来判断bearerHeader是否为undefined的有效方式。你当前的try...catch块相当于“摆设”，和后面的typeof检查没有任何互补性，反而增加了不必要的代码复杂度。

不行。因为前面提到的，访问对象不存在的属性不会抛出错误，所以哪怕authorization头不存在，try块里的代码也不会触发catch逻辑，你的403响应永远不会通过这个catch返回。这种情况下，必须用typeof检查（或者直接判断bearerHeader === undefined）才能正确捕获请求头缺失的情况。

既然try...catch没用，直接去掉就好，简化后的代码会更清晰：

module.exports = function verifyToken(req, res, next) {
  // Get auth header value
  const bearerHeader = req.headers['authorization'];

  if (typeof bearerHeader === 'undefined') {
    // Forbidden
    return res.sendStatus(403);
  }

  // Split at the space
  const bearer = bearerHeader.split(' ');
  // Get token from array
  const bearerToken = bearer[1];
  // Set the token
  req.token = bearerToken;
  next();
}

另外补充一点：如果bearerHeader存在但格式不对（比如没有空格分割），split(' ')可能会返回长度为1的数组，此时bearer[1]会是undefined，如果后续逻辑依赖req.token，你可能需要额外处理这种情况，但这是另一个问题啦。

内容的提问来源于stack exchange，提问作者July333