调用API失败：userAPI.js中按ID查询用户数据的问题排查

我帮你梳理了这段代码里的几个关键问题，这些应该就是导致API无法正常调用的核心原因：

1. 数据库查询方法调用错误 #

你这里写了 db.query =(sql,[user_id],function(err,results){ —— 这是把db.query赋值成了一个函数调用表达式，而不是真正调用db.query方法。这种写法会直接覆盖掉数据库驱动提供的查询方法，自然无法执行SQL查询。

2. else分支存在未定义变量 #

当req.params.id不存在时，你返回的response是results，但这个变量只在if分支的查询回调里定义，这里会直接抛出ReferenceError，导致响应异常。

3. 错误处理方式不当 #

直接throw err会导致Node.js进程崩溃，而不是给客户端返回合法的错误响应。在Express环境中，应该通过响应对象返回错误信息，同时终止请求处理流程。

4. 路由参数判断逻辑可优化 #

因为这是/:id路由，只要请求匹配该路由，req.params.id就会存在（可能是空字符串，但不会是undefined），建议添加空值判断，避免执行无意义的空ID查询。

修正后的完整代码 #

router.get('/:id', function(req, res, next) {
  const user_id = req.params.id;
  
  // 校验ID是否有效（非空且非空白字符）
  if (user_id && user_id.trim()) {
    const sql = "SELECT * FROM users WHERE id = ?";
    // 正确调用db.query方法，去掉多余的等号
    db.query(sql, [user_id], function(err, results) {
      if (err) {
        // 返回500错误，避免进程崩溃
        return res.status(500).json({
          "status": 500,
          "error": err.message,
          "response": null
        });
      } else {
        res.status(200).json({
          "status": 200,
          "error": null,
          "response": results
        });
      }
    });
  } else {
    // 返回400错误，明确提示ID无效
    res.status(400).json({
      "status": 400,
      "error": "Invalid user ID provided",
      "response": null
    });
  }
});

额外优化说明 #

• 用res.json()替代JSON.stringify：Express会自动处理JSON序列化和响应头设置，代码更简洁规范。
• 用return终止错误分支的后续执行：避免错误处理后继续执行正常响应逻辑。
• 增加ID的空白字符校验：过滤掉类似/ 这种无效的空白ID请求。

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