Railsにおいてdatetime_selectでのパラメータの受け取り方について

Question

前提・実現したいこと

勤怠表のウェブアプリを作成しています。
出社と退社の打刻時間編集画面にてdatetime_selectにより任意の時間を選択して更新するという機能を実装したいのですが、パラメータの受け取り方でわからない部分があったのでご存知の方いらっしゃいましたらよろしければ教えてください。

発生している問題

更新ボタンを押した後にリダイレクトをし任意のページに戻ることはできるのですが、
表示時間は編集前の元の時間から更新されていない状態です。

該当のソースコード

rails 5.1.6

[DB]
出社時間: at_in
退社時間: at_out

<div>
    <%= form_for(@punch) do |f| %>
        <%= render 'shared/error_messages', object: f.object %>
    
        <%= f.label :at_in, "出社時間" %>
        <%= f.datetime_select :at_in, minute_step: 5 %>
        
        <%= f.label :at_out, "退社時間" %>
        <%= f.datetime_select :at_out, minute_step: 5, include_blank: true %>
     <% end %>
        <%= f.submit "更新", class: "btn btn-primary" %>
    <% end %>
</div>

更新ボタンを押してdatetime_selectで渡されたパラメータ

"punch" => {
  "at_in(1i)" => "2018", 
  "at_in(2i)" => "10", 
  "at_in(3i)" => "15", 
  "at_in(4i)" => "10", 
  "at_in(5i)" => "00",
  "at_out(1i)" => "2018"
  "at_out(2i)" => "10"
  "at_out(3i)" => "15"
  "at_out(4i)" => "19"
  "at_out(5i)" => "30"
}

対応controller

　def edit
    @punch = Punch.find_by(params[:id])
　end

  def update
    @punch = Punch.find_by(params[:id])
    @punch.at_in = (params[:punch][:at_in])
    @punch.at_out = (params[:punch][:at_out])
    if @punch.save
      flash[:success] = "打刻時間を更新しました"
      redirect_to '/punches'
    else
      render 'edit'
    end
  end

datetime_selectフォームののパラメータをat_inとat_outで区別して受け取る方法がどうやらうまくいっていないのかと思うのですが方法が見つからない状態です。
解決方法有りましたらよろしくお願いします。

Answer 1

これは結構僕も悩みましたが、date_selectやdatetime_select等は、本来別々の値の年、月などを
よろしくやってくれるので、punch_paramsという様なストロングパラメーターを定義し、そこに
at_in, at_outなどを入れて、@punch.update_attributes(puch_params)という様にやった場合
あまり考えなくてもよいのですが、別個の値としてやろうとすると、params[:punch][:at_in]では
取り出せないんです
そこで、このように書いてあげる必要があります

@punch.at_in = DateTime.new(
  params[:punch]["at_in(1i)"].to_i,
  params[:punch]["at_in(2i)"].to_i,
  params[:punch]["at_in(3i)"].to_i,
  params[:punch]["at_in(4i)"].to_i,
  params[:punch]["at_in(5i)"].to_i
)

Comments

[退会済みユーザー]

回答ありがとうございます！
またお返事遅くなり大変申し訳ございません。
なるほど、一つ一つ記述する必要があるのですね。
ストロングパラメーターでの取得は試してみたのですが、自分の意図した値を取得できなかったので困っていました。
是非活用させていただきます！