Question

我正在尝试在Jquery的每个循环中对API进行外部AJAX调用。

这是我到目前为止的代码。

getStylesInfo(tmpMake, tmpModel, tmpModelYear, tmpSubmodel).done(function(data){
    var holder = [];

    $.each(styles, function(index, value) {
        var tempValue = value;
        var temp = getNavigationInfo(value.id);

        $.when(temp).done(function(){
            if(arguments[0].equipmentCount == 1){
                holder.push(tempValue);
                console.log(holder);
            }
        });
    });
});

console.log(holder);

function getStylesInfo(make, model, year, submodel){
    return $.ajax({
    type: "GET",
    url: apiUrlBase + make + '/' + model + '/' + year + '/' + 'styles?  fmt=json&' + 'submodel=' + submodel + '&api_key=' + edmundsApiKey + '&view=full',
   dataType: "jsonp"
});   


function getNavigationInfo(styleId){
    return $.ajax({
    type: "GET", 
    url: apiUrlBase + 'styles/' + styleId + '/equipment?availability=standard&name=NAVIGATION_SYSTEM&fmt=json&api_key=' + edmundsApiKey,
    dataType: "jsonp"
});

getStylesInfo（）返回类似于此的内容。一组对象，其中包含有关汽车模型的信息。

var sampleReturnedData = [{'drivenWheels': 'front wheel drive', 'id': 234321}, {'drivenWheels': 'front wheel drive', 'id': 994301}, {'drivenWheels': 'rear wheel drive', 'id': 032021}, {'drivenWheels': 'all wheel drive', 'id': 184555}];

我试图遍历sampleReturnedData并使用getNavigationInfo（）函数将每个id作为参数用于不同的AJAX调用。

我想循环查看结果并进行检查。如果是，那么我想将整个对象推到持有者数组。

问题是该函数外的console.log（holder）返回一个空数组。 if语句中的console.log（holder）工作正常。

我不确定这是否是范围/提升问题或我使用延迟的方式有问题？

我已阅读this个问题，很多人都喜欢这个问题。他们建议使用

async:false

或者更好地重写代码。我已多次尝试并使用控制台调试器。我不想把它设置为假。我不确定到底发生了什么。

我还通过this文章了解了提升。

我认为这与推迟有关，但我没有足够的JS知识来解决这个问题。

谢谢！

Answer 1

我不确定这是否是范围/提升问题或我使用延迟的方式有问题？

事实上，它是：

holder仅在回调函数中声明（作为局部变量），因此它在函数外部undefined。
并且在异步回调函数向数组填充值之前执行console.log，因此即使holder在范围内，它仍然是空的。另请参阅Why is my variable unaltered after I modify it inside of a function? - Asynchronous code reference

所以你应该做的就是重写代码以正确使用promises： - ）

getStylesInfo(tmpMake, tmpModel, tmpModelYear, tmpSubmodel).then(function(data) {
    var holder = [];
    var promises = $.map(data.styles, function(value, index) {
        return getNavigationInfo(value.id).then(function(v){
            if (v.equipmentCount == 1)
                holder.push(value);
        });
    });
    return $.when.apply($, promises).then(function() {
        return holder;
    }); // a promise for the `holder` array when all navigation requests are done
}).then(function(holder) {
    console.log(holder); // use the array here, in an async callback
});

Javascript范围/提升或承诺/延期？

1 个答案: