Angular promises in loop

Question

I've got loop function with promises:

for (var a in $scope.atrakcje) {
  if ($scope.atrakcje[a].x && $scope.atrakcje[a].y) {
    App.countDistance($scope.atrakcje[a].x, $scope.atrakcje[a].y).then(function(km) {
      $scope.atrakcje[a].distance = km;
    });
  }
}

The problem with this loop is that all km values are assigned to the last $scope.atrakcje element. It should assign the first promise to the first element, the second to the second and so on. How to do it?

This solves your problem stackoverflow.com/questions/1451009/… — user2422457
– user2422457, Commented Mar 19, 2014 at 12:29
possible duplicate of Javascript closure inside loops - simple practical example — Bergi
– Bergi, Commented Mar 19, 2014 at 15:36

user2422457 · Accepted Answer · 2014-03-19 12:30:25Z

3

Try:

for (var a in $scope.atrakcje) {
  if ($scope.atrakcje[a].x && $scope.atrakcje[a].y) {
    (function(index) {
      App.countDistance($scope.atrakcje[index].x, $scope.atrakcje[index].y).then(function(km) {
        $scope.atrakcje[index].distance = km;
      });
    })(a);
  }
}

answered Mar 19, 2014 at 12:30

user2422457

Sign up to request clarification or add additional context in comments.

Comments

Anders Ekdahl · Accepted Answer · 2014-03-19 12:31:01Z

0

The problem here is that your then function is called after the for loop is finished, which means that when the promise is resolved, the a variable will contain the last key from $scope.atrakcje. Create a closure around it to capture the value of the key, something like:

Object.keys($scope.atrakcje).forEach(function(a) {
  if ($scope.atrakcje[a].x && $scope.atrakcje[a].y) {
    App.countDistance($scope.atrakcje[a].x, $scope.atrakcje[a].y).then(function(km) {
      $scope.atrakcje[a].distance = km;
    });
  }
});

answered Mar 19, 2014 at 12:31

Anders Ekdahl

23k4 gold badges73 silver badges61 bronze badges

Collectives™ on Stack Overflow

Angular promises in loop

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related