railsのparams、session周りに関する質問

Question

疑問

railsので何かを参照する、showメソッドの関数はこういう物がノーマルだと思う。以下に記す。

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

しかしながら、ログイン機能を保持しておりそのsessionの中にその人のユーザーIDを持たせるのであれば、以下の様なshowメソッドでも良い気がしました。（他人のアカウントは見れないし

def show
  @user = User.find(session[:user_id])
end

これは何か問題があったりするのでしょうか？こういったコードを見かけたことがあまりないなと思いました。
ご教授いただければ幸いです。

Answer 1

要件次第なところはあります。セッションに何をどこまで持たせるかとか。
そもそもparamsとsessionだとデータの取得元が違うので、利用される場面も違います。

params→リクエストを元に検索
session→セッションに保持された情報を元に検索

今回だと、なにか特定のユーザーの情報を確認したいか、ログインユーザーの情報を確認したいか、と要件が明確に違います。
「ログインユーザーの情報」なら、そもそもセッションにモデルごと保管してしまえばわざわざfindする必要もなかったりします。

Comments

[m.ts10806]

「他のユーザーの情報は見せない」なら、「そもそもそういうルーティングを作らない」という設計もあり得ます。
この場合はリクエストを使わずセッションを使い、ルーティングURLも固定で良くなります。

[zuck_]

なるほどです。ありがとうございました！

Answer 2

これは何か問題があったりするのでしょうか？

resource（単数形）を使うなどで適切なルーティングを引けば、特に問題ありません。

通常のresourcesでshowを立てると、内部で全く使わないとしても、users/1のようなID付きのURLでないとアクセスできません。