回调函数访问闭包变量?

时间:2013-04-30 16:44:47

标签: javascript angularjs closures promise

以下代码有一个错误,但它打开了一个有趣的问题。它使用angular的$ http服务返回一个promise。解析后,回调函数无权访问s1变量。

    var ids = [];
    ids = ['81074'];

    // show what's in
    for (var s=0;s<ids.length;s++) {
        var s1 = s;
        $http.get('http://mydomain.com/api/catalog/product/' + ids[s]).
                 success(function(data) {
        // console.log(s1); <--- reference error
            $scope.products[s].setProductData(data);

        }).error(function(err){
                console.log('--------- error',err);
            });
    };

s1给出了ReferenceError:s1未在调试器中定义

有趣的是,它确实可以访问for循环变量s,它总是1 - 正如预期的那样,因为承诺在增加后得到解决(因此错误BTW)

有人可以解释原因吗?

感谢

利奥尔

1 个答案:

答案 0 :(得分:9)

这是for循环中异步回调的经典问题。 ss1都只声明一次。即使你在循环中声明var s1;,JavaScript也没有块范围,所以它是无关紧要的。

这意味着所有迭代都共享相同的变量ss1,因此当回调完成时,$scope.products[s]正在查找ids.length + 1产品。

相反,请执行:

var ids = [];
ids = ['81074'];

function launchRequest(s) {
    $http.get('http://example.com/api/catalog/product/' + ids[s]).success(function (data) {
        $scope.products[s].setProductData(data);
    }).error(function (err) {
        console.log('--------- error', err);
    });
}

// show what's in
for (var s = 0; s < ids.length; s++) {
    launchRequest(s);
}

...在launchRequest中引入了一个新的函数级别范围,这意味着当回调结算时,s在函数内仍然是s