current_user.admin?について

Question

###実現したいこと
view/posts/index.html.slimでcurrent_user.admin?を使えないのはなぜでしょうか。

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

メソッドが見つかりませんと出ます

エラーメッセージ
ActionView::Template::Error (undefined method `admin?' for nil:NilClass):

該当のソースコード

＠該当箇所
- if current_user.admin?
  = link_to '編集', edit_post_path(plan), class: 'btn btn-primary mr-3'
  = link_to '削除', post,method: :delete, data:{confirm: "「#{post.title}」を削除します。よろしいですか？"}, class: 'btn btn-danger'

＠current_user指定場所(controllers/application_controller.erb)
class ApplicationController < ActionController::Base
  helper_method :current_user


  private

  def current_user
    @current_user ||= User.find_by(id: session[:user_id]) if session[:user_id]
  end
end

＠posts_controllerのindexアクション
  def index
    @q = Plan.ransack(params[:q])
    @plans = @q.result(distinct: true).page(params[:page]).per(15).recent
  end

試したこと

indexアクションに@user = User.find_by(params[:id])を追加して、
if current_user.admin?の部分を @user.admin?にしたらエラーが起きなくなった。

しかし、なぜcurrent_userだとadmin?メソッドが効かないのか。
また、view/users/index.html.slimではif current_user.admin?でエラーなく実装することができました。
それはなぜなのでしょうか。どなたかご教授いただけると幸いです。

補足情報（FW/ツールのバージョンなど）

Ruby version 2.6.3
Rails 5.2.3

Comments

[asm]

> @user = User.find_by(params[:id])
posts/indexのparams[:id]ってユーザーIDなのでしょうか？
非ログイン時にもadmin判定されてそうな危うさを感じます。

[ina76]

> asmさん
おっしゃる通りで、postのidでした。
教えていただきありがとうございます。

Answer 1

「current_userだとadmin?メソッドが効かない」ではなく「current_userがnilである」のが問題の本質だと思います。

コントローラから@current_userとして渡しているのであれば、@current_user.admin?とするのが正しいのではないでしょうか。

Comments

[ina76]

そういうことでしたか！本質的な問題は、nilだからなんですね。
@current_user.admin?でも例外が発生してしまったので、
nilの時に例外がでないようなメソッドを作って対応してみます！
ありがとうございます！

Answer 2

自己解決したので、ご報告させていただきます。
iwamotさんがおっしゃってた通り、current_userがnilであることで、admin?メソッドが使えず、例外が発生しておりました。

そこで下記のメソッドを定義し、if user_logged_in? && current_user.admin?とすることで例外がでなくなりました。

def user_logged_in?
!current_user.nil?
end

user_logged_in?メソッドは、もしcurrent_userがnilの場合は、falseを返すということなので、falseが発生した時点で、current_user.admin?は読み込まれなくなり、例外が発生しないということになります。

ご回答くださった皆さまありがとうございました。

Comments

[asm]

current_user&.admin?
が個人的にはおすすめです。

[ina76]

なるほど！ぼっち演算子を使うという手がありましたか！教えていただきありがとうございます！