Node.js重命名文件后打开遇异步问题：文件未就绪

你的核心问题是writeFiles函数没有正确等待所有文件操作完成就返回，导致ShowFile提前启动外部阅读器，此时文件可能还未就绪。结合你提供的简化代码，主要存在以下问题：

• writeFileSync是同步方法，不需要传入回调函数，你写的回调永远不会执行
• 即使实际代码中用了异步的fs.rename/fs.writeFile，当前的Promise也没有等待这些异步操作完成就调用了fullfill
• 错误处理不规范，同步方法的错误应该用try/catch捕获，异步方法要在回调里处理并调用reject

修复方案 #

情况1：使用同步文件操作（适合简单场景） #

如果你的文件操作逻辑适合用同步方法，直接去掉多余的Promise，让writeFiles同步完成后再执行后续操作：

ShowFile(oldName, newname).then(([child, tempfiles]) => {
    child.on('close', (code) => {
        for (let x of tempfiles) {
            try {
                fs.renameSync(x.newname, x.oldname);
            } catch (err) {
                console.error('重命名失败:', err);
            }
        }
    });
});

async function ShowFile(oldName, newname) {
    let tempfiles = writeFiles(oldName, newname);
    // 确保文件操作完成后再启动子进程
    let child = cp.exec(`${getCommandLine()} "${pathToPdfReader}" "${newname}"`);
    return [child, tempfiles];
}

function writeFiles(oldName, newname) {
    let tempfiles = []; // 实际从其他来源获取的文件名数组
    try {
        for (let x of tempfiles) {
            fs.renameSync(oldName, newname);
            // 同步写入空文件，无回调
            fs.writeFileSync(oldName, '');
        }
        return tempfiles;
    } catch (err) {
        console.error('文件操作失败:', err);
        throw err;
    }
}

情况2：使用异步文件操作（适合高并发场景） #

如果用异步方法，必须等待所有异步操作完成后再resolve Promise，推荐用fs.promises（Node.js 10+支持）简化异步流程：

const fs = require('fs').promises;

ShowFile(oldName, newname).then(([child, tempfiles]) => {
    child.on('close', async (code) => {
        for (let x of tempfiles) {
            try {
                await fs.rename(x.newname, x.oldname);
            } catch (err) {
                console.error('重命名失败:', err);
            }
        }
    });
});

async function ShowFile(oldName, newname) {
    let tempfiles = await writeFiles(oldName, newname);
    let child = cp.exec(`${getCommandLine()} "${pathToPdfReader}" "${newname}"`);
    return [child, tempfiles];
}

async function writeFiles(oldName, newname) {
    let tempfiles = []; // 实际从其他来源获取的文件名数组
    const tasks = [];
    for (let x of tempfiles) {
        // 将每个异步操作包装成Promise，加入任务列表
        tasks.push(
            fs.rename(oldName, newname)
                .then(() => fs.writeFile(oldName, ''))
        );
    }
    // 等待所有异步任务完成
    await Promise.all(tasks);
    return tempfiles;
}

关键注意点 #

• 同步方法会阻塞事件循环，适合文件操作少的场景；异步方法非阻塞，适合大量文件操作
• 启动子进程前必须确保所有文件写入/重命名操作完全完成，await是关键
• 错误处理要到位，避免未捕获的异常导致脚本崩溃

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