Node.js Express MVC中res.render未定义TypeError问题求助

为什么会出现TypeError: Cannot read property 'render' of undefined？ #

你原来的路由写法犯了一个Express的基础错误：

router.post('/search-noms', function (req, res) {nomenclatureController.searchNoms(req.body.idNoms)})

这里你手动调用searchNoms时，只传了req.body.idNoms这一个参数，但你的控制器函数searchNoms的定义是接收req和res两个Express核心对象的。这就导致：

• 控制器里的req变量实际变成了req.body.idNoms这个字符串值
• res变量直接变成了undefined（因为你没传第二个参数）
  所以当你调用res.render时，自然会报错“无法读取undefined的render属性”。

而当你改成router.post('/search-noms', nomenclatureController.searchNoms)时，Express会自动把当前请求的req、res（还有可选的next）对象传递给searchNoms函数，这时候控制器里的res是有效的响应对象，所以能正常渲染视图。

如何在保持正常运行的同时，传递req.body.idNoms参数？ #

推荐三种方案，按优雅程度排序：

方案1：在控制器内部提取参数（最推荐）

保持路由的简洁写法，直接把控制器函数绑定给路由，然后在控制器里从req对象中取出需要的idNoms：

// 路由代码
router.post('/search-noms', nomenclatureController.searchNoms)

// 控制器代码
exports.searchNoms = (req, res) => { 
  // 直接从req.body里提取idNoms参数
  const idNoms = req.body.idNoms;
  getDatabaseModel.searchNoms(idNoms).then(function(result) { 
    console.log(result); 
    res.render('Home/home-page', { nomenclature: result }); 
  }).catch((err) => setImmediate(() => { throw err; })) 
};

这种方式完全符合Express的设计规范，代码可读性和可维护性都最高，也是社区通用的写法。

方案2：路由中传递完整的req和res

如果你一定要在路由层做一些处理再调用控制器，可以把完整的req和res对象传给控制器函数：

// 路由代码
router.post('/search-noms', function (req, res) {
  // 这里可以额外做一些参数校验或预处理
  nomenclatureController.searchNoms(req, res);
})

// 控制器代码保持你原来的写法即可
exports.searchNoms = (req, res) => { 
  getDatabaseModel.searchNoms(req).then(function(result) { 
    console.log(result); 
    res.render('Home/home-page', { nomenclature: result }); 
  }).catch((err) => setImmediate(() => { throw err; })) 
};

这种方式适合需要在路由层统一处理某些逻辑的场景，比如参数校验、权限判断等。

方案3：修改控制器参数（不推荐）

如果你坚持要在路由层传递idNoms而非完整的req，可以修改控制器的参数定义，同时把res也传过去：

// 路由代码
router.post('/search-noms', function (req, res) {
  nomenclatureController.searchNoms(req.body.idNoms, res);
})

// 控制器代码
exports.searchNoms = (idNoms, res) => { 
  // 注意这里模型调用的参数要改成idNoms，而不是req
  getDatabaseModel.searchNoms(idNoms).then(function(result) { 
    console.log(result); 
    res.render('Home/home-page', { nomenclature: result }); 
  }).catch((err) => setImmediate(() => { throw err; })) 
};

这种方式不推荐，因为它打破了Express控制器函数(req, res, next)的标准参数约定，会让代码变得不直观，后期维护成本更高。

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