AngularJs : 使用条件在服务中回调或承诺

Question

我有一个特定的案例，我不确定我是否应该实施回调或承诺。我对承诺完全陌生，才刚刚开始理解它的概念。因此，我不想陷入 anti patterns.

我也阅读并重新阅读了 angular 文档 $q。

这就是我想要实现的：

如果使用承诺：

OauthService.token().then(function(access_token){
    config.url = config.url+'?access_token='+access_token;
});

return config;

如果使用回调：

OauthService.token(function(access_token){
    config.url = config.url+'?access_token='+access_token;
});

return config;

Oauth 服务不仅仅是一个 http 请求，如果其中确实有一些条件可以让我认为我应该使用回调而不是承诺。

所以我的服务现在使用回调。

OauthService.js :

app.factory('OauthService', function($http, $localStorage) {

return {
    token : function(callback){

        // Get actual token
        access_token = $localStorage.getObject('access_token');
        // Get actual identity
        identity_token = $localStorage.getObject('identity_token');

        // IF no user logged
        if(isObjectEmpty(identity_token)){

            // IF access_token does NOT exist OR will expires soon
            if( isObjectEmpty(access_token) || Date.now() > (access_token.expires_at - (600*1000)) ){

                // Create an anonymous access_token
                $http
                    .get(domain+'/oauth/v2/token?client_id='+public_id+'&client_secret='+secret+'&grant_type=client_credentials')
                    .then(function (response) {

                        $localStorage.setObject('access_token', {
                            key: response.data.access_token,
                            type: 'anonymous',
                            expires_at: Date.now()+(response.data.expires_in*1000)
                        });

                        // return access token here
                        callback(response.data.access_token);

                    });
            }

        }
        // IF user is logged
        else{

            // IF access_token does NOT exist OR will expires soon OR is anonymous
            if( isObjectEmpty(access_token) || Date.now() > (access_token.expires_at - (600*1000)) || access_token.type == 'anonymous' ){
                // Create an access_token with an identity
                $http
                    .get(domain+'/oauth/v2/token?client_id='+public_id+'&client_secret='+secret+'&api_key='+identity_token+'&grant_type=http://oauth2.dev/grants/api_key')
                    .then(function (response) {

                        $localStorage.setObject('access_token', {
                            key: response.data.access_token,
                            type: 'identity',
                            expires_at: Date.now()+(response.data.expires_in*1000)
                        });

                        // return access token here
                        callback(response.data.access_token);

                    });
            }

        }

        // return access token here (if the previous token has not changed of type or expired)
        callback(access_token.key);

    }
};

})

所以，如果我更愿意遵守承诺，我应该如何实现它？

Answer 1

在您的操作中设置条件与回调与承诺无关。普通回调是一种糟糕的异步操作方式，您应该尽可能使用 promises。

您可以重写 token 方法以使用这样的承诺：

app.factory('OauthService', function($http, $localStorage, $q) {
    return {
        token : function(callback){

            // Get actual token
            access_token = $localStorage.getObject('access_token');
            // Get actual identity
            identity_token = $localStorage.getObject('identity_token');

            // IF no user logged
            if(isObjectEmpty(identity_token)){

                // IF access_token does NOT exist OR will expires soon
                if( isObjectEmpty(access_token) || 
                    Date.now() > (access_token.expires_at - (600*1000)) ){

                    // Create an anonymous access_token
                    return $http
                        .get(domain+'/oauth/v2/token?client_id='+public_id + 
                             '&client_secret=' + secret + '&grant_type=client_credentials')
                        .then(function (response) {
                            $localStorage.setObject('access_token', {
                                key: response.data.access_token,
                                type: 'anonymous',
                                expires_at: Date.now() + 
                                       (response.data.expires_in * 1000)
                            });

                            return response.data.access_token;
                        });
                }
            }

            // IF user is logged
            else {
                // IF access_token does NOT exist OR will expire soon OR is anonymous
                if( isObjectEmpty(access_token) || 
                    Date.now() > (access_token.expires_at - (600*1000)) || 
                    access_token.type == 'anonymous' ){

                    // Create an access_token with an identity
                    return $http
                        .get(domain+'/oauth/v2/token?client_id='+public_id+'&client_secret='+secret + 
                             '&api_key='+identity_token+'&grant_type=http://oauth2.dev/grants/api_key')
                        .then(function (response) {
                            $localStorage.setObject('access_token', {
                                key: response.data.access_token,
                                type: 'identity',
                                expires_at: Date.now()+
                                      (response.data.expires_in * 1000)
                            });

                            return response.data.access_token;    
                        });
                }

            }

            // return access token here (if the previous token has not changed of type or expired)
            return $q.when(access_token.key);    
        }
    };
});

然后您可以进行一些重构以将其简化为：

app.factory('OauthService', function($http, $localStorage, $q) {
    function expiresSoon(access_token) {
        return Date.now() > (access_token.expires_at - (600*1000));
    }

    function getAccessToken(url, type) {
        return $http
            .get(url)
            .then(function (response) {
                $localStorage.setObject('access_token', {
                    key: response.data.access_token,
                    type: type,
                    expires_at: Date.now() + 
                        (response.data.expires_in * 1000)
            });

                return response.data.access_token;
            });
    }

    return {
        token : function(callback){

            // Get actual token
            access_token = $localStorage.getObject('access_token');
            // Get actual identity
            identity_token = $localStorage.getObject('identity_token');

            // IF no user logged
            if(isObjectEmpty(identity_token)){

                // IF access_token does NOT exist OR will expires soon
                if( isObjectEmpty(access_token) || expiresSoon(access_token) ) {
                    var url = domain + '/oauth/v2/token?client_id=' + public_id + 
                              '&client_secret=' + secret + '&grant_type=client_credentials';
                    // Create an anonymous access_token
                    return getAccessToken(url, 'anonymous');
                }
            }

            // IF user is logged
            else {
                // IF access_token does NOT exist OR will expire soon OR is anonymous
                if( isObjectEmpty(access_token) || 
                    expiresSoon(access_token) || 
                    access_token.type == 'anonymous' ){

                    var url = domain+'/oauth/v2/token?client_id=' + public_id+
                              '&client_secret='+secret + 
                              '&api_key='+identity_token + 
                              '&grant_type=http://oauth2.dev/grants/api_key';

                    // Create an access_token with an identity
                    return getAccessToken(url, 'identity');
                }
            }

            // return access token here (if the previous token has not changed of type or expired)
            return $q.when(access_token.key);    
        }
    };
});

AngularJs : 使用条件在服务中回调或承诺

AngularJs : Callback or promise in a service using conditions

javascript

callback

angularjs

angular-promise