2012-02-15 2 views
-1

나는 내 컨트롤러를 더 튼튼하게 만들려고 노력하고 있습니다. 이를 달성하기 위해 대부분의 노력이 named_scopes 등을 만들었습니다.Rails 스키니 컨트롤러, 뚱뚱한 모델 - 더 많은 도움이 필요합니다.

이러한 종류의 코드가 모델에 있어야합니까? 그렇다면 어떻게해야합니까?

def show 
    ### params[:date] = {"month"=>"2", "year"=>"2012"} 
    @date = Time.parse(params[:date][:month] + '/' + params[:date][:year]) 
    ... 
end 
+1

컨트롤러에 문제가있는 것은 아닙니다. –

+2

길게 대답 : 마른 체형 컨트롤러에 너무 매달 리지 마십시오. 거기에 비즈니스 논리가 없으면 괜찮을 것입니다. 여기에서하는 일은 단지 뷰를위한 var를 준비하는 것입니다. 그것이 있어야하는 곳입니다. –

+0

컨트롤러가 작동 할 데이트 대상을 만들고있는 한 해당 코드는 컨트롤러에서 문제가 없습니다. 그러나 컨트롤러에서 날짜 문자열의 서식을 지정하기 시작하면 특정보기이며 도우미 메서드에 배치해야합니다. – MunkiPhD

답변

0

나는 이것이 당신이 할 수있는 가장 얇은 생각하지만

** 헬퍼에서 호출 할 수있는 도우미 메서드에이 시간 파서를 얻을 수 있다면 그것은 더 우아하고 재사용 가능한 것 컨트롤러뿐만 아니라, 도우미

def time_parser(month, year) 
    Time.parse(month + '/' + year) 
end 

및 컨트롤러에서이 같은

뭔가

def show 
    ### params[:date] = {"month"=>"2", "year"=>"2012"} 
    @date = time_parser(params[:date][:month],params[:date][:year]) 
    ... 
end 

건배

관련 문제