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

器用貧乏です。はげたかです。日記です。

どうも、はげたかです。今まで器用貧乏に生きてきました。ジェネラリスト・フルスタックエンジニアを目指しています。

今日の一言:カテゴリ分けをしっかりしたらPVあがるかな?

【Rails】View からモデル層を呼んじゃっていいの?ダメー!!!

ちょっとブログ作成の波が落ちたところで、Railsに関するベストプラクティス的なやつを読んでいる。

そしたら、昨日記事にした、疑問が解決した。

  • 疑問
ぎもん
Article.all のようにコントローラから取得しなくても良い感じのものはどれがベストプラクティス?
コントローラーに以下を記述して、view に渡すべきなのか

> @article = Article.all


直接 view に以下を記述するべきなのか....
> <% Article.all.each do |article| %>
>     <%= article.... %>
> <% end %>

どっちがベストプラクティス?????
  • 解決策
ビューからモデル層を直接呼び出さないでください。
ビューの中で複雑な体裁出力を作らないでください。ビューヘルパー、またはモデルのメソッドに体裁を出してください。
部分テンプレートやレイアウトを使用してコードの重複を軽減させてください。

miyamae/rails-style-guide · GitHub

=> 要するに..

ビューからモデル層を直接呼びださないでください。ということは、.html.erbから

Article.all

を呼び出すなってことだよね。俺は多様していた。気をつけよう。

この他にこの記事を読んで、すごくごちゃごちゃさせちゃってるなぁってのが印象だ。まず一回目だから適当にやってみるのも良いよね。

次、作りなおすときはこういうのに気をつける。

このように、いろいろ気付けると成長するよね!実際は自分でコードを書いて、色々指導してもらうのが一番だと思うけど...

おすすめ記事等ありましたら教えて下さい!