将数据从服务传递到控制器

Question

i have a factory am passing it to controller LoginCtrl

.factory('Fbdata', function(){
    var service = {
        data: {
          apiData: []
        },
        login: function () {
            facebookConnectPlugin.login(["email"],
              function() {
            facebookConnectPlugin.api("me/?fields=id,email,name,picture", ["public_info","user_birthday"],
                function (results) {
                    service.data.apiData = results;
                    console.log(service.data.apiData);
                    return results;
                },
                function (error) {
                    console.error('FB:API', error);
                });
              },
               function(err) {
                console.error('FB:Login', err);
            });
        }
    };
    return service;
})

LoginCtrl:

.controller('LoginCtrl', function($scope, Fbdata){
    $scope.login = function(){
        if (!window.cordova) {
            var appId = "appId";
            facebookConnectPlugin.browserInit(appId);
        }

        $scope.loginData = Fbdata.login();
        console.log(Fbdata.data.apiData);
        // got empty array []
        $scope.retVal= angular.copy(Fbdata.data.apiData);
    };
})

the Fbdata.data.apiData return empty array and i only could see the returned data from the login success function in the console . my template which is has LoginCtrl as controller:

  <div class="event listening button" ng-click="login();">Login with Facebook</div>
    <h2>{{loginData.name}}</h2>
    <h2>{{retVal.name}}</h2>

Answer 1

There is a variety of ways to achieve this, example:

Now I have never used Cordova Facebook Plugin so I'm not sure if you need to run the api function after the log in, or how those procedures need to be ordered. But I wanted to show you an example of how to retrieve the data from the factory using your code sample. Hope that helps

---

Edit 2 :

I have changed my code to using promises that way we make sure that we don't call one without the other being completed, I am not a fan of chaining the login and api functions within one function since it is possible(?) that you may need to call login() but don't want to call api() , please try my code and paste in your console logs in the bottom of your question.

Factory:

// Let's add promises to our factory using AngularJS $q
.factory('Fbdata', ['$q', function($q){
    // You could also just replace `var service =` with `return` but I thought this
    // would make it easier to understand whats going on here.
    var service = {
        // I generally nest result to keep it clean when I log
        // So results from functions that retrieve results are stored here
        data: {
            login: [],
            api: []
        },
        api: function() {
            var q = $q.defer();
            facebookConnectPlugin.api("me/?fields=id,email,name,picture", ["public_info","user_birthday"],
                function (results) {
                    // assign to the object being returned
                    service.data.api = results;
                    // The data has returned successfully so we will resolve our promise
                    q.resolve(results);
                },
                function (error) {
                    // We reject here so we can inform the user within through the error/reject callback
                    q.reject(error);
                    console.error('FB:API', error);
                });
            // Now that we have either resolved or rejected based on what we received
            // we will return the promise
            return q.promise;
        },
        login: function () {
            var q = $q.defer();
            facebookConnectPlugin.login(["email"], function (results) {
                // assign to the object being returned
                service.data.login = results;
                q.resolve(results);
            }, function(err) {
                q.reject(error);
                console.error('FB:Login', err);
            });
            return q.promise;
        }
    };
    return service;
}])

Controller:

.controller('LoginCtrl', function($scope, Fbdata){
    $scope.login = function(){
        if (!window.cordova) {
            var appId = "appid";
            facebookConnectPlugin.browserInit(appId);
        }
        // By using the promises in our factory be can ensure that API is called after
        // login by doing the following

        // then(success, reject) function allows us to say once we have a return, do this.
        Fbdata.login().then(function () {
            $scope.loginData = Fbdata.data.login;
            // Check what was returned
            console.log('loginData', $scope.loginData);
            Fbdata.api().then(function () {
                $scope.apiData = Fbdata.data.api;
                console.log('apiData', $scope.apiData);
            }, function () {
                // Tell the user that the api failed, if necessary
            });
        }, function () {
            // Tell the user that the log in failed
        });
    };
});