ruby-on-rails - このif句を書くためのより良い/よりクリーンな方法

Question

UsersController以下は私のアクションからのコードです。セッションからのcurrent_userオブジェクトがあり、アクションに何も渡されない場合は、現在のユーザーのIDを使用したいと思います。

def show
  id = params[:id]
  id = current_user.id if id.blank?
  @user = User.find(id)
end

最終的には、他のユーザーの情報を編集できる管理者がいるようにするためにこれを行っています。だから私は使用しません@user = User.find(current_user.id)。

考え/提案？おそらく、三項演算子も実行できます。

params[:id].present? ? id = params[:id] : id = current_user.id

しかし、おそらくもっと良い方法があると思います。

Answer 1

私はこの方法が好きです

def show
  @user = User.find(params[:id] || current_user.id)
end

Answer 2

CanCangemを使用できます

私はこのチュートリアルに従います：TonyAmoyalのWebサイト