データの取得をお待ちしておりますangularJS

Question

私のアプリには少し問題があります。私は取得し、無名関数の外でデータ配列を返したいと思います。私は約束を使用し、私の問題はランダムな値でランダムな配列の長さを返すサービスをしようとすることです。

私はこの問題を知らないが、私は約束を使うかどうか分からない。

getCurrentExchangeTo : function(year, month, country){ 

      var def = $q.defer(); 

      var numberDayPerMonth = [ 
       31, 
       9, 
       31, 
       30, 
       31, 
       30, 
       31, 
       30, 
       31, 
       30, 
       31, 
       30, 
      ]; 

      var vm = this; 
      this.country = country; 

      this.getCurrentExchangeFor = []; 
      var hello = "gello" 

      for(var i = 0; i < numberDayPerMonth.length; i++){ 
       if((i + 1) === month){ 
        for(let j = 1; j < numberDayPerMonth[i]; j++){ 
         $http.get('http://api.fixer.io/2000-02-0' + j + '?symbols=USD').then(function (success) { 
          let countryDay = vm.country         
          vm.getCurrentExchangeFor[j] = success.data.rates[countryDay]; 
          def.resolve(getCurrentExchangeFor) 
         }); 
        } 
       } 
      } 
      return def.promise 
     } 

と

getCurrentExchangeService.getCurrentExchangeTo(2015, 2, 'USD').then(function (data) { 
     console.log(data) 
    }); 

Answer 1

あなたは物事が複雑になり過ぎています。

特に、外部ループは必要ではなく、どちらも遅延型ではありません。

$http.get()は明らかに約束を返し、配列にプッシュして最後に$q.all()で集約することができます。

getCurrentExchangeTo: function(year, month, country) { 
    var numberDayPerMonth = [ 31, 29, 31, 30, 31, 30, 31, 30, 31, 30, 31, 30 ]; 
    var promises = []; 
    for(let i=1; i<=numberDayPerMonth[month-1]; i++) { 
     promises.push($http.get('http://api.fixer.io/2000-02-0' + i + '?symbols=USD').then(function(response) { 
      return response.data.rates[country]; 
     })); 
    } 
    return $q.all(promises); 
}