読者です 読者をやめる 読者になる 読者になる

viewの条件式を減らす

rails

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でガリガリ書くのはダメですね。