Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Angular js returning undefined object from factory

I have a controller and factory defined as below.

myApp.controller('ListController', 
        function($scope, ListFactory) {
    $scope.posts = ListFactory.get();
    console.log($scope.posts);
});

myApp.factory('ListFactory', function($http) {
    return {
        get: function() {
            $http.get('http://example.com/list').then(function(response) {
                if (response.data.error) {
                    return null;
                }
                else {
                    console.log(response.data);
                    return response.data;
                }
            });
        }
    };
});

What confuses me is that I get the output undefined from my controller, and then the next line of console output is my list of objects from my factory. I have also tried changing my controller to

myApp.controller('ListController', 
        function($scope, ListFactory) {
    ListFactory.get().then(function(data) {
        $scope.posts = data;
    });
    console.log($scope.posts);
});

But I receive the error

TypeError: Cannot call method 'then' of undefined

Note: I found this information on using a factory through http://www.benlesh.com/2013/02/angularjs-creating-service-with-http.html

like image 625
user1876508 Avatar asked Jul 29 '13 20:07

user1876508


2 Answers

You need to either use a callback function or just put a return before $http.get...

 return $http.get('http://example.com/list').then(function (response) {
     if (response.data.error) {
         return null;
     } else {
         console.log(response.data);
         return response.data;
     }
 });
like image 64
zs2020 Avatar answered Sep 23 '22 14:09

zs2020


$http.get is asynchronous so at the time you try to access it (inside your controller) it may not have data (hence you get undefined).

To solve this I use .then() after I call the factory method from my controller. Your factory then would look something like:

myApp.factory('ListFactory', function($http) {
    return {
        get: function() {
            $http.get('http://example.com/list');
        }
    };
});

And your controller:

myApp.controller('ListController', function($scope, ListFactory) {
    ListFactory.get().then(function(response){
        $scope.posts = response.data;
    });
    // You can chain other events if required
});

Hope it helps

like image 2
HaRoLD Avatar answered Sep 22 '22 14:09

HaRoLD