解决 $q.all 中的 Angularjs 承诺的正确方法是什么

Question

我在解决使用 $q.all 的承诺时遇到了问题，有人可以帮帮我吗？

当我有一个单一的承诺时，以下工作正常：

 var dashlettePromise = DashboardsDataService.getTabDetails(dashboardData);
    dashlettePromise.then(function(data) {
      var template = '<div class= "allChartsDiv">';

      for (var dashVar = 0; dashVar < data.tabDetails.length; dashVar++) {
        var dashletteId = data.tabDetails[dashVar].dashletteId; // Added
         ....
      }
    }, function(error) {
      alert(error);
    }, function(progress) {
      // report progress
    });

但我需要进行多次 ajax 调用，所以我使用了 $q.all，如下所示：

var promises = [];
angular.forEach(dashboardslayoutArray, function(dashboardslayout) {
  dashboardData.dashletteBeansList = [];
  dashboardData.dashletteBeansList[0] = dashboardslayout;
  var dashlettePromise = DashboardsDataService.getTabDetails(dashboardData);
  promises.push(dashlettePromise);

});

现在我需要一一解决 $q.all 中的所有承诺，与我为单个承诺解决的承诺完全相同，如上所示。所以我使用了下面显示的代码，但它没有按预期工作。我开始怀疑我在 $q.all(promises) 中使用的逻辑来解决这些承诺。这是正确的方法吗？或者任何人都可以提出更好的方法来解决 $q.all 中的承诺？

$q.all(promises)
  .then(function(allData) {
    // all promises were resolved here                          
    angular.forEach(promises, function(eachPromise) {

      eachPromise.then(function(data) {

        for (var dashVar = 0; dashVar < data.tabDetails.length; dashVar++) {
          var dashletteId = data.tabDetails[dashVar].dashletteId; // Added
          var axisType = data.tabDetails[dashVar].axisType;
            .....
        }

      }, function(error) {
        alert(error);
      })

    })

  });

Answer 1

问题在于$q.all 不会在已解决状态下返回 Promise 本身，而是直接返回它们的值。因此，尽管您希望在 $q.all 调用中获得承诺，但您正在获得价值。为了规避这种情况，您可以在关闭时访问 Promise。

尝试调整代码以处理 .all 调用中的值而不是承诺：

$q.all(promises)
  .then(function(allData) {
    // all promises were resolved here                          
    allData.forEach(function(data) {
    // no `then` needed here
        for (var dashVar = 0; dashVar < data.tabDetails.length; dashVar++) {
          var dashletteId = data.tabDetails[dashVar].dashletteId; // Added
          var axisType = data.tabDetails[dashVar].axisType;
            .....
        }
    });
});

所以总的来说你的方法是有效的（干得好）-但可以更好:)

另请注意，可能应该避免进度事件，并且不是未来的证据。