JavaScript：在forEach中累加变量计算目录总大小的问题求解

我一眼就看出你代码里的几个典型问题，导致总大小没正确累加，反而输出了单个文件的大小：

问题分析 #

1. 累加时机完全错误：你在forEach循环开头就执行totalSize += size，但这时候size要么是初始值0，要么是上一个文件的大小，根本不是当前文件的实际大小！只有当你确认当前路径是文件（进入else分支）后，才拿到了正确的文件大小，这时候才应该执行累加操作。
2. 重复调用FS函数造成冗余：原代码先调用fs.lstatSync判断是否为目录，又调用fs.statSync拿文件大小——其实lstatSync返回的stats对象已经包含了size属性，完全没必要重复调用。
3. 同名函数的作用域混淆：外层已经定义了walk函数，内部又嵌套了一个同名的walk，虽然JS允许这种写法，但会让变量作用域变得混乱，增加调试难度。
4. 无效变量干扰逻辑：n变量的增减操作对总大小计算没有任何帮助，反而会让代码可读性下降。

修正后的代码 #

const fs = require('fs');
const path = require('path');

// 先确保你已经初始化了写入流（原代码里的listFiles）
const listFiles = fs.createWriteStream('copyList.xml');

function calculateDirTotalSize(targetDir) {
  // 总大小变量放在最外层，确保所有递归调用共享同一个累加容器
  let totalSize = 0;

  function traverseDir(currentDir) {
    fs.readdirSync(currentDir).forEach(fileName => {
      const fullFilePath = path.join(currentDir, fileName);
      const fileStats = fs.lstatSync(fullFilePath);

      if (fileStats.isDirectory()) {
        // 递归处理子目录，子目录内的文件大小会自动累加到外层的totalSize
        traverseDir(fullFilePath);
      } else {
        // 拿到当前文件的实际大小，立即累加到总大小
        const currentFileSize = fileStats.size;
        totalSize += currentFileSize;
        
        // 保留你原有的写入逻辑
        listFiles.write(currentFileSize + "\n");
        // 调试用：打印单个文件大小（和你原来的输出一致）
        console.log(currentFileSize);
      }
    });
  }

  // 启动遍历
  traverseDir(targetDir);
  // 关闭写入流（避免内存泄漏）
  listFiles.end();
  
  // 返回最终计算的总大小
  return totalSize;
}

// 调用示例：替换成你的目标目录
const total = calculateDirTotalSize('./your-directory-path');
console.log('目录总大小:', total);

关键改动说明 #

• 正确的累加时机：只有在确认是文件后，才将当前文件的大小加到totalSize中，确保每一次累加的都是有效数据。
• 共享的累加变量：totalSize定义在外层函数中，所有递归调用的traverseDir都会操作同一个变量，不会出现重置或丢失的情况。
• 复用FS返回值：只调用一次fs.lstatSync，既判断了是否为目录，又拿到了文件大小，提升了代码效率。
• 清晰的命名：把内部遍历函数改名为traverseDir，避免和外层函数同名，提升代码可读性。
• 完善的流处理：添加了listFiles.end()，确保写入流正常关闭，避免内存泄漏。

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