如何防止Node.js Express GET路由传入错误ID时出现冻结无响应问题

• 无匹配数据时未返回响应：当传入的ID不存在时，Movie.findOne会返回null，现有代码仅在movie存在时调用res.render返回页面，无匹配数据时没有任何返回动作，请求会一直挂起直到服务端超时，最终抛出500错误
• 无异常捕获逻辑：如果传入的ID格式不符合数据库要求（比如ID为整型时用户传入字符串），数据库查询会抛出异常，现有代码没有捕获逻辑，异常直接透传导致500
• 同时使用async/await和.then的冗余写法，提高了逻辑遗漏的概率

直接替换原有控制器逻辑即可：

albumEdit: async (req, res) => {
  try {
    const editInfoId = req.params.id;
    // 可选：提前校验ID格式，比如要求为数字，不符合直接跳转错误页，减少无效数据库查询
    if (!/^\d+$/.test(editInfoId)) {
      return res.redirect('/error/404');
    }
    // 移除冗余的.then，直接用await接收查询结果
    const movie = await Movie.findOne({ where: { id: editInfoId } });
    if (movie) {
      return res.render('admin/movies/edit', { title: 'Edit Movie On Page One', movie });
    }
    // 无匹配数据时跳404错误页
    return res.redirect('/error/404');
  } catch (err) {
    // 捕获所有异常，打印日志后跳500错误页
    console.error('查询编辑信息失败：', err);
    return res.redirect('/error/500');
  }
},

你可以在Express入口文件的路由配置最后，加全局错误处理中间件，覆盖所有未被捕获的异常，避免单接口问题影响整体服务：

// 注意：全局错误处理中间件必须保留四个参数 err/req/res/next，顺序不能改
app.use((err, req, res, next) => {
  console.error('全局捕获异常：', err);
  // 开发环境可以返回错误详情，生产环境隐藏敏感信息
  const errorInfo = process.env.NODE_ENV === 'development' ? err : {};
  res.status(err.status || 500);
  res.render('error', {
    message: '服务暂时不可用',
    error: errorInfo
  });
});

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