使用NodeJS批量复制文件仅最后一个成功的问题排查

问题原因 #

1. 换行符处理错误：Windows系统中文本文件的换行符是\r\n，你用split("\n")拆分后，除最后一行外，每个文件名末尾都会附带\r（回车符）。比如原文件名是test.txt，拆分后变成test.txt\r，复制时会找不到对应源文件，只有最后一行无额外\r，所以能成功复制。
2. 异步操作未同步：fs.cp是异步方法，你在readFile的回调里直接打印count时，大部分复制任务还未执行完成，导致count显示的不是最终成功数量。
3. 未过滤空行：如果FileNames.txt存在空行，会触发空文件名的复制操作，引发错误却未被处理。
4. count变量未初始化：代码中未初始化count，执行count++时可能出现NaN的问题。

解决办法 #

修改代码，处理换行符、过滤空行，并用异步控制确保所有复制任务完成后再统计结果：

const fs = require('fs').promises; // 使用Promise版本的fs，简化异步流程
let count = 0; // 初始化计数变量

async function readFileContents() {
  try {
    const data = await fs.readFile("F:/Some Folder/FileNames.txt", "utf8");
    // 兼容Windows/Unix换行符，同时过滤空行与纯空白行
    const fileNames = data.split(/\r?\n/).filter(name => name.trim() !== '');
    
    // 批量执行复制任务，等待所有任务完成
    await Promise.all(fileNames.map(async (fileName) => {
      try {
        await fs.cp(
          `F:/Some Folder/Origin Folder/${fileName}`,
          `F:/Some Folder/Target Folder/${fileName}`
        );
        console.log(`Transferred ${fileName}`);
        count++;
      } catch (err) {
        console.error(`Failed to transfer ${fileName}:`, err.message);
      }
    }));
    
    console.log(`Total transferred files: ${count}`);
  } catch (err) {
    console.error('Failed to read FileNames.txt:', err.message);
  }
}

readFileContents();

关键修改说明 #

• 使用fs.promises的异步方法，结合async/await让异步流程更直观可控。
• 用split(/\r?\n/)拆分文本，同时兼容两种主流换行格式。
• 用filter过滤空行，避免无效的复制操作。
• 通过Promise.all等待所有复制任务完成后，再输出最终计数，保证统计准确。
• 增加分层错误捕获，分别处理文件读取和单文件复制的错误，便于排查问题。

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