使用ExcelJS写入Excel后文件损坏，求解决方案

嘿，我仔细看了你的代码，导致Excel文件损坏的核心原因是异步操作的顺序没处理对，还有一些小细节可能引发问题，咱们一步步来修复：

为什么文件会损坏？ #

• 你直接在readFile的Promise外面调用了callback()，这会导致函数在文件读写操作还没完成时就提前结束，文件写入到一半被中断，自然就损坏了。
• workbook.xlsx.writeFile()是异步的Promise操作，你没等它执行完就break了，写入过程被强行终止，文件结构不完整。
• 遍历行的时候，getColumn(1).values的数组包含索引0的空值（ExcelJS行从1开始），你的循环条件i <= y.length可能会遍历到不存在的行，引发潜在错误。

修复后的完整代码 #

var Excel = require('exceljs');

setExcelData: function (sheetName, keyword, value123, callback) {
    var workbook = new Excel.Workbook();
    
    // 用Promise链式调用确保所有异步操作按顺序执行
    workbook.xlsx.readFile('example.xlsx')
        .then(() => {
            const worksheet = workbook.getWorksheet(sheetName);
            // 用eachRow遍历有效行，避免处理空行
            worksheet.eachRow({ includeEmpty: false }, (row, rowNumber) => {
                if (row.getCell(1).value === keyword) {
                    row.getCell(2).value = value123;
                    // 写入文件后再调用回调，确保操作完成
                    return workbook.xlsx.writeFile('example.xlsx')
                        .then(() => callback(null, "数据写入成功"))
                        .catch(err => callback(err));
                }
            });
        })
        .catch(err => callback(err));
}

几个关键优化点 #

• 改用worksheet.eachRow()遍历行，自动跳过空行，比手动循环数组更安全。
• 所有异步操作都放在Promise链里，确保文件读取、修改、写入的顺序正确，不会中途中断。
• 加入了错误捕获逻辑，不管是读文件还是写文件出错，都能通过callback把错误传递出去，方便排查问题。
• 只有在文件写入完成后才调用callback，外部调用者能准确知道操作是否成功。

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