Rails Way 处理多条路线上可用的操作

Question

我有以下路线：

resources :users do
  # List reviews made by user
  resources :reviews, :only => [ :index ]
end

resources :products do
  # List reviews by product, and provide :product_id for creation
  resources :reviews, :only => [ :index, :new, :create ]
end

# Other actions don't depend on other resources
resources :reviews, :except => [ :index, :new, :create ]

一切看起来都正确，除了ReviewsController#index：

def index
  if params[:user_id]
    @reviews = Review.find_all_by_user_id params[:user_id]
  else
    @reviews = Review.find_all_by_product_id params[:product_id]
  end
  respond_with @reviews
end

我想知道是否有上述问题的标准解决方案，或者是否有更好的方法。

Answer 1

你所拥有的很好，但如果你愿意，你也可以使用两种不同的操作。这种方法应该允许您稍后更轻松地更改视图，并且更安全一些。

match '/products/:product_id/reviews' => 'reviews#product_index'
match '/users/:user_id/reviews' => 'reviews#user_index'

它还将使您的控制器代码更加简洁，并且不易受到诸如 /products/10/reviews?user_id=100 之类的奇怪查询的影响，这会导致显示用户的评论而不是产品的评论。

def product_index
  @reviews = Review.find_all_by_product_id params[:product_id]
  respond_with @reviews
end

def user_index
  @reviews = Review.find_all_by_user_id params[:user_id]
  respond_with @reviews
end

另一种选择是也使用不同的控制器：

match '/products/:product_id/reviews' => 'product_reviews#index'
match '/users/:user_id/reviews' => 'user_reviews#index'

Answer 2

有些插件有办法为你加载资源，例如 declarative_authorization 或 cancan，我确信还有其他插件。

我见过的其他解决方案是在控制器中创建一个私有方法来加载对象，并且该方法中的逻辑本质上与您此处的逻辑相同；它只是将其移出索引操作本身。该方法也可以称为前置过滤器。

另一种执行逻辑的方法是从父对象开始（如果您也需要父对象，那就太好了：

before_filter :load_collection, :only => :index

private
def load_collection
  if params[:user_id]
    @user = @parent = User.find(params[:user_id])
  else
    @product = @parent = Product.find(params[:product_id])
  end
  @reviews = @parent.reviews
end

Answer 3

def index
  key = [:user_id, :product_id].find{|k| params[k]}
  @reviews = Review.where(key => params[key]).first
  respond_with @reviews
end