JavaScript Promise回调体外访问数据报错及Excel读取方法问题

这个问题的核心在于Promise是异步执行的——当你在回调函数外面尝试访问ColumnDataList时，里面的Excel读取逻辑还没跑完，数组自然是空的。正确的做法是把数据的处理和返回都放在Promise的链式调用里，或者用async/await语法让代码更直观。

方法一：完善Promise链式返回 #

你可以直接在then回调里返回填充好的ColumnDataList，这样调用这个方法的地方就能通过.then()拿到正确的数据：

this.getColumnData = function(FileName, SheetName, ColumnName) {
    var workbook = new Excel.Workbook();
    return workbook.xlsx.readFile(FileName).then(function() {
        var ColumnDataList = new Array();
        var worksheet = workbook.getWorksheet(SheetName);
        for(var v=2; v<=worksheet.actualRowCount; v++) {
            ColumnDataList.push(worksheet.getCell(ColumnName+v).value);
        }
        // 在这里返回填充好的数组
        return ColumnDataList;
    });
};

调用的时候这样用：

this.getColumnData("test.xlsx", "Sheet1", "A")
    .then(function(data) {
        // 这里的data就是已经填充好的ColumnDataList
        console.log(data);
    })
    .catch(function(err) {
        // 别忘了捕获可能的错误，比如文件不存在、工作表找不到等
        console.error("读取数据出错：", err);
    });

方法二：用async/await简化代码（更推荐） #

如果你的运行环境支持ES2017+（现代浏览器、Node.js 8+），可以用async/await把异步代码写成同步风格，可读性更强：

this.getColumnData = async function(FileName, SheetName, ColumnName) {
    var workbook = new Excel.Workbook();
    // await会等待Promise完成再继续执行
    await workbook.xlsx.readFile(FileName);
    var ColumnDataList = new Array();
    var worksheet = workbook.getWorksheet(SheetName);
    for(var v=2; v<=worksheet.actualRowCount; v++) {
        ColumnDataList.push(worksheet.getCell(ColumnName+v).value);
    }
    return ColumnDataList;
};

调用的时候：

// 注意调用async函数的地方也要用await，或者用.then()
async function useData() {
    try {
        var data = await this.getColumnData("test.xlsx", "Sheet1", "A");
        console.log(data);
    } catch(err) {
        console.error("读取数据出错：", err);
    }
}

useData();

关键注意点 #

• 永远不要在Promise回调外面依赖异步操作生成的数据——异步代码的执行顺序是“先注册回调，继续执行后续代码，等异步操作完成后再跑回调”，所以回调外的变量还没被赋值。
• 一定要处理错误：无论是用.catch()还是try/catch，避免因为文件读取失败、工作表不存在等情况导致程序崩溃。

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