web-dev-qa-db-ja.com

Sequelizeのトランザクションでループとプロミスを使用する

私は現在、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);
        });
javascriptnode.jsexpresspromisesequelize.js
16
ReturnToZero

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データになります

32
Datsik

Sequelizeに同梱されているブルーバードの組み込みループ構成を使用する必要があります。

    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
6
Benjamin Gruenbaum

Node.jsの実装によっては、これが役立つ場合があります。私は、express、POSTGRES、sequelizeを使用して同じ設定をしています。

個人的には、読みやすさの点で、then/catchよりもasync/await（ES6）の実装をお勧めします。また、外部から呼び出すことができる関数を作成すると、再利用性が向上します。

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;
    }
  }
}
2
Ryan

最初： 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);
      }
    });
  });
}
0
Eri Jonhson