在 Sequelize 的事务中使用循环和承诺

Question

我目前正在构建一个 Nodejs、Express、Sequelize (w. PostgreSQL) 应用程序，并且在将 Promise 与事务和循环一起使用时遇到了一些问题。

我正在尝试弄清楚如何在事务中使用 for 循环。我正在尝试遍历成员列表并在数据库中为每个成员创建一个新用户。

我知道下面的代码是错误的，但它显示了我正在尝试做的事情。

谁能指出我正确的方向？

        var members = req.body.members;
        models.sequelize.transaction(function (t) {
            for (var i = 0; i < members.length; i++) {
                return models.User.create({'firstname':members[i], 'email':members[i], 'pending':true}, {transaction: t}).then(function(user) {
                    return user.addInvitations([group], {transaction: t}).then(function(){}).catch(function(err){return next(err);});
                })
            };
        }).then(function (result) {
            console.log("YAY");
        }).catch(function (err) {
            console.log("NO!!!");
            return next(err);
        });

Answer 1

你应该使用Promise.all

    var members = req.body.members;
    models.sequelize.transaction(function (t) {
        var promises = []
        for (var i = 0; i < members.length; i++) {
            var newPromise = models.User.create({'firstname':members[i], 'email':members[i], 'pending':true}, {transaction: t});
           promises.push(newPromise);
        };
        return Promise.all(promises).then(function(users) {
            var userPromises = [];
            for (var i = 0; i < users.length; i++) {
                userPromises.push(users[i].addInvitations([group], {transaction: t});
            }
            return Promise.all(userPromises);
        });
    }).then(function (result) {
        console.log("YAY");
    }).catch(function (err) {
        console.log("NO!!!");
        return next(err);
    });

我认为您不需要在后续交易中catch，因为我认为它会跳出交易的陷阱

抱歉格式化。在移动。

Promise.all 会在运行 .then 之前等待所有的 Promise 返回（或失败），并且.then 回调将是来自每个数组的所有 Promise 数据

Answer 2

您需要使用 sequelize 附带的 bluebird 内置循环结构：

    var members = req.body.members;
    models.sequelize.transaction(t => 
      Promise.map(members, m => // create all users
        models.User.create({firstname: m, email: m, 'pending':true}, {transaction: t})
      ).map(user => // then for each user add the invitation
         user.addInvitations([group], {transaction: t}) // add invitations
    )).nodeify(next); // convert to node err-back syntax for express

Answer 3

根据您对 Node.js 的实现，这可能会有所帮助。我使用 express、POSTGRES 和 sequelize 进行了相同的设置。

我个人更喜欢 async/await (ES6) 实现而不是 then/catch，因为它更易于阅读。同时创建一个可以在外部调用的函数也提高了可重用性。

async function createMemeber(req) {
let members = req.body.members;
  for (var i = 0; i < members.length; i++) {
    // Must be defined inside loop but outside the try to reset for each new member;
    let transaction = models.sequelize.transaction();
    try { 
      // Start transaction block.
      let user = await models.User.create({'firstname':members[i],  'email':members[i], 'pending':true}, {transaction});
      await user.addInvitations([group], {transaction}));

      // if successful commit the record. Else in the catch block rollback the record.
      transaction.commit();
      // End transaction block.
      return user;
    } catch (error) { 
      console.log("An unexpected error occurred creating user record: ", error);
      transaction.rollback();
      // Throw the error back to the caller and handle it there. i.e. the called express route.
      throw error;
    }
  }
}

Answer 4

如果有人正在寻找使用 async 和 await 的 typescript v4.0.5 的解决方案，这对我来说是可行的。也许你也可以在你的 javascript 应用程序上使用它，但这取决于它的版本。

const array = ['one','two','three'];
const createdTransaction = sequelize.transaction();
    
const promises = array.map(async item => {
      await model.create({
          name: item,
      },
      { transaction: createdTransaction },
     );
});

Promise.all(promises).then(async values => {
    await createdTransaction.commit();
});

Answer 5

第一：https://caolan.github.io/async/docs.html

所以，很容易：

// requiring...
const async = require('async');

// exports...
createAllAsync: (array, transaction) => {
  return new Promise((resolve, reject) => {
    var results = [];
    async.forEachOf(array, (elem, index, callback) => {
      results.push(models.Model.create(elem, {transaction}));
      callback();
    }, err => {
      if (err) {
        reject(err);
      }
      else {
        resolve(results);
      }
    });
  });
}