web-dev-qa-db-ja.com

約束の中の約束

私はPromiseでこのコードを書こうとしています。しかし、Promiseとloopの中にpromiseを書く方法がわかりません。このように考えてみましたが、insertBook関数が非同期になります。 bookIdを同期的に取得するにはどうすればよいですか？

update: function(items, quotationId) {
  return new Promise(function(resolve, reject) {
    knex.transaction(function (t) {
      Promise.bind(result).then(function() {
        return process１
      }).then(function() {
        return process２
      }).then(function() {
        var promises = items.map(function (item) {
          var people = _.pick(item, 'familyName', 'firstNumber', 'tel');
          if (item.type === 'book') {
            var book = _.pick(item, 'name', 'bookNumber', 'author');
            var bookId = insertBook(t, book);
            var values = _.merge({}, people,  {quotation: quotationId}, {book: bookId});
          } else {
            var values = _.merge({}, people,  {quotation: quotationId});
          }
          return AModel.validateFor(values);
        });
        return Promise.all(promises);
      }).then(function(items) {
        var insertValues = items.map(function (item) {
          return People.columnize(item);
        });
        return knex('people').transacting(t).insert(insertValues);
      }).then(function() {
        return process5
      }).then(function() {
        ...........

      }).then(function() {
        t.commit(this);
      }).catch(t.rollback);
    }).then(function (res) {
      resolve(res);
    }).catch(function(err) {
      reject(err);
    });
  });
}

function insertBook(t, book){
  return Promise.bind(this).then(function () {
    return Book.columnizeFor(book);
  }).then(function (value) {
    return knex('book').transacting(t).insert(value, "id");
  });
}
javascriptpromisesails.jsknex.js
9
通りすがりのおっさん

Bookidを同期的に取得する必要はなく、非同期的に正しく処理できます。また、すべての本の挿入を順番に実行することもできるので、Promise.allの部分をリファクタリングしました。 （あなたにアイデアを与えるためだけにそれを行いました。並列挿入が許可されている場合、Promise.allは正常に機能するはずです）。さらに、Promise.bindは使用しないでください。正直なところ、私はそれが何をするのかさえ知りません、確かに一つのこと：それは標準的な約束では機能しません。だからここに私がそれがどのように機能するべきだと思うかの例があります：

update: function(items) {
  return new Promise(function(resolve) {
    knex.transaction(function (t) {
      resolve(Promise.resolve().then(function() {
        return process1;
      }).then(function() {
        return process2;
      }).then(function() {
        var q = Promise.resolve(), results = [];
        items.forEach(function (item) {
          q = q.then(function() {
            var book = _.pick(item, 'name', 'bookNumber', 'author');
            return insertBook(t, book);
          }).then(function(bookId) {
            var people = _.pick(item, 'familyName', 'firstNumber', 'tel');
            var values = _.merge({}, people,  {book: bookId});
            return AModel.validateFor(values);
          }).then(function(item) {
            results.Push(item);
          });
        });
        return q.then(function() {
          return results;
        });
      }).then(function(items) {
        return process4
      }).then(function() {
        t.commit(result);
      }).catch(function(e) {
        t.rollback(e);
        throw e;
      }));
    });
  });
}

function insertBook(t, book){
  return Promise.resolve().then(function () {
    return Book.columnizeFor(book);
  }).then(function (value) {
    return knex('book').transacting(t).insert(value, "id");
  });
}
1
Tamas Hegedus

insertBookがあなたにできる約束を返すと仮定します

var people = _.pick(item, 'familyName', 'firstNumber', 'tel');
if (item.type === 'book') {
  var book = _.pick(item, 'name', 'bookNumber', 'author');
  return insertBook(t, book)
    .then(bookId => _.merge({}, people,  {quotation: quotationId}, {book: bookId}))
    .then(AModel.validateFor)
} else {
  return Promise.resolve(_.merge({}, people,  {quotation: quotationId}))
    .then(AModel.validateFor)
}
0
Mauricio Poppe