rails 6 でバリデーションを作動できない

Question

実現したいこと

rails６を使ってタスク管理アプリを作成中です。
タスク作成の際にバリデーションで予定期間のエラーを表示したいです。
具体的には開始日と終了日があり、まずどちらかもしくは両方が空欄だとエラーメッセージを出し、開始日より前の日付で終了日を入力するとエラーメッセージを出すようにしたいのです。

発生している問題・分からないこと

開始日終了日両方が空欄の場合、片方空欄の場合どちらもエラーが出ます。
ただ開始日より早い日を終了日に指定した場合はメッセージが出ます。

エラーメッセージ

両方空欄の場合
NoMethodError in TasksController#create
undefined method `<=' for nil:NilClass
Extracted source (around line #7):
              
  def start_end_check
    errors.add(:end, "は開始日より前の日付は登録できません。") unless
    self.start <= self.end 
  end

片方空欄の場合
ArgumentError in TasksController#create
comparison of Date with nil failed
Extracted source (around line #7):
              
  def start_end_check
    errors.add(:end, "は開始日より前の日付は登録できません。") unless
    self.start <= self.end 
  end

該当のソースコード

<h1>タスク一覧</h1>
<div><% flash.each do |key, value| %>
  <%= content_tag(:div, value, class: key) %>
 <% end %>
</div>
<table class="head">
  <thead>
    <tr>
      <th>iD</th>
      <th>タイトル</th>
      <th>開始日</th>
      <th>終了日</th>
      <th>終日</th>
      <th>更新日</th>
    </tr>
  </thead>
  <tbody>
    <p>今日の日付：<%= l @tasks_today, format: :default %></p>

    <% @tasks.each do |task| %>
      <tr>
        <td class="id"><%= task.id %></td>
        <td class="title"><%= task.title %></td>
        <td class="start"><%= l task.start, format: :default%></td>
        <td class="end"><%= l task.end, format: :default %></td>
        <td class="comp"><% if task.completed == 1 %>
          ○
         <% else %>
          <!-- タスクが未完了の場合表示しない -->
         <% end %></td>
        <td class="up"><%= l task.updated_on, format: :time %></td>
        <td><%= link_to '編集', edit_task_path(task),class:"edit" %></td>
        <td><%= link_to '詳細', task_path(task),class:"show" %></td>
        <td class="delete"><%= button_to '削除', task_path(task.id), method: :delete, data: {confirm: "削除してもよろしいですか?"} %>
        </td>
      </tr>
    <% end %>
  </tbody>
</table>
<p>合計タスク数:<%= @tasks_count %></p>
<%= link_to 'タスクの追加', new_task_path, class:"add" %>

class Task < ApplicationRecord
  validates :title, :description, :start, :end, presence:  { message: "を入力して下さい。" }

  validate :start_end_check
  def start_end_check
    errors.add(:end, "は開始日より前の日付は登録できません。") unless
    self.start <= self.end 
  end


  validates :title, length: { maximum: 20, message: "は20字までにして下さい。"}
  validates :description, length: { maximum: 500, message: "は500字までにして下さい。"}
end

class CreateTasks < ActiveRecord::Migration[6.1]
  def change
    create_table :tasks do |t|
      t.string :title
      t.text :description
      t.date :start
      t.date :end
      t.boolean :check
      t.integer :completed
      t.datetime :updated_on

      t.timestamps
    end
  end
end

class TasksController < ApplicationController
  def index
    @tasks = Task.all
    @tasks_count = Task.count
    @tasks_today = Date.today
  end

  def new
    @task = Task.new
  end

  def create
    @task = Task.create(task_params)
    if @task.save
       redirect_to controller: :tasks, action: :index
       flash[:notice] = "タスクを作成しました。"
    else
       flash.now[:alret] = "作成に失敗しました"
       render 'new',status: :unprocessable_entity
    end
  end
  def edit
    @task = Task.find(params[:id])
  end
  def show
    @task = Task.find(params[:id])
  end
  def update
  @task = Task.find(params[:id])
  if @task.update(task_params)
    flash[:notice] = "更新しました。"
    redirect_to tasks_path
   else
    flash.now[:notice] = "更新に失敗しました"
    render 'edit',status: :unprocessable_entity
   end
  end
  def destroy
    @task = Task.find(params[:id])
    @task.destroy
    redirect_to tasks_path,notice:"削除しました。"
  end
  

  private
   def task_params
    params.require(:task).permit(:title,:description,:start,:end,:completed,:updated_on)
   end
  end

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

似た事例を見つけ、nilメソッドやblancメソッドを使うらしいということは分かりましたが、具体的にどう使えばいいか分からない状態です。

補足

特になし

Answer 1

開始日終了日両方が空欄の場合、片方空欄の場合どちらもエラーが出ます。

エラーメッセージのとおりです。空欄（nil）の場合、日付の大小比較自体が不可能です。

presenceがあっても他のバリデーションは実行されますので、nilの場合は大小比較のバリデーションを無言でスルーさせましょう。

  def start_end_check
    return unless start && self.end # 片方がnilならそのまま抜ける
    errors.add(:end, "は開始日より前の日付は登録できません。") unless start <= self.end 
  end

Comments

[G23]

回答ありがとうございます。！
無事に解決しました。