get data promise angularJS

Question

I have a little problem on my app. I would like to get and return the data array outside the anonymous function. I use promise, aim my problem is when I try my service it return a random array lenght with random values.

I do not know the problem, I do not know if I use the promise.

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
        }

and

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

javascript is async.. when u call the api the loop continues and keeps calling the api. Each call returns in different time.. so you get random numbers... — ziaulain
– ziaulain, Commented May 31, 2017 at 20:09
This right .. But do you have an idea for result my problem ?? — simonmnt
– simonmnt, Commented May 31, 2017 at 20:49
Your return statement should come after the resolve statement in your loop. — Mike Feltman
– Mike Feltman, Commented May 31, 2017 at 21:11
@MikeFeltman I return my variable after my loop, I don't understand.. — simonmnt
– simonmnt, Commented May 31, 2017 at 21:34

Roamer-1888 · Accepted Answer · 2017-05-31 22:12:49Z

3

You are overcomplicating things.

In particular, the outer loop isn't necessary, neither is the Deferred.

$http.get() clearly retuns a promise, which can be pushed onto an array and finally aggregated with $q.all().

As far as I can tell, you want the following :

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);
}

edited May 31, 2017 at 22:12

answered May 31, 2017 at 21:57

Roamer-1888

19.3k5 gold badges35 silver badges45 bronze badges

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

1 Comment

Roamer-1888 Over a year ago

You need to check that numberDayPerMonth array - April, June, September & November have 31 days, Also, for February, with reference to this answer, you can write [..., leapYear(year)?29:28, ...].

Collectives™ on Stack Overflow

get data promise angularJS

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related