然后在 Promise.all 完成之前执行答案

【问题标题】：Then executes before Promise.all finishes然后在 Promise.all 完成之前执行
【发布时间】：2021-07-23 03:52:00
【问题描述】：

我有这段代码，我希望所有 ChargeCreatePromises 在 orderUpdatePromises 执行之前完成执行：

return Q.all(chargeCreatePromises)
  .then(function() {
    return Q.all(orderUpdatePromises)
  })

代码结构在这里：

function createForThisMonth(order) {
  return Q.all([
    Order.findOne({ _id: order._id }),
    Charge.findOne({ 
      "orders._id": order._id, 
      chargeMonth: moment().format("YYYY-MM")
    })
  ]).spread(function(order, charge) {
    console.log("completed: ", order.completed)

    if (!charge && !order.completed) {
      return createChargeFromOrder(order)
      // returns a promise with Charge.findOrCreate(newCharge);
    }
  })
}

function orderUpdate(order) {
  return Q(Order.findOneAndUpdate({
    { _id: order._id },
    { $set: { completed: true } }
  }))
}

function process(orders) {
  var chargeCreatePromises = [];
  var orderUpdatePromises = [];

  orders.forEach(function(order) {
    if (order.category === 'monthly') {
      chargeCreatePromises.push(createForThisMonth(order))
    }

    orderUpdatePromises.push(orderUpdate(order))
  })

  return Q.allSettled(chargeCreatePromises)
    .then(function() {
      return Q.all(orderUpdatePromises)
    })
}

并且在不同的执行中，日志是：

completed: true
completed: false
completed: false
completed: true
completed: false

所以，我认为有时then(Q.all(orderUpdatePromises)) 在Q.all(chargeCreatePromises) 完成之前执行。

谁能帮助我了解这里发生了什么并实现我的目标？

问 -> https://github.com/kriskowal/q

节点 4

编辑：

我将Q.all(chargeCreatePromises) 更改为Q.allSettled(chargeCreatePromises)，但行为仍然存在。我用这个更改更新了上面的代码。

【问题讨论】：

我在尝试理解orderUpdatePromises(orderUpdate(order))，orderUpdatePromises 不是数组吗？
你打算orderUpdatePromises.push(orderUpdate(order)) 吗？请注意，您不会在 orderUpdate 中返回任何内容。你应该在那里添加return...
对不起，是的...我会编辑
我修复了代码
也 createForThisMonth() 不返回任何东西 => chargeCreatePromises 不是一个承诺数组，它是一个 undefined 数组

标签： node.js mongodb promise q

【解决方案1】：

我希望所有chargeCreatePromises 在orderUpdatePromises 执行之前完成执行

当您在循环中调用 createForThisMonth(…) 和 orderUpdate(…) 时开始执行 - 一次。 Q.all 代码只等待它们，但它们会同时运行并按任意顺序完成。

如果您希望所有 chargeCreatePromises 先完成，请将对 orderUpdate(…) 的调用放在 .then() 处理程序中：

function process(orders) {
  var chargeCreatePromises = orders.filter(function(order) {
    return order.category === 'monthly';
  }).map(createForThisMonth);

  return Q.allSettled(chargeCreatePromises).then(function() {
    var orderUpdatePromises = orders.map(orderUpdate);
    return Q.all(orderUpdatePromises);
  });
}

【讨论】：

确实是这样！分离循环调用...