Promise.all返回undefined值的问题排查（附JavaScript代码示例）

你的Promise.all返回全是undefined，核心问题出在**query函数没有正确返回Promise**，还有几个细节错误：

• 第一：query函数内部调用了rp(url)（这是一个Promise实例），但你没有把这个Promise返回出去。所以data.map(query)生成的数组里，每个元素都是undefined，而非Promise实例，Promise.all等待一堆undefined自然只会返回全是undefined的数组。
• 第二：then里的return new Promise(resolve => res)是错误写法，这会创建一个永远处于pending状态的Promise（因为你没调用resolve方法），就算rp请求成功，这个Promise也不会完成。
• 第三：catch块只打印了"error"，但没有重新抛出错误，这样如果某个请求失败，错误会被吞掉，Promise.all无法感知到失败，最终会得到不完整的结果。

我们需要修正query函数的几个关键问题，同时优化代码的严谨性：

1. 确保query函数返回rp(url)生成的Promise链
2. 去掉多余的Promise包装，直接返回请求结果
3. 在catch里重新抛出错误，让Promise.all能捕获到失败的请求
4. 用const/let声明变量，避免全局污染

下面是修正后的完整代码：

const rp = require('request-promise'); // 确保已引入request-promise

const query = function(entity) {
  // 替换为你的SPARQL请求URL，记得用encodeURIComponent处理参数
  const url = `你的SPARQL请求地址?参数=${encodeURIComponent(entity)}`;
  // 必须返回Promise链
  return rp(url)
    .then(function(res) {
      console.log(`实体${entity}的请求结果:`, res);
      // 直接返回结果，无需额外包装
      return res;
    })
    .catch(function(err) {
      console.log(`实体${entity}请求出错:`, err);
      // 重新抛出错误，让Promise.all能捕获到批量请求的失败
      throw err;
    });
};

async function musicData() {
  const data = [1,2,3,4];
  const queries = data.map(query);
  const results = await Promise.all(queries);
  return results;
}

musicData()
  .then(function(resp) {
    console.log("所有请求完成，结果集合:", resp);
  })
  .catch(function(err) {
    console.log("批量请求失败:", err);
  });

• 如果SPARQL接口返回的是JSON格式响应，可以在rp的配置里添加json: true，自动解析响应结果，无需手动处理
• 可以给rp添加超时、重试等配置，提升请求的稳定性

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