each文の中で変数を取り出したい

Question

前提・実現したいこと

each文の中で変数pを取り出したいです。
binding.pryで止めるとp.idでちゃんと「１」と取り出したい値が取り出せるのですが、いざコードに戻ると下記のエラーが出てしまいます。

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

undefined method `id' for nil:NilClass

該当のソースコード

 def index
    @users = User.all
 end

 <% @users.each do |user| %>
    <% p = Post.find_by(id: user.id) %>
       <%= p.id %>
       <% binding.pry %>
 <% end %>

試したこと

binding.pryでデバッグすると取り出したい値が取り出せました。
ご教授いただけると幸いです。

Answer 1

Post.find_byは、条件で見つからなかった場合にnilを返します。

全UserについてPostがあればいいのですが、ループで回すとPostのないUserが1人でもいた場合にpがnilとなります。

binding.pryで止めた時に値が取れたのは、そのときのUserにPostがあったからで、回し続けていけばエラーになるかと思います。

（本題ではありませんが、pというメソッドがありますので、変数名にpを使うのはおすすめできません）

Comments

[Yohei04]

ご回答ありがとうございます????‍♂️
find_byにはそういった仕様があったのですね。
回答いただいた通り、PostのないUserを削除したら上手くいきました。

pというメソッド知らなかったす。。以後変数では使用しないようにします。

迅速な回答とても助かりました。ありがとうございました！！