Angular JSの連鎖約束の問題

Question

私は論理的な問題があります。

私はlocalStorageにオブジェクトの配列を持っています。私がしたいのは、それぞれが効果的にAPI呼び出しを行い、localStorageに新しいアイテムをプッシュし、すべてが完了したらroute to a new componentだけです。

if(migrationName){ 

     angular.forEach(JSON.parse($window.localStorage.selectedItems), function(item) { 
      var url = '/api/get_all_prices/'+item.details.id+'/us-1'; 
      HttpWrapper.send(url,{"operation":'GET'}).then(function(pricingOptions){ 
       item.selectedMapping = pricingOptions[0]; 
       vm.selectedItems[type].push(item); // Store it in a variable first  
       $window.localStorage.setItem('selectedItems',JSON.stringify(item)); 
      }); 

     }); 
     $rootRouter.navigate(["MigrationRecommendation"]); // Route once everything is done 
} 

私はこれが間違っていることを知っています。

私はループのたびにlocalStorageを設定していますが、配列内のすべてが実行された後は処理ができません。

ロジックはどのように変更できますか？

Answer 1

あなたはArray.prototype.mapで作成した約束の配列で$ q.allを使用する必要があります。

var items = JSON.parse($window.localStorage.selectedItems) 

var promises = items.map(function(item) { 
    var url = '/api/get_all_prices/' + item.details.id + '/us-1'; 
    return HttpWrapper.send(url, {"operation": 'GET'}).then(function(pricingOptions) { 
    item.selectedMapping = pricingOptions[0]; 
    vm.selectedItems[type].push(item); // Store it in a variable first  
    $window.localStorage.setItem('selectedItems', JSON.stringify(item)); 
    }); 
}); 

$q.all(promises).then(function() { 
    $rootRouter.navigate(["MigrationRecommendation"]);  
})