rubyでnilになる可能性のある値をif文条件に使いたいけどNoMethodErrorが出てしまう

Question

下記の条件文でNoMethodErrorが出てしまうのですがどう書くとnilが入ってきてもerrorを回避できるでしょうか？

if current_user.id == user.id
else
end

• current_user.idはnilの可能性もある

Answer 1

Ruby2.3で追加された&.(Safe Navigation Operator)や、Railsに限ればtryを使うと楽です。

Comments

[you88]

ありがとうございます！(Safe Navigation Operator)使ってみます

Answer 2

まえもってnilかどうか判断しとけばいいんでは

Comments

[you88]

if current_user.id.nil?
if current_user.id == user.id
else
end
else
end

って感じですかね？スマートに書く方法ないかなと思い

[y_waiwai]

それを条件式でANDしましょう
nil判定の方を先にね

[coco_bauer]

NoMethodErrorのエラーが出たのは、user か current_userがnilなので、クラスが判らない。
だから、.idなんて書かれても、 そんなMethodありゃしませんというメッセージが表示された、という事です。
current_user.idがnilである（current_userは、idというメソッドを持っている）というのは問題ではなくて、userかcurrent_userがnilであることが問題のはずですから、userとcurrent_userがnilでないことを判断すべきではないでしょうか？

[you88]

>current_user.idがnilである（current_userは、idというメソッドを持っている）というのは問題ではなくて、userかcurrent_userがnilであることが問題のはずですから、userとcurrent_userがnilでないことを判断すべきではないでしょうか？

なるほど！

if current_user.nil? && current_user.id == user.id
else
end

って感じですかね？