【Rails6】decimalのselect_boxの取り扱いについて

Question

f.selectのdecimalの型の取り扱いに困っています。
フォームから→更新→primary_achieveは1を選択→バリデーションエラー→render :edit
このときに入力された値の1を引き継ぐのことができません。

f.objectの中身を見ると、primary_achieveには0.1e1が入っています。
f.selectのvalueに0.1e1がないため、選択されていないと思われます。

どのようにすれば、値を引き継ぐことができますか？

class CreateSales < ActiveRecord::Migration[6.1]
  def change
    create_table :sales do |t|
      t.decimal :primary_achieve
    end
  end
end

= form_for(@sales, url: request_url) do |f|
  = f.select(:primary_achieve, options_for_select(1..3), { include_blank: "選択してください。" })

def update
  @sale.assign_attributes(sale_params)
  if @sale
  else
    render :edit
  end
end

Comments

[winterboum]

options_for_select(1..3) これでは 0.5は扱えないのでは？？

[ssk]

言葉足らずで申し訳ございません。
他の項目でも同じように、selectを利用しておりそこで0.5を使用しております。
今回の例は1でしたので、0.5は必要ないかと思い省いていました。

[winterboum]

下手な編集すると返ってわからなくなります。
問題が起きたときのcodeに差し替えてください

[ssk]

編集いたしました。
0.5というのが紛らわしかったので、削除し整数にしました。その場合のdecimalの型の取り扱い方法をご教授いただけますと幸いです。

Answer 1

整数値だけで良いのでしたらかんたんです
options_for_select(1..3, selected: f.object.primary_achieve.to_i)

Comments

[ssk]

ありがとうございます。
そして、二度手間になってしまい申し訳ございません。

0.5などの小数点も含めたい場合は難しいでしょうか？

中身は以下のようになっております。

= f.select(:primary_achieve, options_for_select( (0..1).step(0.5).map{|n| n%1 == 0 ? n.to_i : n} ), { include_blank: "選択してください。" })

[winterboum]

decimal で select したことが無いので推定とかなのですが、
不思議なことがあります。decimaal の 0.1e1 == 1 なのになぜoptions_for_select が等しいと見てくれないのか。
試しに、2つ目の引数として、 f.object.to_f を入れるとどうなりますか？
1.0 を 1 にする変換をやめるのも試してください。

[ssk]

以下の形で試したところ、0.5のみ入力内容をバリデーションエラー後に引き継ぐことができていました。

options_for_select( (0..1).step(0.5).map{|n| n%1 == 0 ? n.to_i : n}, f.object.primary_achieve.to_f)

[ssk]

以下のように事前に整形するようにしてみました。
動作はしていますが、なんか微妙な書き方のような気がしています...
もっと短く効率的に書くことは難しいでしょうか？

selected = if f.object.primary_achieve.present?
f.object.primary_achieve%1 == 0 ? f.object.primary_achieve.to_i : f.object.primary_achieve.to_f
else
nil
end
options_for_select( (0..1).step(0.5).map{|n| n%1 == 0 ? n.to_i : n}, selected)

[winterboum]

Saleに primary_achieve を このように整形するmethodを作り、何を求めているのか(HowでなくWhatで）わかる名前をつけ、
options_for_select( (0..1).step(0.5).map{|n| n%1 == 0 ? n.to_i : n}, f.object.メソッド)
ですね。

[ssk]

ありがとうございます。
メソッドに切り出して、スッキリ書くことができました。