controller(concern)のパラメータ部分を疎結合化したい

Question

Ruby on Railsチュートリアルを参考にログイン周りをconcernに分割、共通化しました。

ユーザ系の画面で利用する分には問題なかったのですが、
今回、記事登録画面で利用する際に
concern側のparams[:id]を取得する箇所でエラーになりました。

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

今まではUserコントローラで使ってたのでparams[:id]で問題なかった（意識してなかった）のですが、
今回は、Postコントローラから利用するためparams[:id]だと渡せず、
params[:user_id]とすると通るのですが、そうすると今度は既存のUserコントローラから動かなくなります。

そこでconcern側にparamsパラメータを渡すような作り（viewのパーシャルのような）にするべきなのかなと思いました。

この具体的な方法が分かりません。
分かる方教えていただけないでしょうか？

もしくは、根本的に違うよって部分があればご指摘いただけると幸いです。

宜しくお願い致します。

・ログイン周りの共通処理を行うコントローラ(concern)

class LoginController < ApplicationController
  private
    
    # beforeアクション 正しいユーザーかどうか確認
    def correct_user
      @user = User.find(params[:id])
      redirect_to(root_url) unless @user == current_user
    end  
end

・ユーザ編集画面などからconcernを利用してました

class UsersController < LoginController
  before_action :logged_in_user, only: [:edit, :update]
  before_action :correct_user,   only: [:edit, :update]

  def edit
    @user = User.find(params[:id])
  end
  
  def update
    @user = User.find(params[:id])
    if @user.update_attributes(user_params)
      flash[:success] = "更新しました"
      redirect_to mypage_path(@user)
    else
      render 'edit'
    end
  end
end

・記事登録画面、ここからもconcernを利用したい

class PostsController < LoginController
  before_action :logged_in_user, only: [:new, :create, :edit, :update]
  before_action :correct_user,   only: [:new, :create, :edit, :update]
  
  def new
    @post = Post.new(user_id: current_user)
  end
  
  def create
    @post = Post.new(post_params)
    if @post.save
      flash[:success] = "記事を登録しました。"
      redirect_to mypage_path(current_user)
    else
      render 'new'
    end
  end
  

end

ちなみに記事管理画面は、以下のように呼び出してます。

パス

アプリURL/user/4/posts/new

route

  resources :user, only: [] do 
    resources :posts
  end

Answer 1

根本的に違うよって部分があればご指摘いただけると幸いです。

current_userについてしか操作を認めないのであれば、そもそもidとしてURL経由で渡す必然性そのものがないのではないでしょうか。

Comments

[pecchan]

有難う御座います。

はい、仰る通り
「current_userについてしか操作を認めない」
です。

理解力が悪く、大変申し訳ないのですが、以下の意味がよく分かりません。
>そもそもidとしてURL経由で渡す必然性そのものがないのではないでしょうか。

※転機ミスで一部修正しました。

[maisumakun]

操作対象のuserをcurrent_userにしてしまえば、User.find(params[:id])そのものが不要となります。

[maisumakun]

URLはmy_page/editのようなものでいいでしょう。

[pecchan]

ご指摘有難う御座います。

>操作対象のuserをcurrent_userにしてしまえば、User.find(params[:id])そのものが不要となります。

PostsController の newから
@user = User.find(params[:id])
の行を削除しました。

[pecchan]

>URLはmy_page/editのようなものでいいでしょう。

PostはUserが多数所持するので、
てっきり以下のようなURLになるべきと思っていました
アプリURL/user/4/posts/new

なるほど、以下のようになるようにしてみます。
アプリURL/post/id/new

URLにurser_idを含める必要がないのでは？
というご指摘がやっと理解できました。