投稿者のみが消去できる。if文で分けるのにuser_signed_in?は必要？current_user.id == @tweet.user_idだけでいい？

Question

とあるカリキュラムに沿って勉強しています。
deviseを使っています。
簡単なメッセージ投稿アプリを作っていて、メッセージには投稿者の情報が紐づいています。
メッセージを消去するリンクを投稿者本人にのみ表示しようと思うのですが、それをif文で表示非表示を分けます。

カリキュラムには
if user_signed_in? && current_user.id == @tweet.user_id
と書いてあるのですが、
current_user.id == @tweet.user_idがtrueになるのはif user_signed_in?の時だけなので、if user_signed_in?が不要に思います。
先輩方いかがでしょうか？

Answer 1

current_user.id == @tweet.user_idがtrueになるのはif user_signed_in?の時だけなので、if user_signed_in?が不要に思います。

current_userを使う場合、ログイン状態の確認は常に必要です。
ログインしていない状態でページを表示すると、current_userはnilですよね？
だから、user_signed_in?を削除してif current_user.id == @tweet.user_idと書いてしまうと、
if nil.id == @tweet.user_idを実行しようとしてエラーが発生します。

したがって、まずログインしているか＝current_userがnilでないかを確認した上で、
current_userを使う必要があります。

Comments

[annaPanda]

なるほど、コンソールで
```
[1] pry(main)> nil == 1
=> false
```
とまで見ていたのでfalseで返ると思っていたのですが、nilにidをかけあわせるとダメなんですね。
ありがとうございました。