###前提・実現したいこと
下記のメソッドで、循環複雑度が高い!とRobocopさんに怒られてしまいます。
が、シンプルにする方法が全く思い浮かばず...
このメソッドをシンプルにする方法ってありますか?
ちなみに、このコードはrailsのcontrollrtに記述してあるものです。
お願いします!
###該当のソースコード
ruby
1 def show 2 @url = request.url 3 @event = Event.find(params[:id]) 4 @num1 = User.where('date1 = 1 and event_id = ' + @event.id.to_s) 5 @num2 = User.where('date2 = 1 and event_id = ' + @event.id.to_s) 6 @num3 = User.where('date3 = 1 and event_id = ' + @event.id.to_s) 7 @num4 = User.where('date4 = 1 and event_id = ' + @event.id.to_s) 8 @num5 = User.where('date5 = 1 and event_id = ' + @event.id.to_s) 9 end
###試したこと
Helperを使ってみたのですが、うまくいかず。
文字列のところはtimesメソッドを使えばいけるともうのですが、変数(@num1, @num2, etc...)をどうすれば良いのかわからず。っていう感じです。
回答1件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2016/09/15 07:48