質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

633閲覧

子インスタンスを持つ親スタンスメソッドでnilよりfalseを返したい

JuniorSirius

総合スコア38

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2021/11/27 07:27

編集2021/11/27 08:43

userdiarieshas_manyしていますが、diaryを持っていないuserに対して、rspecでテストをかけると
以下のコードのuser.diaries.order(created_at: :desc)diariesnilになってしまい、created_atでメソッドエラーが起きてしまうのですがこのメソッドでdiaryを持っていない、つまりnilなユーザーでuser.cannot_postfalseが帰るような組み方が知りたいです。お願いいたします。
if文(if diaries)で試しに組んでみましたが、nilエラーが起きました.

user.rb

ruby

1def cannot_post? 2 latest_diary = diaries.order(created_at: :desc).limit(1) 3 #現在時刻と最終投稿の時刻の差分が1日を経過していなかったらtrue 4 (Time.zone.now - latest_diary.first.created_at) < 1.day 5 end
  1. User インスタンスメソッド cannot_post? 1日以内に投稿していない場合 falseを返す
Failure/Error: (Time.zone.now - latest_diary.first.created_at) < 1.day NoMethodError: undefined method `created_at' for nil:NilClass

user_spec.rb

ruby

1describe 'cannot_post?' do 2 context '1日以内に投稿した場合' do 3 it 'trueを返す' do 4 create(:diary, user: user_a, created_at: Time.zone.now) 5 expect(user_a.cannot_post?).to be true 6 end 7 end 8 context '1日以内に投稿していない場合' do 9 let(:can_post_user) { create(:user) } 10 it 'falseを返す' do 11 expect(can_post_user.cannot_post?).to be false 12 end 13 end 14 end

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

y_waiwai

2021/11/27 07:33

言ってることが違うようですが、nilを返したいんでしょうか
JuniorSirius

2021/11/27 07:36 編集

すんません、全くの逆でfalseでした。true/falseでfalseを返したいの間違いでした。 よろしくお願いいたします
guest

回答1

0

自己解決

ありがとうございました。
やはり無闇にユースケースを1つのセンテンスでまとめようとしてロジックを混同混乱させてはいけないですね。
場合わけでnilケースのユーザーの場合でもその条件に合致するようfalseを返すようにしたらうまくいきました。

ruby

1def cannot_post? 2 if diaries.any? 3 latest_diary = diaries.order(created_at: :desc).limit(1) 4 #現在時刻と最終投稿の時刻の差分が1日を経過していなかったらtrue 5 (Time.zone.now - latest_diary.first.created_at) < 1.day 6 else 7 return false 8 end 9 end

投稿2021/11/27 08:49

JuniorSirius

総合スコア38

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問