基于Promise的远程目录递归扫描异步问题求助

兄弟，我太懂你这个坑了！异步递归里乱用return真的会踩大雷，我之前也栽过类似的跟头😅。你遇到的核心问题就是：在递归调用scanDir()前加了return，导致循环刚处理到第一个子目录就直接终止了函数，后面的子目录和文件根本没机会执行，可不就只扫第一个子目录嘛！

我先给你还原一下大概你之前的错误代码（猜的，应该八九不离十）：

async function scanDir(path) {
  const items = await remoteScanLib(path);
  for (const item of items) {
    if (item.isDirectory) {
      // 这里的return直接终止了函数，后面的循环都走不到
      return scanDir(item.path);
    } else {
      console.log("找到文件：", item.path);
    }
  }
}

正确的写法分两种场景： #

场景1：需要收集所有文件的路径

如果你要把所有扫描到的文件路径汇总返回，那可以用串行或者并行的方式处理：

串行处理（逐个扫子目录）

这种方式逻辑简单，适合对并发请求有严格限制的场景：

async function scanDir(path) {
  const items = await remoteScanLib(path);
  const allFiles = [];

  for (const item of items) {
    if (item.isDirectory) {
      // 不要return，而是await递归结果，合并到数组里
      const subFiles = await scanDir(item.path);
      allFiles.push(...subFiles);
    } else {
      allFiles.push(item.path);
    }
  }

  return allFiles;
}

并行处理（同时扫多个子目录，效率更高）

如果你的远程服务器允许并发请求，用Promise.all可以大幅提高扫描速度：

async function scanDir(path) {
  const items = await remoteScanLib(path);

  // 把每个子项的处理转换成Promise
  const taskPromises = items.map(async (item) => {
    if (item.isDirectory) {
      // 递归扫子目录，返回子目录的文件数组
      return scanDir(item.path);
    } else {
      // 文件的话返回单元素数组，方便后续合并
      return [item.path];
    }
  });

  // 等待所有任务完成，然后把二维数组扁平化
  const allResults = await Promise.all(taskPromises);
  return allResults.flat();
}

场景2：不需要收集结果，只是处理每个文件（比如打印、上传）

如果只是要处理每个文件，不需要汇总返回，那直接在循环里await递归调用就行，别加return：

async function scanDir(path) {
  const items = await remoteScanLib(path);

  for (const item of items) {
    if (item.isDirectory) {
      // 这里不要return，直接await递归处理
      await scanDir(item.path);
    } else {
      // 这里写你的文件处理逻辑，比如打印、上传等
      console.log("处理文件：", item.path);
    }
  }
}

额外提醒 #

如果是远程扫描，并行请求太多可能会触发服务器的限流机制，要是遇到这种情况，可以用并发控制库（比如p-limit）来限制同时发起的请求数量，避免被服务器拉黑哦～

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