take95
@take95

Are you sure you want to delete the question?

If your question is resolved, you may close it.

Leaving a resolved question undeleted may help others!

We hope you find it useful!

rails コントローラーの記述が長くなりすぎて困っています

railsでアプリケーションを作成しています。
コントローラーの記述が長くなりすぎているのではないかと感じます。
以下、該当箇所です。

controller
def index
  @laundries = Laundry.order('opening_date DESC').with_attached_image.page(params[:page])
end

以下のように表示したいと考えています。
・ 店舗をオープン日順に並べる
・ ページネーションの実施

orderメソッドを使用+kaminariを使用して目的としている表示はできたのですが、Active_strageのN+1問題が発生していることに気がつきました。
改善しようと思いwith_attached_imageを追加し、ログを確認したところ、画像はまとめて取得はできているのですが、indexアクションの記述が長すぎている気がします。

記述をコンパクトに行える方法がありましたら教えていただきたいです。
よろしくお願い致します。

1

2Answer

indexアクションの記述が長すぎている気がします。

全然長くないと思います。
コンパクトに行える方法の回答ではなくてすみません。

0Like

Comments

  1. @take95

    Questioner

    これはまだ長くないのですね!今まで書いてきたものが短すぎて長く感じていました。
    回答ありがとうございました!

よく使う、あるいは単体で書くと長すぎる組み合わせであれば、モデルにscopeとして立てる、というのも1つの選択肢です。

0Like

Comments

  1. @take95

    Questioner

    なるほど!そのような選択肢もあるのですね。
    勉強します!
    ありがとうございました。

Your answer might help someone💌