現在時刻を取得してtime型カラムへ保存させたい

Question

前提・実現したいこと

Ruby on Rails初心者です。
学習の一貫として勤怠管理アプリを作っています。
現在時刻を取得し、time型カラムへ時刻を保存するためcreateアクションの設定とストロングパラメータの設定を行いました。

def create
    @time_card = Time.current
    @time_card = Timecard.create(timecard_params)
    if @time_card.save
      redirect_to root_path
    else
      render :index
    end  
  end
  
  private
  def timecard_params
    params.permit(:start_time, :end_time).merge(user_id: current_user.id)
  end

= form_with(url:"/worktimes", local: true, class: "btn_frame") do |f|
      =f.submit "出勤", id: "start-btn"
      =f.submit "退勤", id: "end-btn"

追記（データベース）

class CreateTimecards < ActiveRecord::Migration[6.0]
  def change
    create_table :timecards do |t|
      t.time :start_time     
      t.time :end_time       
      t.references  :user,  index: true, foreign_key: true
    end
  end
end

発生している問題・エラーメッセージ

ビューには出勤ボタンと退勤ボタンがそれぞれあり、出勤ボタンを押すとstart_timeへ、退勤ボタンを押すとend_timeへ保存されるようにしたいのですが、上手くいきません。

試したこと

createアクションにbinding.pryをかけ、paramsを調べたところ、当然、中身がuser_idのみになっており、start_timeとend_timeは存在しませんでした。ここからの対処方法が分からず、何方かご教示頂けますと幸いです。

Answer 1

params[:commit]　に　"出勤","退勤"が入ってきますから

case params[:commit]
when "出勤" ; @timecard[:start_time] = Time.now
when "退勤" ; @timecard[:end_time] = Time.now
end
としてください。
idを指定しているので　　"出勤","退勤"ではなく"start-btn" "end-btn" になってるかもしれないので、logみてparams確認してください

Comments

[naokit-dev]

[Rails - What is the purpose of params[:commit] - Stack Overflow](https://stackoverflow.com/questions/23382973/rails-what-is-the-purpose-of-paramscommit)

勉強になりました

[sheep1991]

ありがとうございます。
参考にさせて頂きます。

Answer 2

= form_with(url:"/worktimes", local: true, class: "btn_frame") do |f|
      =f.submit "出勤", name: "start_btn"
      =f.submit "退勤", name: "end_btn"

def create
  @time_card = Timecard.new(timecard_params)

  @timecard.start_time = Time.now if params[:start_btn]
  @timecard.end_time = Time.now if params[:end_btn]

  if @time_card.save
    redirect_to root_path
  else
    render :index
  end
end

nameでいいんじゃないでしょうか

Comments

[sheep1991]

yuichi555様ありがとうございます。
ですが、上記コードでも上手くいきませんでした。
大変恐縮ですが、params[:start_btn]とparams[:end_btn]の部分はstart-btnとend-btnに当たるのではないかと考え、修正してみたのですが、それでもやはり上手くいかずといったところです。

[yuichi555]

↑修正しました

Answer 3

= form_with(url:"/worktimes", local: true, class: "btn_frame") do |f|
      =f.submit "出勤", id: "start-btn"
      =f.submit "退勤", id: "end-btn"

def create
    @time_card = Timecard.new(timecard_params)
    case params[:commit]
      when "出勤" ; @time_card.start_time = Time.now
      when "退勤" ; @time_card.end_time = Time.now
    end
    if @time_card.save
      redirect_to root_path
    else
      render :index
    end  
  end

  private
  def timecard_params
    params.permit(:start_time, :end_time).merge(user_id: current_user.id)
  end

ですかね

だめならtypo等を修正して一度質問の内容を整理していただけますか

Comments

[sheep1991]

naokit-dev様
ご回答頂きありがとうございます。

フォームの内容が空だからparamsはからののでは？
すみません。そもそもの解釈が間違っているかもしれませんが、上記の助言を頂き、下記のように記述するべきものかと考え、実装してみたのですが上手くいきませんでした。
=f.submit :start_time value:"出勤", id: "start-btn"
=f.submit :end_time value:"退勤", id: "end-btn"

これが反映されているのはどの部分ですか？
start_timeカラムとend_timeカラムに時刻を保存できた後に、別のビューファイルではあるのですが出勤簿を用意しているので、そちらに反映させるつもりです。

[sheep1991]

winterboum様
ご回答頂きありがとうございます。
頂いた助言を参考に、下記のように修正してみたのですが、start_timeカラムにもend_timeカラムにも時刻が保存されませんでした。
他修正すべき点がございましたら、ご指摘頂けますと幸いです。

def create
case params[:commit]
　when "start-btn" ; @timecard[:start_time] = Time.now
　when "end-btn" ; @timecard[:end_time] = Time.now
end
@time_card = Timecard.create(timecard_params)
if @time_card.save
redirect_to root_path
else
render :index
end
end

[naokit-dev]

@time_card = Timecard.create(timecard_params)

で上書きされているからでは？

とおもったら、その上は@timecardですよ

[sheep1991]

あ、本当ですね。
ただ、@timecard[:start_time]を@time_card[:start_time]のように修正し、@time_card = Timecard.create(timecard_params)をコメントアウトした場合としてない場合とで試してみたのですが、それでも上手くいきませんでした。

[naokit-dev]

> "出勤","退勤"ではなく"start-btn" "end-btn" になってるかもしれないので

winterboumさんがかいとうされていることは試されましたか？

[sheep1991]

はい。paramsの中身を確認したところ、commitは"出勤" となっております。
ですが、そのまま"出勤"としたまま、ボタンをクリックすると下記のエラー文が生じてしまいます。

エラー文
https://i.gyazo.com/eadf1dfb8317715b756d3d961964401c.png

params中身確認
https://i.gyazo.com/d4316f11fe876fe6af6b02740de3d96f.jpg

"出勤"、"退勤"の部分をそれぞれstart-btnとend-btnとしてもダメでした。

[sheep1991]

ありがとうございます！
上記のコードで上手くいきました！