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

Question

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

我的主要功能如下：

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);
        })
},

如果用户没有信用卡，我显然无法将信用卡添加到条带帐户。

函数 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 中的执行顺序将首先评估它，因为它位于 then 的 () 内。

在 Promise 链中，始终只调用第一个函数。示例：

function first () { /*...*/ } // All return promise.
function second() { /*...*/ }
function third () { /*...*/ }

first() // Invoked
    .then(second) // Not invoked. second() will have been bad here.
    .then(third);