如何验证Sequelize事务并使其看起来不错

时间:2017-06-29 13:37:30

标签: javascript node.js validation transactions sequelize.js

我试图在经过几年的努力之后重新学习NodeJS,所以我正在建立一个小型银行网站作为测试。我决定将Sequelize用于我的ORM,但是我以一种我喜欢的方式在人们之间汇款时遇到了一些麻烦。 这是我的第一次尝试:

// myUsername - who to take the money from
// sendUsername - who to send the money to
// money - amount of money to be sent from `myUsername`->`sendUsername`
// Transaction is created to keep a log of banking transactions for record-keeping.
module.exports = (myUsername, sendUsername, money, done) => {
    // Create transaction so that errors will roll back
    connection.transaction(t => {
        return Promise.all([
            User.increment('balance', {
                by: money,
                where: { username: myUsername },
                transaction: t
            }),
            User.increment('balance', {
                by: -money,
                where: { username: sendUsername },
                transaction: t
            }),
            Transaction.create({
                fromUser: myUsername,
                toUser: sendUsername,
                value: money
            }, { transaction: t })
        ]);
    }).then(result => {
        return done(null);
    }).catch(err => {
        return done(err);
    });
};

这很有效,但是当它增加时它没有验证模型。当模型未验证时,我希望事务失败。我的下一次尝试是去回调,这里显示(相同的函数头):

connection.transaction(t => {
        // Find the user to take money from
        return User
        .findOne({ where: { username: myUsername } }, { transaction: t })             .then(myUser => {
                    // Decrement money
                    return myUser
                    .decrement('balance', { by: money, transaction: t })
                    .then(myUser => {
                            // Reload model to validate data
                            return myUser.reload(myUser => {
                                    // Validate modified model
                                    return myUser.validate(() => {
                                            // Find user to give money to
                                            return User
                                            .findOne({ where: { username: sendUsername } }, { transaction: t })
                                            .then(sendUser => {
                                                    // Increment balance
                                                    return sendUser
                                                    .increment('balance', { by: money, transaction: t })
                                                    .then(sendUser => {
                                                            // Reload model
                                                            return sendUser.reload(sendUser => {
                                                                    // Validate model
                                                                    return sendUser.validate(() => {
                                                                            // Create a transaction for record-keeping
                                                                            return Transaction
                                                                            .create({
                                                                                    fromUser: myUser.id,
                                                                                    toUser: sendUser.id,
                                                                                    value: money
                                                                            }, { transaction: t });
                                                                    });
                                                            });
                                                    });
                                            });
                                    });
                            });
                    });
            });
    }).then(result => {
            return done(null);
    }).catch(err => {
            return done(err);
    });

这是有效的,因为钱仍然在人们之间转移,但它仍然没有验证模型。我认为原因是.validate().reload()方法无法在其上添加transaction: t参数。 我的问题是,如果有一种方法可以在交易中进行验证,但我也想帮助修复这个“回调地狱”。同样,我有一段时间没有完成JS,所以现在我可能已经知道了更好的方法。

谢谢!

1 个答案:

答案 0 :(得分:2)

我相信您无法通过Model的{​​{1}}和increment获取验证,并且需要拥有实例。在某些续集模型方法中,您可以配置要运行的验证,但它不是look like it here

我会这样做

decrement