Express中两次GET请求触发id未定义错误的技术问询

你已经把问题根因抓得很准了——核心就是处理机器列表请求的loc_machines_get控制器，在渲染modify-loc.ejs时没有传入模板依赖的id参数，导致新窗口页面里的<%=id%>触发了未定义错误。下面给你几个直接可行的解决方案：

方案1：修改控制器，确保传入id参数 #

你的loc_machines_get目前只传递了位置和机器数据，补上id参数就能直接解决问题。调整后的控制器代码如下：

// 修正后的loc_machines_get控制器
exports.loc_machines_get = async (req, res) => {
  const locationId = req.params.id; // 从URL参数中提取id
  const location = await Location.findById(locationId);
  const machines = await Machine.find({ location: locationId });
  
  // 将id与其他数据一起传给模板
  res.render('modify-loc.ejs', { 
    id: locationId, 
    location, 
    machines 
  });
};

这样模板里的<%=id%>就能正常获取到值，表单的action属性也不会报错了。

方案2：优化模板逻辑，减少对单独id参数的依赖 #

既然两个控制器都会传入location对象（位置详情数据），你可以直接用location._id替代单独的id变量，从根源上避免变量依赖问题。比如：

把模板中的表单action从：

<form action="/locations/modify/<%=id%>" method="POST">

改成：

<form action="/locations/modify/<%=location._id%>" method="POST">

同时把打开机器列表的链接也调整为：

<a href="/location/<%=location._id%>/machines" target="_blank">查看机器列表</a>

这样不管哪个控制器渲染模板，只要有location对象，模板就能正常工作，不用再纠结是否单独传递id。

小提醒：统一路由路径的大小写 #

注意到你的路由存在大小写不一致的情况（比如/Locations/modify/...和/location/...），虽然Express默认不区分路由大小写，但统一成全小写的规范写法，能避免后续出现一些莫名其妙的路径匹配问题。

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