Firebase (Angular) Promises with Loops in Loops

Question

只有在所有的 promise 都解决之后，你必须在循环中等待循环中的异步调用，你如何执行一个函数？

代码被简化到最低限度

$scope.book = function(input) {

  //Get the cart items from DB
  ref.child('/cart/' + userId).once('value').then(function(cartSnap) {

    //Loop through the cart items
    cartSnap.forEach(function(cartItemSnap) {

      var itemId = cartItemSnap.key();

      //Get the inventory items from DB that are in the cart
      ref.child('/inventory/' + itemId).once('value').then(function(inventoryItem) {

        //Loop through booking IDs of inventoryItem
        inventoryItem.child('rentals').forEach(function(rentalSnap) {

          var rentalId = rentalSnap.key();

          //Get bookings from rental/active
          ref.child('/rental/'+ rentalId).once('value').then(function(activeRentals) {

            checkIfBookingIsAllowed();

          });
        });
      });
    });

    //Once everything was checked
    bookRental();
  });
};

为了提高速度，所有请求都可以并行发出，但最终函数 bookRental() 只能在一切都解决后调用。

感谢您的帮助。

编辑： 又一次失败的尝试。 Promise.all('collector') 在所有承诺解决之前被触发。所以“完成”出现在控制台中的所有“检查”之前。

$scope.book = function(input) {

  //Get the cart items from DB
  ref.child('/cart/' + userId).once('value').then(function(cartSnap) {

    //Promise collector
    var collector = [];

    //Loop through the cart items
    cartSnap.forEach(function(cartItemSnap) {

      var itemId = cartItemSnap.key();

      //Get the inventory items from DB that are in the cart
      var promise1 = ref.child('/inventory/' + itemId).once('value').then(function(inventoryItem) {

        //Loop through booking IDs of inventoryItem
        inventoryItem.child('rentals').forEach(function(rentalSnap) {

          var rentalId = rentalSnap.key();

          //Get bookings from rental/active
          var promise2 = ref.child('/rental/'+ rentalId).once('value').then(function(activeRentals) {

            console.log('check');
            collector.push(promise2);

          });
        });

        collector.push(promise1);
      });
    });

    //Once everything was checked
    Promise.all(collector).then(function() {
      console.log('Done');
    });
  });
};

Answer 1

你收集承诺为时已晚。 现在收集它们，而不是以后。 .thens 中的代码稍后运行。

下面是立即运行的：

1. 两个嵌套的 for 循环都运行完成，减去 .thens 中的所有代码。
2. Promise.all(collector)。

此时collector 仍然是空的，所以Promise.all() 完成得非常快。

将collector.push(promise) 调用移到.thens 之外。

更实用的方法：

通过上述修复，您的代码应该可以工作，但更简洁的方法是使用return all promises。虽然forEach 不允许返回值，但map 允许，因此可以将其重写为（简化）：

Promise.all(cartSnap.map(snap => ref.child(itemUrl).once('value').then(item =>
  Promise.all(item.child('rentals').map(snap => ref.child(url).once('value'))))))
.then(allActiveRentals => console.log('Done'));