複数のHelperメソッドを使用し条件に一致したオブジェクトを一覧表示する

Question

現在Ruby on Railsでタスク管理アプリを作成しています。
「sort title status」でタスクの表示を変更する実装したいのですが、sortだけしか機能しません。Helperでそれぞれメソッドを作成し、sort・title・status全てに該当するオブジェクトだけを一覧表示したいです。

ビュー(index.html.erb)

<%= form_with url: root_path, method: :get, local: true do |f| %>
  <%= f.select :sort, options_for_select({作成日時降順: "create_desc", 終了期限昇順: "deadline_asc", 終了期限降順: "deadline_desc"},params[:sort]), include_blank: "作成日時昇順" %>

  <%= f.text_field :title, value: params[:title] %>
  
  <%= f.radio_button :status_id, "1", checked: params[:status_id] == "1" %>
  <%= f.label :status_id, "未着手", {value: 1} %>
  <%= f.radio_button :status_id, "2", checked: params[:status_id] == "2" %>
  <%= f.label :status_id, "着手中", {value: 2} %>
  <%= f.radio_button :status_id, "3", checked: params[:status_id] == "3" %>
  <%= f.label :status_id, "完了", {value: 3} %>
  
  <%= f.submit "検索" %>
<% end %>

<% @tasks.each do |task| %>
  <p><%= link_to task.title, task_path(task) %> : <%= link_to task.status.name, task_path(task) %></p>
<% end %>

コントローラー(tasks_controller.rb)

class TasksController < ApplicationController
  include TasksHelper

  def index
    @tasks = sort
  end

  private

  def task_params
    params.require(:task).permit(:title, :content, :deadline, :status_id)
  end
end

ヘルパー(tasks_helper.rb)

module TasksHelper
  def sort
    @task = if params[:sort] == 'create_desc'
              Task.all.order(created_at: :DESC)
            elsif params[:sort] == 'deadline_asc'
              Task.all.order(deadline: :ASC)
            elsif params[:sort] == 'deadline_desc'
              Task.all.order(deadline: :DESC)
            else
              Task.all.order(created_at: :ASC)
            end
  end

  def title
    if params[:title].present? && params[:status_id].present?
      @task.where('title LIKE ?', "%#{params[:title]}%").where(status_id: params[:status_id])
    elsif params[:title].present?
      @task.where('title LIKE ?', "%#{params[:title]}%")
    end
  end

  def status
    if params[:status_id] == '1'
      @task.where(status_id: 1)
    elsif params[:status_id] == '2'
      @task.where(status_id: 2)
    elsif params[:status_id] == '3'
      @task.where(status_id: 3)
    end
  end
end

現状

①sortは必ず選択され、適応されます
②「sortとtitleで検索」「sortとstatusで検索」「sortとtitleとstatusで検索」というように実装したいです

自分なりに||や&&を使用してみたのですが、できませんでした。ご教授いただければ幸いです。

Answer 1

moduleでやろうとしている処理は本来modelのscopeを利用するべきですが、ここでは触れないで回答します。

あくまで一つの方法としてですが、

Controller

  def index
    @tasks = sort
  end

を

  def index
    @tasks = Task.all # 最初にTaskを全件取得
    title # @tasksにtitleで検索したTaskが代入される。
    status # @tasksにstatusで検索したTaskが代入される。
    sort # @tasksにsortしたTaskが代入される。
  end

に変更し、それぞれのメソッドで@tasksを更新する

module

  def sort
    @tasks = if params[:sort] == 'create_desc'
-             Task.all.order(created_at: :DESC)
+             @tasks.order(created_at: :DESC)
            elsif params[:sort] == 'deadline_asc'
-             Task.all.order(deadline: :ASC)
+             @tasks.order(deadline: :ASC)
            elsif params[:sort] == 'deadline_desc'
-             Task.all.order(deadline: :DESC)
+             @tasks.order(deadline: :DESC)
            else
-             Task.all.order(created_at: :ASC)
+             @tasks.order(created_at: :ASC)
            end
  end

  # titleという名前なのでこのメソッドはタイトルのみ検索するように！
  # @taskに代入まで行う
  def title 
-    if params[:title].present? && params[:status_id].present?
-      @task.where('title LIKE ?', "%#{params[:title]}%").where(status_id: params[:status_id])
-    elsif params[:title].present?
+    if params[:title].present?  
-      @task.where('title LIKE ?', "%#{params[:title]}%")
+      @tasks = @tasks.where('title LIKE ?', "%#{params[:title]}%")
    end
  end

  # @taskに代入まで行う
  def status
-   if params[:status_id] == '1'
+   @tasks = if params[:status_id] == '1'
      @tasks.where(status_id: 1)
    elsif params[:status_id] == '2'
      @tasks.where(status_id: 2)
    elsif params[:status_id] == '3'
      @tasks.where(status_id: 3)
+   else
+     @tasks
    end
  end
end

※moduleの@taskを@tasksに変更しています。
※差分をわかりやすくするためにインデントなどの修正はいれていません。
※また、確認していないためこれをそのまま使っても動くかは不明です。

追伸、動きを確認したらscopeを利用してみるのもいいと思います。

Comments

[kousuke24]

ご説明ありがとうございます！無事実装できました！

scopeについて質問があります。
①scopeは基本的にmodelに記述するもので、Helperには基本記述しない
②クエリをHelperに記述してもいいが、記述する際はscopeが推奨される

上記の認識で間違い無いでしょうか？

[satoshih]

無事動いたようでよかったです。

そうですね。MVC（モデル、ビュー、コントローラー）の責務の話になりますが、`複雑な`検索機能は基本的にモデルの責任で行うべきだと思っています。
（どの程度が複雑かは難しいところですが。。。）

あと、経験的にController（Controllerで読み込むmoduleも含む）で複雑な処理を実装するのはバグの温床になりやすい為、可能であればシンプルに実装したほうがいいと思います。

[kousuke24]

なるほど、、、肝に命じておきます。
繰り返しになってしまいますが、丁寧なご対応ありがとうございました！