enumを利用してcheck_boxを作成し絞り込み検索を行いたい。

Question

やりたいこと

現在railsでタスク管理アプリを作成しています。
enumで定義したタスクの状態を検索フォームにcheck_boxとして
（□未着手　□着手中　□完了）このような形で作成し、
絞り込み検索を行いたいです。

現在viewには上記のような形でcheck_boxは表示されているのですが、
絞り込みがうまく機能してくれません。

今まではradio_buttonで実装しており、問題なく機能していたのですが
複数選択や選択の取り消しなど、使い易さを考えてcheck_boxに変更したいと考えています。

現在のソースコード

model

class Task < ApplicationRecord
  
  enum status: { waiting: 0, working: 1, completed: 2 }

  def self.status(status_by)
    case status_by
    when "waiting"
      waiting
    when "working"
      working
    when "completed"
      completed
    else
      all
    end
  end
end

controller

class TasksController < ApplicationController
  def index
    @tasks = Task.name_like(params[:name]).
               status(params[:status]).
               order(sort_param)
    @status = params[:status]
　end

メソッドチェインで「タイトル」、「状態」、「並び替え」で検索できるように考えています。

index

<%= form_with url: tasks_path, method: :get do |f| %>
  <p>
    <%= f.label :status, Task.human_attribute_name(:status) %>
    <%= f.collection_check_boxes :status, Task.statuses, :first, :first do |s| %>
    　<%= s.check_box value: @status %>
    　<%= s.label { Task.human_attribute_name("status.#{s.text}")} %>
    <% end %>
　</p>
　<p><%= f.submit "検索" %></p>
<% end %>   

試したこと

check_boxで一つを選択し検索ボタンを押してサーバのlogを見ると、statusのparameterが配列の中身に問題がありそうなのと、そもそもcollection_check_boxesを使用するのが正しいのかが分かりません。

Started GET "/tasks?name=&status%5B%5D=&status%5B%5D=waiting&sort=new&commit=%E6%A4%9C%E7%B4%A2" for ::1 at 2021-04-22 09:00:01 +0900
Processing by TasksController#index as HTML
  Parameters: {"name"=>"", "status"=>["", "waiting"], "sort"=>"new", "commit"=>"検索"}
  Rendering layout layouts/application.html.erb
  Rendering tasks/index.html.erb within layouts/application
  Task Load (1.9ms)  SELECT "tasks".* FROM "tasks" ORDER BY created_at DESC
  ↳ app/views/tasks/index.html.erb:39
  Rendered tasks/index.html.erb within layouts/application (Duration: 10.0ms | Allocations: 7853)
[Webpacker] Everything's up-to-date. Nothing to do
  Rendered layout layouts/application.html.erb (Duration: 16.0ms | Allocations: 11084)
Completed 200 OK in 18ms (Views: 14.6ms | ActiveRecord: 1.9ms | Allocations: 12039)

バージョン情報

ruby 3.0.0
Rails 6.1.3.1

その他

radio_buttonで問題なく機能していた際のコードも載せておきます。

<%= f.label(:status, Task.human_attribute_name(:status)) %>
<%= f.collection_radio_buttons(:status, Task.statuses, :first, :first) do |r| %>
  <%= r.radio_button(checked: r.value == params[:status]) %>
  <%= f.label(Task.human_attribute_name("status.#{r.text}")) %>
<% end %>

調べても情報が少ないため質問させていただきました。
実装のアドバイスをいただけると大変助かります。
よろしくお願いいたします。

Comments

[shinoharat]

> statusのparameterが配列の中身に問題がありそうなのと、そもそもcollection_check_boxesを使用するのが正しいのかが分かりません。

使うメソッドは「collection_check_boxes」で問題ありません。

配列の中身に問題があると推察されているようですが、未着手にチェックを入れて
　"status"=>["", "waiting"]
が送られてきたのであれば、動作としては想定通りな気がします。

なぜ問題があると思われたのでしょうか？
（空文字が要らないって意味ですかね？）

[Shokichi]

レビューいただきありがとうございます！
配列の中身には問題はなかったのですね、問題があると考えた理由としては仰る通り空文字が要らないと思ったためです。選択した未着手が[ "waiting" ]として一つだけ渡されるものなのかと考えました。
私のcollection_check_boxesメソッド内の書き方に問題があるのでしょうか。

[scivola]

やりたいことは，たとえば「着手中」と「完了」にチェックを入れた場合，〈status が「着手中」もしくは「完了」〉なレコードに絞り込む，ということでしょうか？
もしそうなら Task モデルの status メソッドを変える必要がありますね。

[Shokichi]

> やりたいことは，たとえば「着手中」と「完了」にチェックを入れた場合，〈status が「着手中」もしくは「完了」〉なレコードに絞り込む，ということでしょうか？

説明が足らずすみません。その通りです。
確かにstatusメソッドの条件分岐がradio_buttonの時のままでした...
ここを選択した状態を表示するように変えてあげないといけないということですね。

Answer 1

Task.statuses.keys & ["", "working", "completed"]

で

["working", "completed"]

が取れると思います（Array#& は集合の積演算）。
一方

Task.where(role: ["working", "completed"])

とすると，「role の値が "working" もしくは "completed" である」という ActiveRecord_Relation が得られますよね。

以上をまとめると，Task.status メソッドはやめて，

scope :status, -> s { where role: Task.statuses.keys & s }

みたいなスコープを導入すればよさそうです。

なお，べつに Task.statuses.keys との積集合を取らなくても，単純に params[:status] から空文字列を削除してもいいですし，何なら空文字列を残したままでも検索結果は変わらないはずです。
しかし，ユーザーが自由に制御できる params[:status] の値をそのまま where に使うのはちょっとどうかと思ったので。

Comments

[Shokichi]

とても分かり易いご回答をいただきありがとうございます！
無事にチェックボックスを完成させることができました。
「Array#&」についてはRubyの勉強をする中で理解していたつもりでいましたが、実際にこういった形で実装の中での使い方を知ることができ、理解を深めることができました。
scopeとクラスメソッドの使い分けについても、今後勉強していきたいと思います。