레일 컨트롤러에 문이 있다면?

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

나는 더 가서 before_filter에 더 DRY 방식 좋을 것 :

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 콜백을 통해 그것을 할 것입니다 : 당신이 당신의 생성과 파괴 방법에 @track 인스턴스 변수를 가지고 있음을 보장 것이 설정에 따라서

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 

을 ensure_track의 원인은 두 가지 방법 전에 호출됩니다 .

논리를 적용하는 것이 의미가있는 경우 ... Product이라는 상수가 있는지 테스트하고 왜 Release이라는 상수가 없는지 왜 테스트하고 싶습니까? 질문은 params[:product_id] 또는 params[:release_id]이 있으면 어떨까요? 필터에 대한 Rails Action Controller Guide 참조 :

는하지만 다른 질문 :

UPDATE입니다.