Node.js+Redis环境下Promise未resolve及retArr返回不完整问题

这事儿我太熟了——核心问题就是你的代码没有正确等待所有异步Redis操作完成，就提前返回了retArr。控制台能看到日志，是因为异步操作确实在后面执行了，但你的函数已经先把不完整的数组返回给调用方了。

最常见的错误场景：用了forEach处理异步任务 #

很多人会踩这个坑：forEach根本不支持async/await，它会直接遍历完所有元素，不会等里面的异步回调执行完毕。比如你可能写了类似这样的代码：

async function fetchAllEmails() {
  const retArr = [];
  const listNames = ['newsletter', 'promo', 'support'];

  // 这里的async回调会被当成独立的Promise，forEach不会等它们完成
  listNames.forEach(async (list) => {
    const emails = await redis.smembers(`email:list:${list}`);
    retArr.push(...emails);
    console.log(`Added emails from ${list}`, retArr); // 这里日志能看到数据，但函数已经返回了
  });

  return retArr; // 这里直接返回，此时大部分异步操作还没结束
}

正确的解决方法 #

方法1：用for...of串行处理（适合需要按顺序执行的场景）

for...of是支持async/await的循环结构，它会等待每次迭代里的异步操作完成后，再进入下一次循环：

async function fetchAllEmails() {
  const retArr = [];
  const listNames = ['newsletter', 'promo', 'support'];

  for (const list of listNames) {
    const emails = await redis.smembers(`email:list:${list}`);
    retArr.push(...emails);
    console.log(`Added emails from ${list}`);
  }

  return retArr; // 这里会等所有循环里的await都完成后才返回
}

方法2：用Promise.all并行处理（效率更高，推荐）

如果你的Redis操作之间没有依赖关系，用Promise.all可以同时发起所有请求，大幅提升速度：

async function fetchAllEmails() {
  const listNames = ['newsletter', 'promo', 'support'];

  // 先把所有Redis请求包装成Promise数组
  const emailPromises = listNames.map(list => 
    redis.smembers(`email:list:${list}`)
  );

  // 等待所有Promise完成，拿到所有结果的二维数组
  const allEmailLists = await Promise.all(emailPromises);

  // 把二维数组扁平成一维数组
  const retArr = allEmailLists.flat();

  console.log('All emails loaded', retArr);
  return retArr;
}

额外注意：确保Redis客户端支持Promise #

如果你的Redis库还是回调式的（比如旧版redis包），要先把它转成Promise风格：

const redis = require('redis');
const { promisify } = require('util');

const redisClient = redis.createClient();
// 把smembers方法转成Promise版本
const smembersAsync = promisify(redisClient.smembers).bind(redisClient);

// 之后就可以用await smembersAsync(...)了

简单总结：永远不要在forEach里处理异步任务，要么用for...of串行等待，要么用Promise.all并行等待，确保所有异步操作都完成后再返回结果。

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