如何在Node.js同一文件中调用其他函数？（附代码示例）

你在同一个模块里调用users函数时遇到的报错，主要源于两个核心问题：函数作用域找不到users，以及调用时缺少必要的请求响应参数，下面一步步帮你解决：

问题分析 #

1. 你直接把函数挂载到exports上（exports.users = function(...)），但模块内部并没有定义名为users的本地变量，所以直接调用users()会提示"users is not defined"。
2. 就算找到users函数，你调用时没传req和res参数，而users函数依赖这两个参数来处理请求和渲染页面。

解决方案 #

方案一：先定义本地函数再导出（推荐） #

这种方式逻辑最清晰，模块内部可以直接调用本地函数，同时也能正常导出供外部路由使用：

// 先定义本地的users函数
function users(req, res) {
  var sql = "SELECT * FROM table";
  connection.query(sql, function(err, result) {
    if (err) {
      console.log(err);
    } else {
      res.render('user', { data: result });
    }
  });
}

// 导出供外部路由使用
exports.users = users;

exports.deleteuser = function(req, res) {
  console.log(req.params);
  // 直接调用本地的users函数，同时传入req和res参数
  users(req, res);
};

方案二：通过exports/module.exports调用 #

如果不想修改原有导出方式，也可以直接通过exports对象调用已导出的函数，记得必须传参：

exports.users = function(req, res) {
  var sql = "SELECT * FROM table";
  connection.query(sql, function(err, result) {
    if (err) {
      console.log(err);
    } else {
      res.render('user', { data: result });
    }
  });
};

exports.deleteuser = function(req, res) {
  console.log(req.params);
  // 通过exports调用users函数，传入req和res
  exports.users(req, res);
  // 也可以用 module.exports.users(req, res);
};

额外优化建议 #

1. 完善deleteuser的业务逻辑：你现在的deleteuser函数只调用了users，但没处理删除操作，应该先执行删除SQL，再调用users渲染更新后的列表：

exports.deleteuser = function(req, res) {
  console.log(req.params);
  var email = req.params.email;
  // 注意：这里建议用参数化查询防止SQL注入，不要直接拼接字符串！
  var sql = "DELETE FROM appuser WHERE email = ?";
  connection.query(sql, [email], function(err, result) {
    if (err) {
      console.log(err);
      res.status(500).send('删除失败，请稍后重试');
      return;
    }
    // 删除成功后调用users函数，刷新用户列表
    exports.users(req, res);
  });
};

2. 修正app.js的路由配置：如果要使用你定义的deleteuser函数，需要把路由改成：

app.get('/deleteuser/:email', company.deleteuser);

3. 移除服务器端的alert：你app.js里的alert("deleted")是浏览器端API，Node.js服务器无法执行，换成服务器端日志或者在前端页面做提示即可。

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