Javascript异步功能：回调不起作用

Question

我正在尝试从Javascript中的异步函数返回结果。

我已经看到了这个问题： 如何从异步调用返回响应？ ，而我正在尝试实现回调解决方案，但是出了点问题。

这是我的代码：

function getLocation(callback){
    var lat;
    var long;
    if (navigator.geolocation) {
        navigator.geolocation.getCurrentPosition(function(position)
                {
                    lat = position.coords.latitude;
                    long = position.coords.longitude;
                }, function()
                {
                    console.log('Please enable geolocation in your browser.');
                });
    } else {
        alert('It seems like geolocation is not enabled in your browser.');
    }
var res = {
        "lat": lat,
        "long": long
};
callback(res);
}

function getEventNearYou(){
    var list = [];
    getLocation(function(obj){
        var lat = obj.lat;
        var long = obj.long;
        $.ajax({
                url : 'http://www.skiddle.com/api/v1/events/search/?api_key=myapikey' + '&latitude=' + lat + '&longitude=' + long + '&radius=800&eventcode=LIVE&order=distance&description=1',
                type : "GET",
                async : false,
                success : function(response) {
                        $(response).find("results").each(function() {
                        var el = $(this);
                        var obj = {
                                    "eventname" : el.find("eventname").text(),
                                    "imageurl" : el.find("imageurl").text(),
                                  }
                        list.push(obj);
                        });
                        } 
                });
        return list;
    });
}

基本上，我想找到我的当前位置，并向www.skiddle.com创建HTTP get请求以检索该位置附近的事件。

这就是我所谓的函​​数：

var x = getEventNearYou();

但是我似乎犯了一个错误，因为我遇到了错误的请求错误（未定义lat和long ）。

Answer 1

您使用回调的方式会削弱其用途。 您仍在以同步样式编写代码。 您应该继续参考： 如何从异步调用返回响应？

同时，我将重组您的代码以显示其完成方式。

1. 根据经验，如果函数内的任何代码使用回调，则该外部函数也需要接受回调作为参数。

2. 如果在回调函数中为变量赋值，则不会在回调函数之外定义该变量（因为异步代码始终在同步代码之后运行）。

3. 不要使用async: false

function getLocation(callback){
    if (navigator.geolocation) {
        navigator.geolocation.getCurrentPosition(function (position) {
            var lat = position.coords.latitude;
            var long = position.coords.longitude;
            var res = { "lat": lat, "long": long }; 
            callback(res);
        }, function() {
            console.log('Please enable geolocation in your browser.');
        });
    } else {
        alert('It seems like geolocation is not enabled in your browser.');
    }
}

function getEventNearYou(callback) {
    getLocation(function(pos){
        var lat = pos.lat;
        var long = pos.long;
        $.ajax({
            url: 'http://www.skiddle.com/api/v1/events/search/?api_key=myapikey' + '&latitude=' + lat + '&longitude=' + long + '&radius=800&eventcode=LIVE&order=distance&description=1',
            type: "GET",
            success: function(response) {
                var list = [];
                $(response).find("results").each(function() {
                    var el = $(this);
                    var obj = {
                        "eventname": el.find("eventname").text(),
                        "imageurl" : el.find("imageurl").text(),
                    };
                    list.push(obj);
                });
                callback(list);
            } 
        });
    });
}

然后使用回调函数调用getEventNearYou ：

getEventNearYou(function (events) {
    // code which relies on `events` can be called from here
});

Answer 2

只是在4castle的出色答案中添加了更多解释。 在您编写的代码中，用Javascript调用getLocation()函数时，将发生以下情况：

1. var lat和var long用undefined值实例化。
2. 调用navigator.geolocation.getCurrentPosition()会花费很长时间。
3. 同时，Javascript寻找其他事情要做，并移至函数的下一行。
4. 使用当前未定义的lat和long实例化res对象。 也就是说， res是： { "lat": undefined, "long": undefined }
5. callback函数在res上调用。 这意味着使用lat和long未定义值callback 。
6. navigator.geolocation.getCurrentPosition()最终完成了所要做的工作，并将值分配给lat和long ，但是现在为时已晚，因为您的回调函数已经在未定义的值上调用了。

在4castle的修改后的代码中，步骤4-5被放置在navigator.geolocation.getCurrentPosition()的匿名回调函数中，这意味着直到getCurrentPosition()完成获取位置后，这些行才会执行。