Если оператор Rails Controller?

Question

Могу ли я использовать оператор if в своем контроллере или это плохая практика?

В обоих моих создают и разрушающих действий для TracksController, я хочу сделать что-то вроде этого:

if Product 
@product = Product.find(params[:product_id]) 
@track = @product.tracks.create(params[:track]) 

eslif Release 
@Release = Release.find(params[:release_id]) 
@track = @release.tracks.create(params[:track]) 
end 

Есть ли лучший способ сделать это?

Answer 1

Я бы пойти дальше и предложить более DRY подхода к before_filter:

class TracksController < ApplicationController 
    before_filter :get_track_parent, only: [ :create, :destroy ] 

    def create 
    @track = @parent.tracks.create(params[:track]) 
    ... 
    redirect_to @parent 
    end 

    private 

    def get_track_parent 
    if params[:product_id].present? 
     @parent = Product.find(params[:product_id]) 
    elsif params[:release_id].present? 
     @parent = Release.find(params[:release_id]) 
    end 
    end 
end 

Я родитель, потому что мы получили контекст для модели отношений, но я предполагаю, что есть лучший термин для описания общности между выпуском и обработкой товара.

Answer 2

Я хотел бы сделать это с помощью before_filter обратного вызова:

class TracksController < AC 
    before_filter :ensure_track, :only => [ :create, :destroy ] 

    private 

    def ensure_track 
    if Product 
     @product = Product.find(params[:product_id]) 
     @track = @product.tracks.create(params[:track]) 
    elsif Release 
     @release = Release.find(params[:release_id]) 
     @track = @release.tracks.create(params[:track]) 
    end 
    end 
end 

Так что с этой установкой он гарантировал, что у вас есть @track переменную экземпляра в вашем создавать и уничтожать методы, вызвать ensure_track получает вызывается перед этими двумя методами ,

Я не уверен, что если логика, которую вы применяете, имеет смысл ... Почему вы хотите проверить, существует ли константа с именем Product, а если нет константы с именем Release? Может быть, вопрос должен быть, если присутствует params[:product_id] или params[:release_id] !?

Но это другой вопрос :)

UPDATE: См Rails Action Controller Guide для фильтров.