RubyにおけるOR演算子の挙動について

Question

railsで簡単なWebアプリケーションを作成しています。
その中で、フィルタを使って特定のユーザーに制限をかける機能を実装しようとしているのですが、
if文を使って判定する行で期待する動きをせずに困っています。

def correct_user
      @event=Event.find_by(id: params[:event_id])
      @join_request = current_user.join_requests.find_by(event_id: params[:event_id])
      redirect_to root_url　if @event.user_id != current_user.id || @join_request.nil?
end

どのような処理を書いているかというと、イベントを作成したユーザーとjoin_requestを持っているユーザー以外はリダイレクトされるという処理です。
イベントを作成したユーザーはそのイベントに対するjoin_requestを持てないように設計しているのでOR(ここでは || )を使って制限をかけようとしています。どちらかの条件に引っかかるはずです。

以下のように条件が一つの時はそれぞれ期待する動きをしてくれます。

redirect_to root_url if @event.user_id != current_user.id
#もしくは
redirect_to root_url if @join_request.nil?

ORを使って条件を書くとこのようになります。

 redirect_to root_url if @event.user_id != current_user.id || @join_request.nil?

上記のように書くことで、二つのパターンに対応できるかと考えていましたが、実際にはイベントを作成したユーザーでログインした場合、
join_requestを持っているユーザーでログインした場合どちらもリダイレクトされてしまいます。正しいユーザーでログインしていますが、リダイレクトされます。correct_user以外にbefore_actionで定義しているフィルタはありません。

なぜこのように書くと期待した動きをしないのでしょうか。
ヒントや皆さんが思いついた挙動の確認方法などを教えていただけると嬉しいです。

他のコードを見たい方はコメントお願いします。
よろしくお願いします。

Comments

[gouf]

「root_url　if」のスペースが半角ではなく、全角スペースになっているようですが、現行のソースコードもそのようになっていますか? (あるいは単純に質問投稿時の編集ミスですか?)

[Sano]

コメントありがとうございます。
質問投稿時の編集ミスです。実際のコードは半角になっています。

Answer 1

何も不思議なことがないです。

イベントを作成したユーザーはそのイベントに対するjoin_requestを持てないように設計しているのでOR(ここでは || )を使って制限をかけようとしています。どちらかの条件に引っかかるはずです。

文字通りどちらかの条件に引っかかるはずです。

ユーザー	@event.user_id != current_user.id	@join_request.nil?	or
作成者	false	true	true
join持ち	true	false	true
作成者でもjoin持ちでもない	true	true	true
作成者かつjoin持ち	存在しない	存在しない	存在しない

なので、この場合「作成者でもjoin持ちでもない」時にリダイレクトしたいのならば&&を使うのが適切です。