使用Axios+forEach请求GitHub API返回空数组的问题及解决方法

核心问题 #

1. forEach无法处理异步流程：forEach会直接遍历完所有元素，不会等待内部的await请求完成，导致函数提前返回空的stats数组。
2. 函数未返回异步结果：当前函数同步返回数组，但内部是异步操作，必须返回Promise才能让调用者等待结果。
3. 变量与模板字符串错误：repoLang未声明会污染全局作用域；Authorization头用双引号包裹，内部的${token}不会被解析为变量值。
4. 数据结构不合理：直接push(repoLang.data, repoName)会把仓库名和语言数据拆成数组的独立元素，后续无法对应关联。

修复后的函数 #

const axios = require('axios'); // 确保已引入axios

async function getRepositoryLanguages(owner, repos) {
  const stats = [];
  // 用Promise.all等待所有异步请求完成
  await Promise.all(repos.map(async (repoName) => {
    try {
      const repoLang = await axios.get(
        `https://api.github.com/repos/${owner}/${repoName}/languages`,
        {
          headers: {
            Authorization: `token ${token}` // 用反引号解析模板变量
          },
        }
      );
      // 存入关联对象，保证数据对应关系
      stats.push({
        repoName,
        languages: repoLang.data
      });
    } catch (error) {
      console.error(`仓库${repoName}语言数据获取失败:`, error.message);
      // 可选：存入错误信息，保留失败记录
      stats.push({
        repoName,
        error: error.message
      });
    }
  }));
  return stats;
}

Express路由中的使用示例 #

app.get('/stats', async (req, res) => {
  try {
    const username = req.query.username; // 从请求参数获取用户名
    if (!username) {
      return res.status(400).json({ success: false, error: '缺少用户名参数' });
    }
    const repos = await getRepositoryNames(username); // 假设该函数返回Promise
    const stats = await getRepositoryLanguages(username, repos);
    res.json({ success: true, data: stats });
  } catch (error) {
    res.status(500).json({ success: false, error: error.message });
  }
});

关键说明 #

• 用map+Promise.all替代forEach，确保所有异步请求完成后再返回结果。
• 函数标记为async，自动返回Promise，调用时通过await获取最终填充后的数组。
• 将仓库名与语言数据封装为对象，让返回结构更清晰，便于后续逻辑处理。
• 错误处理中添加仓库名标识，方便定位问题，也可根据业务需求过滤失败记录。

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