Question

我正在尝试用Promise编写这段代码。但我不知道如何在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");
  });
}

Answer 1

您不需要同步获取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");
  });
}

Answer 2

假设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)
}

承诺内部承诺

2 个答案: