AngularJS - forEach() with empty array from $resource

Question

I have an array that I get from a service but in the controller I get an empty value in the forEach() function. This is the code.

Controller

Here, both 'products' and 'copyProducts' are empty. I need to work with the 'copyProducts' array into the forEach() function.

app.controller("controllerApp", function($scope, serviceApp){

  var products = serviceApp.query(); 
  $scope.copyProducts = products;

  angular.forEach($scope.copyProducts, function(value, key){
    console.log("object: "+value);
  })

});

Service

app.factory("serviceApp", function($resource){
  return $resource("products.json", {}, {
        getAll: {
            method: "GET",
            isArray: true
        }
    })
})

Aruna · Accepted Answer · 2016-12-22 20:42:54Z

6

Your code is wrong since .query() is asynchronous so it doesn't finish immediately and the result is not ready on the next line synchronously. So it needs a callback function to trigger once it's done with it's work.

serviceApp.query().$promise.then(function(res) {
  $scope.products = res;
  $scope.copyProducts = res;
  angular.forEach($scope.copyProducts, function(item) {
    console.log(item)
  })
});

Alternative:

serviceApp.query({}, function(res, headers){
  //etc
});

By the way, if you want to use the getAll method you have defined in your resource then you would not be using query()

  serviceApp.getAll().$promise.then(function(res){}).....etc

edited Dec 22, 2016 at 20:42

Aruna

12k3 gold badges31 silver badges42 bronze badges

answered Dec 22, 2016 at 20:30

yBrodsky

5,0614 gold badges24 silver badges35 bronze badges

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

1 Comment

shaunhusain Over a year ago

Yup just to point out why it's wrong, .query() is asynchronous so it doesn't finish immediately so it needs a callback function to trigger once it's done with it's work. Also note its good to include error handling functions as well ($qProvider now complains by default if you don't have this)

Collectives™ on Stack Overflow

AngularJS - forEach() with empty array from $resource

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related