如何解决GeoTab API调用报错：需同时提供成功与错误回调

你遇到的第一个错误是因为调用api.call时传了成功回调但没传错误回调，GeoTab的API要求如果使用回调模式，必须同时提供成功和错误两个回调。而你同时混用了async/await和回调函数，不仅导致了API报错，还引发了fuelTypeDevices未定义的问题。

最佳解决方案：使用async/await + try/catch（推荐） #

直接抛弃回调，用async/await的异步范式处理，同时用try/catch捕获错误，并且提前声明变量避免未定义问题：

async function getFuelTypeDevices() {
    // 提前声明变量并初始化，确保始终有定义
    let fuelTypeDevices = [];
    try {
        // 用await直接接收API返回结果，无需回调
        const results = await api.call("Get", {
            typeName: "Device",
            search: {
                groups: fuelType,
                fromDate: new Date().toISOString()
            }
        });
        // 用map简洁提取设备ID
        fuelTypeDevices = results.map(device => device.id);
    } catch (error) {
        // 统一捕获并处理错误，可根据需求打印日志或抛出给上层
        console.error("获取设备数据失败:", error);
        // 如果需要让调用者处理错误，可重新抛出
        // throw error;
    }
    // 返回结果，无论成功失败都有值
    return fuelTypeDevices;
}

改动说明 #

• 变量声明：用let在函数内部声明fuelTypeDevices并初始化为空数组，避免全局变量污染，同时保证即使请求失败，后续代码也不会遇到未定义报错。
• 抛弃回调：await会直接等待API请求完成并返回结果，无需回调函数，自然避免了回调必须配对的问题。
• 错误捕获：try/catch可以捕获所有异步过程中的错误，包括API请求失败、参数错误等，便于统一处理。
• 简化数据处理：用数组map方法替代手动循环，代码更简洁易读。

若坚持使用回调模式（不推荐） #

如果一定要用回调，必须同时传入成功和错误回调，且注意异步逻辑的执行顺序：

function getFuelTypeDevices() {
    let fuelTypeDevices = [];
    api.call("Get", {
        typeName: "Device",
        search: {
            groups: fuelType,
            fromDate: new Date().toISOString()
        }
    }, 
    // 成功回调
    function(results) {
        fuelTypeDevices = results.map(device => device.id);
        // 后续逻辑需在这里处理，因为异步操作完成后才会执行回调
    }, 
    // 错误回调（必须传入）
    function(error) {
        console.error("获取设备数据失败:", error);
        // 处理错误逻辑
    });
}

内容的提问来源于stack exchange，提问作者B. Latif