viewの条件式を減らす

viewにロジックを書きすぎるな！ってことです。

自戒の意味を込めて書きます。

- if area? && action_name == "show"
  - if citys?
     = render "foo"
  - else
     = render "bar"
- else
  = render "bar"

こういうのがあったとします。

viewにこんだけロジックが積まれるとしんどいので、まとめます。

def foo?
  if area? && action_name == "show"
    if citys?
      true
    end
  end
end

- if doo?
  = render "foo"
- else
  = render "bar"

こんな感じになるかと思います。

viewにロジックが散らばると、同一条件で別の場所で表示するときにまた書かないといけなくなります。

ロジックをviewでガリガリ書くのはダメですね。