【Rails】nil can't be coerced into Integerの空欄エラーを、エラーが出ないでバリデーションかけられるようにしたい

Question

実現したいこと

ここに実現したいことを箇条書きで書いてください。

• nilの場合は「~~」といった書き方にすると空欄でもエラーが出ないでバリデーションかけられるようにしたい

前提

Ruby on Railsで民泊アプリを作成しています。
チェックイン・チェックアウト・宿泊人数を空入力してsubmitすると、以下のようなエラーメッセージになってしまいます。

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

nil can't be coerced into Integer

つまり、nilを整数にできないよ！！ってエラーらしいです。

該当のソースコード

def sum_of_days
    unless start_date.nil? && end_date.nil?
      sum_of_days = (end_date.to_date - start_date.to_date).to_i
    else
      sum_of_days = 0
    end
  end

  def sum_of_price
    unless total_people.nil? && sum_of_days.nil?
      (room.fee * total_people * sum_of_days).to_i
    else
      sum_of_price = 0
    end
  end

試したこと

自分なりに、nilの場合でも、エラーが出ないでバリデーションかけられるようにif構文を書いてみたのですが、まだ駄目みたいです。
この場合は私の書き方が間違っているのでしょうか？

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

ここにより詳細な情報を記載してください。
ブランチ名：develop
GitHub

Answer 1

この場合は私の書き方が間違っているのでしょうか？

はい。

unless total_people.nil? && sum_of_days.nil?は、「両方がnil」でなければ、つまり片方でもnil以外の値ならという条件になっています。

Comments

[maisumakun]

&&を否定すると読みづらいので、if total_people.present? && sum_of_days.present?と肯定形で書くのがわかりやすいです。

[cooler_114]

なるほど！
maisumakunさま、ご指摘ありがとうございます！
修正してもう一回やってみます！