如何从回调函数获取值到外部作用域？修复getHighScores函数返回null问题

你写的getHighScores函数期望从JSON文件里读取指定游戏的高分数据，排序后返回前N个，但运行后console.log输出的是null。先看一下你的问题代码：

function getHighScores(game, amount) { 
  var highScores = null; 
  fs.readFile('scores.json', 'utf8', function (error, data) { 
    var scoresObj = JSON.parse(data); 
    var gameScores = scoresObj.games[game]; 
    sortedScores = gameScores.sort(compareScoreObjects); 
    highScores = sortedScores.slice(0, amount); 
  }); 
  return highScores; 
} 
console.log(getHighScores('snake', 10));

问题根源 #

你已经精准命中了核心原因：fs.readFile是异步非阻塞的函数——它会立刻返回，让JavaScript继续执行后面的return highScores;，而文件读取的操作会在后台默默完成，等读取结束后才会触发你传入的回调函数。这就导致函数返回的时候，回调里给highScores赋值的逻辑还根本没运行，自然返回的是初始值null。

能不能保持现有逻辑让函数直接返回数据？ #

很遗憾，不行。异步操作的结果没法同步返回——JavaScript单线程的特性决定了，你不能强行让代码等待异步操作完成再继续执行（除非用同步版本的fs.readFileSync，但不推荐在业务逻辑里用，会阻塞整个Node.js进程）。

正确的处理方式 #

我们有两种常用的方案来处理这种异步场景，都能完美解决你的问题：

方案1：使用回调函数传递结果

修改getHighScores，让它额外接受一个回调参数，在文件读取完成并处理好数据后，调用这个回调把结果（或错误）传出去：

function getHighScores(game, amount, callback) { 
  fs.readFile('scores.json', 'utf8', function (error, data) { 
    // 别忘了处理文件读取的错误！
    if (error) {
      return callback(error);
    }
    try {
      var scoresObj = JSON.parse(data); 
      var gameScores = scoresObj.games[game]; 
      // 注意：sort会修改原数组，最好先复制一份再排序，避免破坏原始数据
      var sortedScores = [...gameScores].sort(compareScoreObjects); 
      var topScores = sortedScores.slice(0, amount); 
      callback(null, topScores);
    } catch (parseError) {
      // 处理JSON解析失败的错误
      callback(parseError);
    }
  }); 
} 

// 调用方式
getHighScores('snake', 10, function(error, scores) {
  if (error) {
    console.error('获取高分失败:', error);
    return;
  }
  console.log(scores);
});

方案2：用Promise + async/await（更现代的写法）

把异步操作封装成Promise，然后用async/await让代码看起来像同步逻辑，可读性更好：

// 先引入Promise版本的fs模块
const fs = require('fs').promises;

async function getHighScores(game, amount) { 
  try {
    const data = await fs.readFile('scores.json', 'utf8'); 
    const scoresObj = JSON.parse(data); 
    const gameScores = scoresObj.games[game]; 
    // 复制数组后再排序，保护原始数据
    const sortedScores = [...gameScores].sort(compareScoreObjects); 
    return sortedScores.slice(0, amount);
  } catch (error) {
    console.error('处理过程出错:', error);
    throw error; // 把错误抛出去，让调用方自行处理
  }
} 

// 调用方式一：用.then()链式调用
getHighScores('snake', 10)
  .then(scores => console.log(scores))
  .catch(error => console.error(error));

// 调用方式二：在async函数里用await
async function main() {
  try {
    const scores = await getHighScores('snake', 10);
    console.log(scores);
  } catch (error) {
    console.error(error);
  }
}
main();

额外小提示 #

Array.sort()是原地排序，会直接修改原数组里的gameScores数据。如果不想破坏JSON文件里的原始数据结构，最好先复制一份数组再排序，比如用扩展运算符[...gameScores]或者gameScores.slice()来创建副本。

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