check_boxでmultipleをtrueにするとチェックした時に、0と1の両方が送られる

Question

タイトルの通りなのですが、あまり直感的でない仕様なので質問させていただきたいです。

下記のようなformがあり

= form_for @week_selection do |f|
  - days = ['Mon', 'Tue', 'Wen', 'Thu', 'Fri', 'Sat', 'Sun']
  - days.each do |day|
    = f.check_box :days, { multiple: true }, day
    label day
  = f.submit

monとtueにチェックを入れて、submitすると

#送られるパラメーター
{ week_selection: { days: ["0", "Mon", "0", "Tue", "0", "0", "0", "0", "0"] }

このように送られます。
しかし、monとtueのそれぞれに0が付随しており、パラメーターの数が9個になっています。

"0", "Mon", "0", "Tue", "0", "0", "0", "0", "0"

自分的には、0など必要なく

"Mon", "Tue", "0", "0", "0", "0", "0"

これで送ればいいのにと思うのですが、0も送られてしまうこの仕様はどのような使用を想定して作られた仕様なのでしょうか。。。。

何卒よろしくお願いします。

Answer 1

【案１】
perpouh 様の回答にあるように collection_check_boxes を使うのが一番簡単な解決方法だと思います。

check_box に multiple を付けて使うと "0" と "1" が両方送られちゃうので、そうならないよう作られた、複数チェックボックス専用のメソッドが collection_check_boxes です。

実際は曜日ではなくてユーザーの一覧でチェックボックスを作っているとのことなので、おそらく以下のようなコードになると思います。

= f.collection_check_boxes :users, @users, :id, :name

【案２】
細かくデザイン調整したいときに、あえて check_box と multiple で画面を作ることもなくはないです。

その場合は f.check_box の第四引数に nil を指定すれば "0" を送らなくなります。

- @users.each do |user|
  = f.check_box :users, { multiple: true }, user.id, nil
  label = user.name

Answer 2

each で出さずに

f.collection_check_boxes :days, days.map{|v| [v, v]}, :first, :last

じゃだめですかね？
ときに、 form_for って今推奨のメソッドじゃなかったと思うので、なにか参考書とか読まれているならそれ古いかもしれません。

Comments

[tomoharu]

ご回答ありがとうございます。本当は@usersが飛んできていて、それをeachで回しています。そしてそのuser一人一人にcheckboxをつけたいという感じです。そうなるとeachを使うのは避けられないのかと思いまして。。。

[perpouh]

@userがweekを持っているということでしょうか？ちょっとユースケースがわかりにくいので（守秘義務とかに触れない範囲で）正確に書いていただきたいです。eachを避けられないケースは思いつかないです。

[tomoharu]

返信遅れて申し訳ありません。

下記で満たしているかわかりませんが、ご回答いたします。

コードとしては、

```
<%= form_with model: current_company do |f| %>
<% @users.each do |u| %>
<%= u.name %>
<% end %>
<%= f.check_box :status %>
<% end %>
```

このような形でuser一人一人にcheckboxが付いている感じです。