Railsで時間の記録

Question

RailsでDateTimeを使ってボタンで作業時間を記録するアプリを作ろうとしているのですが、

・作業時間テーブル（WorkingHourテーブル）はUserテーブルと紐づいている
・作業開始ボタンでTime.currentを記録
・作業終了ボタンでTime.currentを記録
・それがuserの名前と一緒にviewに表示される
・開始時間も終了時間も上書きされずに追加されていく
例）
ヤマダタロウ
8/10 10:00〜15:00
8/11 10:00〜14:00
8/13 11:00〜12:30
ヤマモトタロウ
8/10 10:00〜19:00
8/12 11:00〜14:00

恥ずかしながらWorkingHourモデルを作成し、以下のコードを書いた段階で思考が停止してしまいました....

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception

  def register_work_starting_time
    @starting_time = WorkingHour.create(user_id: @user.id, starting_time: Time.current)
  end

  def register_work_finished_time
    @finished_time = WorkingHour.create(user_id: @user.id, finished_time: Time.current)
end

どなたか力を貸していただけると助かります。
「この根本を理解してないよー」とか「ヒントはこれ」とか、直接的な回答でなくても大歓迎なので、よろしくお願いいたします。

以下補足--------

・WorkingHourに記録するのはuser_nameではなくuser_idで、user_nameはuser_idから引っ張ってくるべきです（名前が変わった時に記録時の名前が欲しい、とかいう要件でなければ）
→たしかにそうですね。変えました。

・作業開始、作業終了が複数回連続で押される可能性があるのかどうか、可能性がある場合にどうするかが問題になりそうです。
要はボタンの押し忘れの対処方法とかです。
→説明が不足しておりましたが、これは単に時間を記録するアプリではなく、他にもいろいろ要件があるんですが、そのためにマイページをつくり、開始、終了ともに押したら時間記録と一緒にマイページへの遷移を考えました。ひとまず押し忘れと連続押しは回避できるかなと思います。

・どういう風に表示するかのロジックも重要です。例えば日付で区切る、といった場合、作業が日をまたいだ時にどうするのかが悩ましいです
→日付をまたぐことはとりあえず想定せず！

以下補足2--------

漠然と実装したい機能が書いてあるだけで、どこがわかってないのか、どこの部分を教えてほしいかが明確でないと思います。

失礼しました。

まずわからないのは、viewのボタン部分は<%= button_to "ボタン", {コントローラー名など} %>でやろうとしていて、表示の際は名前をuser_idからfind_byで引っ張って来て表示とします。その場合、僕のcontrollerのコードで足りない部分がわからないというのとviewのやり方が適切かどうかがわからず質問させていただきました。

Comments

[nami634]

漠然と実装したい機能が書いてあるだけで、どこがわかってないのか、どこの部分を教えてほしいかが明確でないと思います。

[nami634]

Railsは複雑なところも多いので何がわからないかわからないみたいなことも多いかと思いますが、なぜこのコードにたどり着いたかやなるべくわからないところを絞って質問されないと回答者がどのように答えていいかわかりません。

[anpnt]

すみません、補足させていただきました。

Answer 1

WorkingHourについて

要件が完全にはわからないので難しいのですが、WorkingHourの実装には2通りあると思っていて

1. 時刻と開始か終了かを記録する
2. 開始と終了を1セットで考えて、開始時刻と終了時刻のセットで記録する

この通りに実装すると

• 1の場合

WorkingHour
:user_id
:state
:time

この場合、何時間働いたかを調べる時に少しだけ工夫がいるようになります。

• 2の場合

WorkingHour
:user_id
:start_time
:finish_time

こちらの場合は、終了を記録する時にどのスタートと関連しているかを調べるのに工夫が入ります。

以上のような実装になると思います。
このどちらを使うかは今後の実装であったり、既出の複数回連続で押される可能性などが関わってくるのでケースで判断された方がいいです。

コントローラ

user側の実装がわからないので、何とも言えないのですが、質問に書かれている通り、@user.idでログインしているuserのidが取れるものとします。

1の場合なら

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception

  def register_work_starting_time
    @starting_time = WorkingHour.create(user_id: @user.id, state: "start", time: Time.current)
  end

  def register_work_finished_time
    @finished_time = WorkingHour.create(user_id: @user.id, state: "finish" time: Time.current)
  end
end

またはstateをview側で送ることもできるので

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception

  def register_work_time
    @starting_time = WorkingHour.create(user_id: @user.id, state: params[:state], time: Time.current)
  end
end

2の場合なら

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception

  def register_work_starting_time
    @working_hour = WorkingHour.create(user_id: @user.id, start_time: Time.current)
  end

  def register_work_finished_time
    @working_hour = WorkingHour.where(関連づけたいスタートのworking_hourを探す処理)
    @working_hour.finish_time = Time.current
    @working_hour.save
  end
end

ここはあえて実際に探す処理については書きませんでした。本来であれば、こういったQ&Aサイトはこのような細かな処理について質問するのが良いと思います。
最初は（特にRailsなんかは）漠然と何をしていいかわからないことが多いので仕方ないと思いますが、実装したい機能を構成する細かい実装に分けて、その一つ一つについて問題を解決しながら実装しないといけません。まずは一覧を表示する前に、記録をつけるところ、さらにはスタートだけから始められるのがいいと思います。

Viewの考え方はまちがってないと思いますので、とりあえず自分で実装してみてください。

Comments

[anpnt]

丁寧に回答していただいてありがとうございます。

[anpnt]

ちなみに探してくる処理に関して、
@working_hour = WorkingHour.where(user_id: @current_user.id, finish_time: nil).last
というのはどう思いますか？

user_idがcurrent_userのidであることはview側で行うべきでしょうか？

[nami634]

その取ってき方で問題はないのですが、もしfinish_timeがなかった時の処理を書いてあげるといいと思います。

user_idを入れるのはコントローラでやった方が安全（ビュー側でやることで、悪意のある/間違って操作したユーザーが他の人のものを変えてしまうことや、ビュー側での実装ミスで違ったidを送ってしまうことなどが避けやすい）なので、コントローラで入れてしまう方がいいと思います。

[nami634]

その取ってき方で問題はないのですが、もしfinish_timeがなかった時の処理を書いてあげるといいと思います。

user_idを入れるのはコントローラでやった方が安全（ビュー側でやることで、悪意のある/間違って操作したユーザーが他の人のものを変えてしまうことや、ビュー側での実装ミスで違ったidを送ってしまうことなどが避けやすい）なので、コントローラで入れてしまう方がいいと思います。

[anpnt]

ありがとうございます！

>もしfinish_timeがなかった時の処理
これはどういった意図でしょうか？

whereにfinish_time:nilを入れるのではなく、finish_timeがないときに処理をするという条件を加える、という意味では....ないですよね？汗

[nami634]

すみません、書き間違えました。
finish_timeがないものが複数あった時の処理などを足してあげたほうがいいですね。

[anpnt]

なるほど！ありがとうございます！

Answer 2

「回答」というわけではないですが、気のついたことをメモしておきます。

• WorkingHourに記録するのはuser_nameではなくuser_idで、user_nameはuser_idから引っ張ってくるべきです（名前が変わった時に記録時の名前が欲しい、とかいう要件でなければ）
• 作業開始、作業終了が複数回連続で押される可能性があるのかどうか、可能性がある場合にどうするかが問題になりそうです
  • 要はボタンの押し忘れの対処方法とかです
  • 場合によっては作業期間IDのようなものをうまいこと作って、レコードにそれを記載し、同じIDを作業開始時のレコードと作業終了時のレコードに保存させるというのもあります（が、案外難しいです）。
• どういう風に表示するかのロジックも重要です。例えば日付で区切る、といった場合、作業が日をまたいだ時にどうするのかが悩ましいです

Comments

[anpnt]

ありがとうございます！補足でいろいろ追加しました！