「ActiveRecord::RecordNotFound in MessagesController#index」「Couldn't find User with 'id'=」とエラーが出てしまう。

Question

ツイッターのダイレクトメッセージのような感じで、メッセージやり取り機能を実装したいのですが、下記のエラーが出てしまいます。

ルーティングは下記です。

Rails.application.routes.draw do
  devise_for :users
  root 'messages#index'
  resources :users, only: [:edit, :update] do
    resources :messages, only: [:index, :create]
  end
end

コントローラーは下記です。

class MessagesController < ApplicationController
  before_action :set_user

  def index
    @message = Message.new
    @messages = @user.messages.includes(:user)
  end

  def create
    @message = @user.messages.new(message_params)
    if @message.save
      redirect_to user_messages_path(@user), notice: 'メッセージが送信されました'
    else
      @messages = @user.messages.includes(:user)
      flash.now[:alert] = 'メッセージを入力してください。'
      render :index
    end
  end

  private

  def message_params
    params.require(:message).permit(:content, :image).merge(user_id: current_user.id)
  end

  def set_user
    @user = User.find(params[:user_id])
  end
end

助言頂ければ幸いでございます。

---

【更新】

頂きました助言を元にデバッガ（binding.pry）を使い下記のように出ました。

    26: def set_user
 => 27:   binding.pry
    28:   @user = User.find(params[:user_id])
    29: end

[1] pry(#<MessagesController>)> params
=> <ActionController::Parameters {"controller"=>"messages", "action"=>"index"} permitted: false>
[2] pry(#<MessagesController>)> params[:user_id]
=> nil

「controller」というキーに対し「messages」という値が出ていることと、「action」というキーに対し、「index」という値が出ていることが、何を意味しているのか？

恐れ入りますが、助言を頂ければ幸いでございます。

Answer 1

ありがとうございます！kmdkuk様からのアドバイスもあり、無事解決に至りました。メッセージコントローラー内で、@userをcurrent_userに変更し、before_actionはindexを除外（ここではcreateのみ対象）に変更したところ通りました！

class MessagesController < ApplicationController
  before_action :set_user, only: %i[create]

  def index
    @message = Message.new
    @messages = current_user.messages.includes(:user)
  end

Answer 2

Couldn't find User with 'id'=
とエラーメッセージにある通り，params[:user_id]で指定したUserが作られていないのではないでしょうか?
また，作成したつもりなのにこのエラーが出ているのであれば，
デバッガなどを利用して，params[:user_id]が想定している値になっているかどうか確認してみてはいかがでしょうか．

Comments

[skem]

ありがとうございます！

デバッガ（binding.pry）を利用して作業してみました（※質問内容を更新しました）。
想定した値になっているかどうかの確認を進めたいと思います。

[kmdkuk]

`<ActionController::Parameters {"controller"=>"messages", "action"=>"index"}`の部分ですが，
controllerはリクエストが飛んできた先のコントローラ名，actionはそのリクエストをハンドリングしている，action名です，なので今回の結果で言うと，MessageController#indexにリクエストが飛んできていることを示しています．
やはり，params[:user_id]がnilになってしまっているので，リクエストパラメータとしてuser_idを渡せていないようですね

[skem]

ありがとうございます。[:user_id]を[:id]に変えるなどしてみたのですが、やはり渡せなかったので、ルーティングの方も確認してみます。

[kmdkuk]

それは，ルーティングのパスにuser_idを含めようとしているってことなんですかね？
~/messages/:user_idみたいなのはRESTの観点からあまりよろしくないように思いますが，
~/messages?user_id=1みたいなリクエストを送ることでやりたいことが実現できるのかなと思います．