如何让遍历目录的递归函数具备类型安全性？

一、解决类型安全问题（移除as any） #

原代码中as any的出现是因为递归调用返回的Promise<FileSystemEntry[]>与iterationAttempts数组的Promise<FileSystemEntry>[]类型不兼容。我们可以通过调整数组存储的类型，结合数组扁平化来解决这个问题，同时用类型守卫替代必要的类型断言，彻底消除不安全的类型转换。

优化后的类型安全代码（使用async/await提升可读性）

private async traverseDirectory(entry: FileSystemDirectoryEntry): Promise<FileSystemEntry[]> {
  const reader = entry.createReader();
  const allEntryGroups: FileSystemEntry[][] = [];

  // 类型守卫：精准判断条目是否为目录类型
  const isDirectoryEntry = (entry: FileSystemEntry): entry is FileSystemDirectoryEntry => {
    return entry.isDirectory;
  };

  // 封装readEntries为Promise，避免回调嵌套
  const readBatchEntries = (): Promise<FileSystemEntry[]> => {
    return new Promise((resolve, reject) => {
      reader.readEntries(resolve, reject);
    });
  };

  // 递归读取所有批次的目录条目
  const readAllEntries = async (): Promise<void> => {
    const batchEntries = await readBatchEntries();
    if (batchEntries.length === 0) return;

    // 处理当前批次的所有条目，递归遍历子目录
    const batchResults = await Promise.all(
      batchEntries.map(async (batchEntry) => {
        if (isDirectoryEntry(batchEntry)) {
          return this.traverseDirectory(batchEntry);
        } else {
          return [batchEntry]; // 文件条目包装为单元素数组，统一类型
        }
      })
    );

    allEntryGroups.push(...batchResults);
    await readAllEntries();
  };

  try {
    await readAllEntries();
    return allEntryGroups.flat(); // 扁平化所有条目组，返回最终文件列表
  } catch (err) {
    logger.error('遍历目录时发生错误:', err);
    throw err; // 抛出错误让调用方处理
  }
}

二、可读性与整洁度优化建议 #

• 用async/await替代回调嵌套：将原本的Promise回调逻辑改为线性的async/await写法，消除"回调地狱"，代码逻辑更直观。
• 添加类型守卫函数：通过isDirectoryEntry类型守卫，无需使用as FileSystemDirectoryEntry断言，TypeScript能自动推断类型，提升类型安全性。
• 拆分独立逻辑为函数：将readEntries的Promise封装、批次读取逻辑拆分为单独函数，每个函数职责单一，代码更易维护。
• 完善错误处理：捕获遍历过程中的异常并抛出，同时记录具体错误信息，方便调试和调用方处理异常。
• 语义化变量命名：使用allEntryGroups、readBatchEntries等清晰的变量/函数名，明确代码意图，降低理解成本。
• 统一返回类型：将文件条目包装为单元素数组，让目录递归返回的数组与文件条目类型统一，避免类型不兼容问题。

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