Railsでattr_accessor初期化方法について

Question

前提

DBにstart_datetimeというカラムがありDatetimeクラスで保存しています。
その年月日を保存・取り出すためにstart_day_accessor、時間を保存・取り出すためにstart_time_accessorを準備しています。

class Schedule < ApplicationRecord
  attr_accessor :start_day_accessor
  attr_accessor :start_time_accessor

  before_save :datetime_format
  def datetime_format
    self.start_datetime = Time.zone.parse(self.start_day_accessor 
　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　+ ' ' + self.start_time_accessor).to_datetime
end

<%= form.text_field :start_day_accessor %>
<%= form.select :start_time_accessor, options_for_select(time_array_for_select) %>

# time_array_for_select = ['', '00:00', '00:15', '00:30', '00:45', '01:00',..... ]

実現したいこと

attr_accessorを初期化してDBからの値を渡したいです。

具体的には、

• 編集ページでは、保存されているレコードの値をそれぞれ（Select、Input）のフォームに表示したいです。
• 新規ページでは、バリデーションエラーのときに入力した値を保持したいです。

初期化したい値はそれぞれ下記となります。
:start_day_accessorに@schedule.start_datetime.strftime("%Y-%m-%d")
:start_time_accessorに@schedule.start_datetime.strftime("%H:%M")

試したこと

基本的なことだと思うのですが、かなり検索して手を動かしたのですが、attr_accessorへの理解が足りないため解決策にたどり着けませんでした。
下記だとバリデーションエラーのときに入力した値が保持できませんでした。

<%= form.text_field :start_day_accessor, value: @schedule.start_datetime.nil? ? '' : @schedule.start_datetime.strftime("%Y-%m-%d")　%>
<%= form.select :start_time_accessor, options_for_select(time_array_for_select, @schedule.start_datetime.nil? ? '' : @schedule.start_datetime.strftime("%H:%M")) %>

下記にて解決しました。

#controller
def schedule_params
  params[:schedule][:start_datetime] = (params[:schedule][:start_date] + ' ' + params[:schedule][:start_time]).to_s
  params.require(:schedule).permit(...
end

#view
  <%= form.text_field :start_date %>
  <%= form.text_field :start_time %>

#model
class Schedule < ApplicationRecord
  def start_date
    start_datetime.strftime("%Y-%m-%d") if start_datetime.present?
  end

  def start_time
    start_datetime.strftime("%H:%M") if start_datetime.present?
  end
end

Answer 1

その年月日を保存・取り出すためにstart_day_accessor、時間を保存・取り出すためにstart_time_accessorを準備しています。

このような構造そのものが妥当ではありません（もとのstart_datetimeとどのように同期するかが煩雑となってしまいます）。

Comments

[no1knows]

ありがとうございます。構造そのものが妥当ではないということは、DBにstart_datetimeカラムではなく、DateカラムとTimeカラムを用意したほうが好ましいということでしょうか？

[maisumakun]

> DBにstart_datetimeカラムではなく、DateカラムとTimeカラムを用意したほうが好ましいということでしょうか？

その手もありですが、もう1つの方法としては「日付・時間の読み書きメソッドを、attr_accessorではなくstart_datetimeを読み書きする形で手書きする」という方法も考えられます。

[no1knows]

日付と時間を別々のカラムにするとそれはそれで大変そうなので
> 日付・時間の読み書きメソッドを、attr_accessorではなくstart_datetimeを読み書きする形で手書きする
こちらをやってみたいのですが、もう少し具体的に説明いただけないでしょうか？

（試したことに記載のあるようにstart_datetime.strftime("%Y-%m-%d")で初期値を設定したのですが、バリデーションがうまくいかなかったです。）

[maisumakun]

> start_datetime.strftime("%Y-%m-%d")で初期値を設定したのですが

「初期値を設定する」（start_datetimeと別にstart_dateの値を持つ）という考えそのものが妥当ではないのです。

attr_accessorは使わず、start_dateを「start_datetimeから日付部分だけ取り出すメソッド」として定義してください。

[no1knows]

ありがとうございます！期待通り動作することができました！
（attr_accessorだけでなくモデルの理解が不十分だということがわかりました。。。）