Node.js回调函数异常：sumOfSeconds调用提示fun非函数

嘿，我来帮你捋清楚这个问题！你遇到的fun is not a function错误，核心原因是变量作用域覆盖+异步逻辑处理不当导致的，咱们一步步拆解：

具体问题出在哪？ #

先看你app.get里的这段代码：

app.get('/', function(req, res){
 var best = sumOfSeconds(best);
 var worst = sumOfSeconds(worst);
 // ... 其他代码
});

1. 变量名冲突覆盖函数

在JavaScript的函数作用域里，var声明的变量会被提升到作用域顶部，但只提升声明、不提升赋值。这段代码实际执行顺序是：

• 先在当前路由的作用域里声明var best和var worst，此时它们的值都是undefined
• 然后执行sumOfSeconds(best)——这里的best已经是当前作用域的undefined，完全覆盖了外部的best()函数！
• 当sumOfSeconds里调用fun()时，自然会报错“fun is not a function”，因为你传进去的根本不是函数

而你在外部调用时，没有同名变量覆盖best函数，所以能正常传参，自然没问题。

2. 异步逻辑的坑

另外还有个隐藏问题：你的best()和worst()函数内部有Time.find()异步操作，直接var best = best()是拿不到返回值的，因为异步函数不会同步返回结果，这会导致你渲染页面时best和worst都是undefined。

怎么修复？ #

给你两个实用的修复方案：

方案一：避免变量冲突+回调内处理异步

先把路由里的变量名改成和函数名不同的，再把渲染逻辑放到sumOfSeconds的回调里，确保所有异步操作完成后再渲染：

module.exports = function(app){
 app.get('/', function(req, res){
   // 调用sumOfSeconds，传入回调函数确保数据准备好再执行后续逻辑
   sumOfSeconds(function() {
     // 这里调用外部的best/worst函数，不会被覆盖
     // 但注意：best/worst内部也是异步，需要再处理
     Time.find({}, function(err, data){
       if(err) throw err;
       // 直接在data回调里计算最佳/最差时间，避免多层异步嵌套
       const sumOfSolves = data.map(item => item.min * 60 + item.sec);
       const bestIndex = sumOfSolves.indexOf(Math.min(...sumOfSolves));
       const worstIndex = sumOfSolves.indexOf(Math.max(...sumOfSolves));
       
       res.render('index', {
         times: data,
         best: data[bestIndex].time,
         worst: data[worstIndex].time
       });
     });
   });
 });
}

// 简化sumOfSeconds，其实你可以把计算逻辑合并到路由里，避免全局变量
function sumOfSeconds(fun){
 Time.find({}, function(err, data){
   if(err) throw err;
   sumOfSolves = data.map(item => item.min * 60 + item.sec);
   fun(); // 数据准备好后执行回调
 })
}

方案二：用Promise+async/await优化异步逻辑（更推荐）

把异步操作改成Promise风格，用async/await让代码更易读，同时避免变量冲突：

// 把Time.find封装成Promise函数
function getAllTimes() {
  return new Promise((resolve, reject) => {
    Time.find({}, (err, data) => {
      if(err) reject(err);
      resolve(data);
    });
  });
}

module.exports = function(app){
 app.get('/', async function(req, res){
   try {
     // 等待获取所有时间数据
     const data = await getAllTimes();
     // 计算总秒数数组
     const sumOfSolves = data.map(item => item.min * 60 + item.sec);
     // 找到最佳/最差时间的索引
     const bestIndex = sumOfSolves.indexOf(Math.min(...sumOfSolves));
     const worstIndex = sumOfSolves.indexOf(Math.max(...sumOfSolves));
     
     // 渲染页面
     res.render('index', {
       times: data,
       best: data[bestIndex].time,
       worst: data[worstIndex].time
     });
   } catch(err) {
     console.error(err);
     res.status(500).send('服务器出错啦');
   }
 });
}

这种方式完全不需要全局的sumOfSolves变量，也避免了多层回调嵌套，代码逻辑更清晰。

额外小提醒 #

尽量避免使用全局变量（比如你的sumOfSolves），多次调用时容易出现竞态问题，把数据封装在函数内部或者用闭包管理会更安全。

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