ActiveRecord::RecordNotFound in RelationshipsController#create

Question

https://teratail.com/questions/231324
こちらの記事参考に

f.hidden_field :follow_id,

とするか
params[:follow_id]
とするか。

なお、二つ問題があります。
before_action :set_user　なので、indexなどparamsがこないactionでも呼ばれてしまうので、そのときエラーになります。only: [...] をつけると良いです

set_user　では　user = となっています。これでは 実際のactionでは使えません
@user = として、createなどでは @user で参照するようにしてください

実行しましたが、状況変わらず。。。
ご教授お願いします。。。

エラー画面

Couldn't find User with 'id'=

  private
  def set_user
    @user = User.find(params[:relationship][:follow_id])
  end

end

relationshipsコントローラー

class RelationshipsController < ApplicationController

  before_action :set_user, only: [:create, :destroy]

  def create
    following = current_user.follow(@user)
    if following.save
      flash[:success] = 'ユーザーをフォローしました'
      redirect_to @user
    else
      flash.now[:alert] = 'ユーザーのフォローに失敗しました'
      redirect_to @user
    end
  end

  def destroy
    following = current_user.unfollow(@user)
    if following.destroy
      flash[:success] = 'ユーザーのフォローを解除しました'
      redirect_to @user
    else
      flash.now[:alert] = 'ユーザーのフォロー解除に失敗しました'
      redirect_to @user
    end
  end

  private
  def set_user
    @user = User.find(params[:relationship][:follow_id])
  end

end

relatipnshipモデル

class Relationship < ApplicationRecord

  belongs_to :user
  belongs_to :follow, class_name: 'User'

  validates :user_id, presence: true
  validates :follow_id, presence: true
  
end

userモデル

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  with_options presence: true do
    validates :name
    validates :birth_day
    validates :gender
    validates :hobby
    validates :self_introduction, length: { maximum: 200 }
    validates :image
  end

    enum gender: { man: 0, woman: 1}

    has_one_attached :image

    has_many :relationships
    has_many :followings, through: :relationships, source: :follow
    has_many :reverse_of_relationships, class_name: 'Relationship', foreign_key: 'follow_id'
    has_many :followers, through: :reverse_of_relationships, source: :user

    def follow(other_user)
      unless self == other_user
        self.relationships.find_or_create_by(follow_id: other_user.id)
      end
    end
  
    def unfollow(other_user)
      relationship = self.relationships.find_by(follow_id: other_user.id)
      relationship.destroy if relationship
    end
  
    def following?(other_user)
      self.followings.include?(other_user)
    end


end

relationshipビュー

<% unless current_user == @user %>
  <% if current_user.following?(@user) %>
    <%= form_for(current_user.relationships.find_by(follow_id: @user), html: { method: :delete }) do |f| %>
      <%= hidden_field_tag :follow_id, @user %>
      <%= f.submit 'Unfollow', class: 'btn btn-danger btn-block' %>
    <% end %>
  <% else %>
    <%= form_for(current_user.relationships.build) do |f| %>
      <%= f.hidden_field :follow_id, @user %>
      <%= f.submit 'Follow', class: 'btn btn-primary btn-block' %>
    <% end %>
  <% end %>
<% end %> 

topビュー

<div class="jumbotron">
  <div class="container">
    <nav class="navbar navbar-light">
      <h1>マッチングを探す</h1>
  </div>

  <div class="top-authentication text-center">
  <% if user_signed_in? %>
   <div class="top-authentication__sign-in-btn m-3">
      <%= link_to "ログアウト", destroy_user_session_path, method: :delete, class: "logout" %>
    </div>
    <%= link_to "マイページ", user_path(current_user.id) %>
  <% else %>
    <div class="top-authentication__sign-up-btn m-3">
      <%= link_to "アカウントを作成する", new_user_registration_path, class: "sign-up" %>
    </div>
    <div class="top-authentication__sign-in-btn m-3">
      <%= link_to "ログイン", new_user_session_path, class: "login" %>
    </div>
  <% end %>

   <% @users.each do |user|%>
        <%= link_to user_path (user.id) do %>
         <div class='user-img-content'>
           <%= image_tag user.image, class: "user-img" %>
         </div>
        <% end %>
        <%= render "relationships/follow_button.html.erb" %>

   <% end %>

データベース

class CreateRelationships < ActiveRecord::Migration[6.0]
  def change
    create_table :relationships do |t|
      t.references :user, foreign_key: true
      t.references :follow, foreign_key: { to_table: :users }

      t.timestamps
      t.index [:user_id, :follow_id], unique: true
    end
  end
end

Answer 1

先ほどの参考URLで解決されたと報告されている方はcreateの直下でuserに代入をされているんですが、
この違いがどういう風に影響するのかちょっとわかりません。
followingに代入するときの@userって値取れてるのかな、と。
p '@user'とかで値が取れているのか確認できますでしょうか？

def create
    user = User.find(params[:relationship][:follow_id])

バックアップを取られた上で、ゆとりがあればご一考くださいませ。

以下蛇足です。
ランキングでRubyのタグを付けておられる方を中心に回答依頼を出されると、
僕みたいなノイズに掛からなくて良いのかなぁと思ったりもします。

Comments

[kawakun-----]

何から何までありがとうございます！！
色々試してみます！！