Angular.js正在观看函数调用的结果

时间:2013-03-19 02:46:04

标签: angularjs

以下代码段是否有任何表面上的问题:

  <ul id="entry-name" class="breadcrumb">
      <li ng-repeat="dir in pathElements()" class="active">
          <span ng-show="!$last">
              &nbsp;<a href="#!?path={{dir.url}}">{{ dir.name }}</a>&nbsp;<span ng-show="!$first" class="dividier">/</span> 
          </span>
      </li>
      <li class="active">{{ pathElements()[pathElements().length - 1].name }}</li>
  </ul>

用这个js:

  $scope.pathElements = function() {
      var retval = [];
      var arr = $scope.currentPath().split('/');
      if (arr[0] == '') {
          arr[0] = '/';
      }

      var url = "/";
      retval.push({name: arr[0], url: url});
      for (var i = 1; i < arr.length; ++i) {
          if (arr[i] != '') {
              url += arr[i] + '/';
              retval.push({name: arr[i], url: url});
          }
      }
      return retval;
  };

这似乎导致了“错误:10 $ digest()迭代达成。中止!”错误,但我不确定为什么。是因为pathElements()每次都返回一个新数组吗?有办法解决这个问题吗?

2 个答案:

答案 0 :(得分:4)

是的,这是因为你每次都返回一个新的数组,$ digest循环无限循环(但是Angular会停止它)。你应该在函数之外声明它。

$scope.pathArray = [];
$scope.pathElements = function() {
  // retval becomes $scope.pathArray
  if (weNeedToRecalcAllPathVariables) {
    $scope.pathArray.length = 0;
    // ...
  }

  // ...
}

我们使用$scope.pathArray.length = 0而不是创建一个新的,以避免它连续发射。

您还应该考虑@Flek建议的内容。在您需要重新计算时,只调用此函数一次。所有你绑定的都应该直接在$scope.pathArray上。

如果你确实需要一个函数来测试它的清洁状态,那么至少我建议你创建两个独立的函数,只是为了让每个函数都有自己的属性。

答案 1 :(得分:3)

有关如何在Angular中实现面包屑的一些很好的参考,请查看angular-app项目(breadcrumbs servicehow to use it)。

这是一个演示plunker