2012-02-15 26 views
-1

それを達成するため、私は私のコントローラはskinnier作るしようとしています。これを達成するために、私の努力のほとんどは、この種のコードがモデルであるべきRailsのスキニーコントローラ、脂肪モデル - 必要性より多くのヘルプが

など、建物のnamed_scopesに行き、そうであれば、どのように?

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 

歓迎

関連する問題