在Node.js中使用Q同步承诺的麻烦

Question

我目前正在做一个API在Node.JS与框架Sails.js。这是我第一次使用承诺，我在同步我的承诺时遇到了一些麻烦。

我的主要功能如下:

createCard: function(req, res) {
    checkIfUserHasStripeAccount(req.user)
        .then(addCreditCardToStripeAccount())
        .then(function cardCreated() {
            res.send(200, {
                msg: 'Card created'
            });
        })
        .catch(function handleError(err) {
            res.send(err.httpCode, err.msg);
        })
},

显然，如果用户没有信用卡，我无法将信用卡添加到stripe账户。

函数checkIfUserHasStripeAccount()检查帐户是否存在，如果不存在，则创建它。

下面是这部分的代码:

function checkIfUserHasStripeAccount(user) {
    var deferred = q.defer();
    if (!user.idStripe) {
        createStripeAccountToUser(user)
            .then(function(savedUser) {
                deferred.resolve(savedUser);
            })
            .catch(function(err) {
                deferred.reject(err);
            })
    } else {
        deferred.resolve(user);
    }
    return deferred.promise;
}
function createStripeAccountToUser(user) {
    var deferred = q.defer();
    var jsonUserToCreate = {
        description: user.firstname + ' ' + user.surname,
        email: user.email
    };
    stripe.customers.create(jsonUserToCreate, function(err, customer) {
        if (err) {
            deferred.reject({
                httpCode: 500,
                msg: 'some error'
            });
        } else {
            user.idStripe = customer.id;
            user.save(function(err, savedUser) {
                if (err) {
                    deferred.reject({
                        httpCode: 500,
                        msg: 'some error'
                    });
                }
                deferred.resolve(savedUser);
            });
        }
    });
    return deferred.promise;
}

问题是.then(addCreditCardToStripeAccount())在checkIfUserHasStripeAccount()完成之前执行。

我不知道为什么。我认为.then(addCreditCardToStripeAccount())只会在收到拒绝或解析时执行。

Answer 1

你的想法是正确的。问题是你正在调用你的函数，而不是引用它:

.then(addCreditCardToStripeAccount())

应:

.then(addCreditCardToStripeAccount)

我希望这能工作:

createCard: function (req, res) {
    checkIfUserHasStripeAccount(req.user)
    .then(addCreditCardToStripeAccount)
    .then(function cardCreated(){
        res.send(200, {msg: 'Card created'});
    })
    .catch(function handleError(err) {
        res.send(err.httpCode, err.msg);
    })
},

以后，请注意函数名之后的()调用函数，因为JS中的执行顺序将首先计算它，因为它在随后的()中。

在承诺链中，总是只调用第一个函数。例子:

function first () { /*...*/ } // All return promise.
function second() { /*...*/ }
function third () { /*...*/ }
first() // Invoked
    .then(second) // Not invoked. second() will have been bad here.
    .then(third);