通过$ scope访问函数中的ng-resource会导致无限循环

时间:2015-10-15 08:00:37

标签: javascript ruby-on-rails angularjs ngresource

我有一个Rails应用程序,并使用ng-resource通过JSON API访问两个模型。

在我的模板中,我显示了第一个模型“订单”的列表:

<li ng-repeat="order in orders">
  Order {{order.id}}: 
  Product: {{showProduct(order.product_id)}}
</li>

每个“订单”都包含product_id。现在我想访问第二个模型(“产品”)并在ng-repeat内显示正确的“产品”,其中包含相应“订单”的ID product_id

为了达到这个目的,我想到了使用函数showProduct()并使用order.product_id作为参数。但是,当我这样做时,它会导致无限循环,不断向我的数据库发出GET请求。

这是 app.js

的重要部分
var app = angular.module('shop', ['ngResource']);

app.factory('models', ['$resource', function($resource){
  var orders_model = $resource("/orders/:id.json", {id: "@id"}, {update: {method: "PUT"}});
  var products_model = $resource("/products/:id.json", {id: "@id"}, {update: {method: "PUT"}});

  var o = { 
    orders: orders_model,
    products: products_model
  };
  return o;
}]);

app.controller('OrdersCtrl', ['$scope', 'models', function($scope, models){
  $scope.orders = models.orders.query();

  $scope.showProduct = function(product_id){
    return models.products.get({id: product_id});
  };
}])

这些是我的控制台中的错误(这是有意义的,因为它是一个无限循环):

Error: [$rootScope:infdig] 10 $digest() iterations reached. Aborting!
Watchers fired in the last 5 iterations: []

Uncaught Error: [$rootScope:infdig] 10 $digest() iterations reached. Aborting!
Watchers fired in the last 5 iterations: []

Rails控制台中的GET请求似乎很好。也许我想要比我需要的想法复杂得多。

2 个答案:

答案 0 :(得分:1)

是的,因为你在视图中有showProduct,每个摘要都会调用该函数,每次都会返回一个不同的promise,因此它会中断:

  • 每个摘要多次创建服务器请求
  • 每个请求与其他请求不同
  • 进行10次迭代,直到两个值相等或者抛出异常

我建议你这样做:

models.orders.query().$promise
  .then(function(orders){
    $scope.orders = orders;
    //using lodash here, I recommend you use it too
    _.each(orders, function(order){
      models.products.get({id: order.product_id }).$promise
        .then(function(product){
          order.product = product;
        })
    });
  });

在视图中:

<li ng-repeat="order in orders">
  Order {{order.id}}: 
  Product: {{order.product}}
</li>

但是使用这段代码会触发每个订单的一个查询非常糟糕:它是一个N + 1反模式。

2个解决方案:

  • 让订单直接包含产品数据
  • 只有一个请求包含所有产品ID

答案 1 :(得分:0)

看着你的代码,我首先觉得它应该有效。无论如何,如果没有,那么我会这样尝试:

在您的控制器中:

  $scope.showProduct = function(product_id){
    var product = models.products.get({id: product_id}, function(){
      return product;
    });
  };

一旦数据从服务器到达,这将返回值。