Question

我尝试从JSON返回每个项目对象并使用angular.forEach()评估其值，但只返回最后一项。由于这个原因，我无法进行任何评估。有趣的是，如果我console.log()，它会逐个显示每个项目。

我如何获得每件商品并对其进行评估？

如果你知道更好的方法，请教我。

JS（angularjs）：

angular.module('bLiApp')
  .controller('AddDataCtrl', ['$scope', 'DataService', function ($scope, DataService) {

    DataService.getItems().success(function(data) {

      $scope.items = data.category.items;

      angular.forEach($scope.items, function(item) {

        // This is where I'm having problem with return data...
        if (item.amount < 600) {
          $scope.amountchecker = 'low';
        } else if (item.amount >= 600 && item.amount <= 650) {
          $scope.amountchecker = 'average';
        } else if (item.amount > 650) {
          $scope.amountchecker = 'too high';
        }

      });
    });
  }]);

HTML（angularjs）：

<div class="add-data" ng-controller="AddDataCtrl">
  <div class="data-box clearfix" ng-repeat="item in items">
    <article class="item">
      <p>{{amountchecker}}</p>
      <p><span class="month">{{ item.month.substring(0, 3) }}</span></p>
      <p class="cost">{{item.cost | currency:"&pound;"}}</p>
    </article>
  </div>
</div>

非常感谢

Answer 1

实际上你不需要forEach循环来实现它：

<div class="add-data" ng-controller="AddDataCtrl">

 <div class="data-box clearfix" ng-repeat="item in items">
    <article class="item">
      <p>
         <span ng-if="item.amount < 600">Low</span>
         <span ng-if="item.amount >= 600 && <= 650">Average</span>
         <span ng-if="item.amount < 650">Too high</span>
      </p>
      <p><span class="month">{{ item.month.substring(0, 3) }}</span></p>
      <p class="cost">{{item.cost | currency:"&pound;"}}</p>
    </article>
  </div>
</div>

那应该做的工作。请注意，这将仅显示金额，如果您想在模型中更改它（如果您必须发回刚评估的数据），则应更改控制器中的数据：

  angular.forEach($scope.items, function(item) {
    item.amountChecker; // declare a new property for each item in $scope.items
    // then we set the value for each item
    if (item.amount < 600) {
      item.amountChecker = 'low';
    } else if (item.amount >= 600 && item.amount <= 650) {
      item.amountChecker = 'average';
    } else if (item.amount > 650) {
      item.amountChecker = 'too high';
    }

  });

这应该在$ scope.items对象中添加一个新行，其中amountChecker将分配给每个项目。 然后，在ng-repeat中，您还可以显示该值：item.amountChecker 。

这一次，它会调用每个项目的属性amountChecker，而不是$ scope.amountchecker中存储的最后一个值。

请注意，Goodzilla实际上以较短的方式回答了评论：）

Answer 2

通过$scope.amountchecker，您似乎在循环的每次迭代中覆盖$scope.items。如果$scope.items的商品amount为500，其后amount为650，则$scope.amountchecker最终会设为'平均'，因为650是循环中遇到的最后一项。

解决方案取决于您是否真的希望amountchecker基于所有项目或单个项目。如果是后者，你会改变你的行，如：

$scope.amountchecker = 'low';

要

item.amountchecker = 'low';

Angular js foreach只返回数组中的最后一项

2 个答案: