simple_calendarを利用して予約システムを作成していたがTypeErrorが発生する

Question

前提

Railsで予約システムを作っています。
https://qiita.com/sssssatou/items/2e6606e3ddf9b246a0fb
こちらの記事を参考にアレンジしているのですが、途中でエラーが発生してしまいました。

実現したいこと

ユーザー(生徒)でログイン
↓
授業を受けたい先生を選ぶ
↓
その先生のスケジュールの中から空いている時間は予約を入れることができる

参考にしている記事(https://qiita.com/sssssatou/items/2e6606e3ddf9b246a0fb#3%E4%BA%88%E7%B4%84%E3%81%AE%E6%9C%89%E7%84%A1%E3%81%AB%E3%82%88%E3%81%A3%E3%81%A6%E7%94%BB%E9%9D%A2%E3%81%AE%E8%A1%A8%E7%A4%BA%E3%82%92%E5%A4%89%E6%9B%B4%E3%81%99%E3%82%8B)
では予約の有無を探すときに

module ReservationsHelper

  # 略

  def check_reservation(reservations, day, time)
    result = false
    reservations_count = reservations.count
    # 取得した予約データにdayとtimeが一致する場合はtrue,一致しない場合はfalseを返します
    if reservations_count > 1
      reservations.each do |reservation|
        result = reservation[:day].eql?(day.strftime("%Y-%m-%d")) && reservation[:time].eql?(time)
        return result if result
      end
    elsif reservations_count == 1
      result = reservations[0][:day].eql?(day.strftime("%Y-%m-%d")) && reservations[0][:time].eql?(time)
      return result if result
    end
    return result
  end
end

となっており、開始時間が一致している予約のみが反応し、その開始時間に予約を入れられないようになります。

しかし、私が作ろうと思っているシステムは予約の開始時間から終了時間まで、新規予約を入れることができないものです。それを作るために

if reservations_count > 1
      reservations.each do |reservation|
        result = reservation[:day].eql?(day.strftime("%Y-%m-%d")) && reservation[:time].eql?(time)
        return result if result
      end
    elsif reservations_count == 1
      result = reservations[0][:day].eql?(day.strftime("%Y-%m-%d")) && reservations[0][:time].eql?(time)
      return result if result
    end
    return result

この部分を

if reservations_count > 1
      reservations.each do |reservation|
        result = reservation[:start_time] <= DateTime.parse(day + " " + time + " " + "JST") && DateTime.parse(day + " " + time + " " + "JST") <= reservation[:end_time]
        return result if result
      end
    elsif reservations_count == 1
      result = reservations[0][:start_time] == DateTime.parse(day + " " + time + " " + "JST") && DateTime.parse(day + " " + time + " " + "JST") <= reservations[0][:end_time]
      return result if result
    end
    return result

このように変更しました。
しかし、TypeErrorが発生してしまいます。

rails cで
reservation[:start_time] <= DateTime.parse(day + " " + time + " " + "JST") && DateTime.parse(day + " " + time + " " + "JST") <= reservation[:end_time]
をしましたがTrueもしくはFalseが返ってくるため、間違っていないと思うのですが、解決策などアドバイスをお願いいたします。

該当のソースコード

module ReservationsHelper
  def times
    times = ["9:00",
             "9:30",
             "10:00",
             "10:30",
             "11:00",
             "11:30",
             "13:00",
             "13:30",
             "14:00",
             "15:00",
             "15:30",
             "16:00",
             "16:30",
             "17:00",
             "17:30",
             "18:00",
             "19:00",
             "19:30",
             "20:00"
            ]
  end

  def check_reservation(reservations, day, time)
    result = false
    reservations_count = reservations.count
    if reservations_count > 1
      reservations.each do |reservation|
        result = reservation[:start_time] <= DateTime.parse(day + " " + time + " " + "JST") && DateTime.parse(day + " " + time + " " + "JST") <= reservation[:end_time]
        return result if result
      end
    elsif reservations_count == 1
      result = reservations[0][:start_time] == DateTime.parse(day + " " + time + " " + "JST") && DateTime.parse(day + " " + time + " " + "JST") <= reservations[0][:end_time]
      return result if result
    end
    return result
  end
end

<div class="container">
  <div class="row">
    <div class="col-12 text-center">
      <h1><%= @teacher.teacher_name %>先生予約画面</h1>
      <p>3ヶ月先まで予約することができます。</p>
    </div>
    <div class="col-12 mt-3">
      <%= week_calendar events: @reservations do |date, reservations| %>
        <%= date.day %>
      <% end %>
    </div>
  </div>
</div>

<div class="simple-calendar">
  <div class="calendar-heading">
    <%= link_to "先週", calendar.url_for_previous_view %>
    <% if calendar.number_of_weeks == 1 %>
      <span class="calendar-title"><%= t('simple_calendar.week', default: 'Week') %> <%= calendar.week_number %></span>
    <% else %>
      <span class="calendar-title"><%= t('simple_calendar.week', default: 'Week') %> <%= calendar.week_number %> - <%= calendar.end_week %></span>
    <% end %>
    <%= link_to "来週", calendar.url_for_next_view %>
    <% reservations = Reservation.where(teacher_id: params[:teacher_id]).reservations_after_three_month %>
  </div>

  <table class="table table-striped">
    <thead>
      <tr>
        <th>時間</th>
        <% date_range.slice(0, 7).each do |day| %>
          <th><%= t('date.abbr_day_names')[day.wday] %></th>
        <% end %>
      </tr>
    </thead>

    <tbody>
      <% date_range.each_slice(7) do |week| %>
        <tr>
          <td></td>
          <% week.each do |day| %>
            <%= content_tag :td, class: calendar.td_classes_for(day) do %>
              <% if defined?(Haml) && respond_to?(:block_is_haml?) && block_is_haml?(passed_block) %>
                <% capture_haml(day, sorted_events.fetch(day, []), &passed_block) %>
              <% else %>
                <% passed_block.call day, sorted_events.fetch(day, []) %>
              <% end %>
            <% end %>
          <% end %>
        </tr>
        <% times.each do |time| %>
          <tr>
            <td><%= time %></td>
            <% week.each do |day| %>
              <td>
              <% if check_reservation(reservations, day, time) %>
                <%= '×' %>
              <% else %>
                <%= link_to new_user_teacher_reservation_path(current_user, @teacher.id, day: day, time: time) do %>
                  <%= '○' %>
                <% end %>
              <% end %>
              </td>
            <% end %>
          </tr>
        <% end %>
      <% end %>
    </tbody>
  </table>
</div>

class Reservation < ApplicationRecord
  belongs_to :user
  belongs_to :teacher
  def self.reservations_after_three_month
    # 今日から3ヶ月先までのデータを取得
    reservations = Reservation.all.where("day >= ?", Date.current).where("day < ?", Date.current >> 3).order(day: :desc)
    # 配列を作成し、データを格納
    # DBアクセスを減らすために必要なデータを配列にデータを突っ込んでます
    reservation_data = []
    reservations.each do |reservation|
      reservations_hash = {}
      reservations_hash.merge!(day: reservation.day.strftime("%Y-%m-%d"), time: reservation.time)
      reservation_data.push(reservations_hash)
    end
    reservation_data
  end
end

class ReservationsController < ApplicationController
  before_action :authenticate_user!
  def index
    @reservations = Reservation.all.where("day >= ?", Date.current).where("day < ?", Date.current >> 3).where(teacher_id: params[:teacher_id]).order(day: :desc)
    @teacher = Teacher.find(params[:teacher_id])
  end

  def choice
    @teachers = Teacher.all
  end

  def new
    @user = current_user
    @teacher = Teacher.find(params[:teacher_id])
    @reservation = Reservation.new
    @day = params[:day]
    @time = params[:time]
    @teacher_id = params[:teacher_id]
    @start_time = DateTime.parse(@day + " " + @time + " " + "JST")
    @end_time = @start_time + 90.minutes
  end

  def show
    @reservation = Reservation.find(params[:id])
  end

  def create
    @reservation = Reservation.new(reservation_params)
    if @reservation.save
      redirect_to user_teacher_reservation_path(@reservation.user_id, @reservation.teacher_id, @reservation.id)
    else
      render :new
    end
  end

  private
  def reservation_params
    params.require(:reservation).permit(:day, :time, :user_id, :teacher_id, :start_time, :end_time)
  end
end

# This file is auto-generated from the current state of the database. Instead
# of editing this file, please use the migrations feature of Active Record to
# incrementally modify your database, and then regenerate this schema definition.
#
# This file is the source Rails uses to define your schema when running `bin/rails
# db:schema:load`. When creating a new database, `bin/rails db:schema:load` tends to
# be faster and is potentially less error prone than running all of your
# migrations from scratch. Old migrations may fail to apply correctly if those
# migrations use external dependencies or application code.
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 2022_11_14_060848) do

  create_table "reservations", charset: "utf8mb4", collation: "utf8mb4_0900_ai_ci", force: :cascade do |t|
    t.date "day", null: false
    t.string "time", null: false
    t.bigint "user_id", null: false
    t.bigint "teacher_id", null: false
    t.datetime "start_time", null: false
    t.datetime "created_at", precision: 6, null: false
    t.datetime "updated_at", precision: 6, null: false
    t.datetime "end_time", null: false
  end
略

end

補足情報（FW/ツールのバージョンなど）

ruby '2.7.5'
rails '6.1.6'
simple_calendar '2.0'

Comments

[no1knows]

> TrueもしくはFalseが返ってくるため、間違っていないと思うのですが
とのことですが、エラー文は下記のようになっています。まずはここから調べてみてはいかがでしょうか？
expected numeric rails

またエラー画面は、画像をはるのではなく、コードをコピーしてください。

[winterboum]

＞またエラー画面は、画像をはるのではなく、コードをコピーしてください。
です。
潰れてしまって読めないし、検索もできないし

[keiino0425]

アドバイスありがとうございます！以後気をつけます！

Answer 1

result = reservations[0][:start_time] == DateTime.parse(day + " " + time + " " + "JST") && DateTime.parse(day + " " + time + " " + "JST") <= reservations[0][:end_time]
の
reservations[0][:start_time] は String で DateTime.parse(day + " " + time + " " + "JST") は DateTime です。
ので == できないということでしょう。
参考では reservation[:day].eql?(day.strftime("%Y-%m-%d")) と両方Stringで比べてます。

rails cで 通ったということですが、その時の reservations や day、 time をどのように設定したのか、という所が明らかでないので 通った原因はわかりません。

Comments

[keiino0425]

回答ありがとうございます！classが違うので比較できないんですね！

rails cでは
Reservation.find(7)[:start_time] <= DateTime.parse("2022-11-18" + " " + "10:00" + " " + "JST") && DateTime.parse("2022-11-18" + " " + "10:00" + " " + "JST") <= Reservation.find(7)[:end_time]
として行いましたがTrueが返ってきました。
(Reservation.find(7)[:start_time]は
Fri, 18 Nov 2022 10:00:00.000000000 JST +09:00
Reservation.find(7)[:end_time]は
Fri, 18 Nov 2022 11:30:00.000000000 JST +09:00)

また、Reservation.find(7)[:start_time].class としたところ、Stringが表示されるかと思いましたが
ActiveSupport::TimeWithZoneが表示されました。
これがStringではない理由もわからないのですが教えていただいてよろしいでしょうか。
お願いいたします。

[keiino0425]

>> reservations[0][:start_time] は String で DateTime.parse(day + " " + time + " " + "JST") は DateTime です。
ので == できないということでしょう。
参考では reservation[:day].eql?(day.strftime("%Y-%m-%d")) と両方Stringで比べてます。

と言われていますがscheme.rbで定義されているように
reservationのdayはDateクラス、start_timeはDatetimeクラスなのにStringで表示されるのはなぜなのでしょうか。
rails c でそれぞれ表示すると
Reservation.find(7)[:day].classはDate,
Reservation.find(7)[:start_time].classはActiveSupport::TimeWithZoneでした。

理由を教えていただけますと幸いです。

Answer 2

pry-railsを使ってデバックをしました。

[2] pry(#<#<Class:0x0000aaab14413fb0>>)> Time.zone.parse(day + " " + time + " " + "JST")
TypeError: expected numeric
from /usr/local/bundle/gems/activesupport-6.1.7/lib/active_support/core_ext/date/calculations.rb:94:in `+'

となっていたためparseの中が上手く処理できていないのではないかと思い、

Time.zone.parse(day.to_s + " " + time + " " + "JST")

としたところ上手く行きました。

Comments

[no1knows]

下記も参考になるかもしれいので、お時間があるときにでも確認してみてください。
https://qiita.com/jnchito/items/cae89ee43c30f5d6fa2c