ruby-on-rails – Rails处理多条路线上可用的动作的方法

发布时间：2020-12-17 03:27:16 所属栏目：百科来源：网络整理

导读：我有以下路线： resources :users do # List reviews made by user resources :reviews,:only = [ :index ]endresources :products do # List reviews by product,and provide :product_id for creation resources :reviews,:only = [ :index,:new,:create ]

我有以下路线：

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,: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

我想知道是否有上述问题的标准解决方案,或者是否有更好的方法来做到这一点.

解决方法

你有什么是好的,但如果你想要你也可以使用两种不同的行动.这种方法应该可以让您以后更轻松地更改视图,并且更安全一些.

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'

（编辑：李大同）

【声明】本站内容均来自网络，其相关言论仅代表作者个人观点，不代表本站立场。若无意侵犯到您的权利，请及时与联系站长删除相关内容!