функция обратного вызова доступа к переменным замыкания?

Question

В следующем фрагменте кода есть ошибка, но он открыл интересный вопрос. Его использование угловой $ http службы, которая возвращает обещание. После разрешения функция обратного вызова не имеет доступа к переменной 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 не определено в отладчике

И что интересно, у него есть доступ к для цикла переменной s, которая всегда 1 - как и следовало ожидать, так как обещание получил решен после того, как его прирост (отсюда ошибка BTW)

Может кто-нибудь объяснить, почему?

благодаря

Лиор

Answer 1

Это классическая проблема с асинхронными в for цикле. Оба s и s1 объявляются только один раз. Несмотря на то, что в цикле вы объявляете var s1;, JavaScript не имеет границ блока, поэтому он не имеет значения.

Это означает, что все итерации одни и те же переменные s и s1, поэтому, когда обратный вызов завершается, $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 внутри функции обратного вызова, когда рассасывается.