中間テーブルにデータの保存ができない

Question

前提・実現したいこと

推しメン機能を付けたくて中間テーブルを使ってデータを保存したい

発生している問題・エラーメッセージ

中間テーブルにデータの保存ができない

該当のソースコード

members/index

<div class="member-selects">
  <div class="select">
      <% @members.each do |member| %>
        <tr>
          <%= member.name %>
          <td><%= link_to "推しメンに登録する", add_member_path(member), method: :post %></td>
        </tr>
      <% end %>
  </div>
</div>

class MembersController < ApplicationController
  def index 
    @members = Member.all 
  end
end

class FavoritesController < ApplicationController
  def create
    @user_id = current_user.id
    @member_id = Member.find(params[:id]).id
    @favorite = Favorite.new(member_id: @member_id, user_id: @user_id)
    if @favorite.save
      redirect_to member_index_path
    else
      redirect_to posts_path
    end
    # @favorite = current_user.likes.create(member_id: params[:member_id])
    # redirect_to users_path
  end

  def destroy
  end

  def edit
  end

  def update
  end

end

class Favorite < ApplicationRecord
  belongs_to :user
  belongs_to :member
  validates_uniqueness_of :member_id, :user_id
end

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
         
  has_many :favorites
  has_many :favorited_member, through: :favorites, source: :member

  has_many :comments, dependent: :destroy
  
  has_many :likes, dependent: :destroy
  has_many :liked_comments, through: :likes, source: :comment

  def already_liked?(comment)
    self.likes.exists?(comment_id: comment.id)
  end

  def already_favorited?(member)
    self.likes.exisits?(member_id: member.id)
  end
end

class Member < ApplicationRecord
  has_many :member_comments
  has_many :comments, through: :member_comments
  
  has_many :favorites
  has_many :favorited_users, through: :favorites, source: :user
end

Rails.application.routes.draw do
  devise_for :users

  resource :sessions, only: [:new, :create, :destroy]
  resources :users
  resources :members do
    member do
      post "add", to: "favorites#create"
    end
  end

  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  
  resources :posts, only: [:index]
  resources :comments, only: [:index, :new, :create, :show, :destory, :edit] do
    resources :likes, only: [:create, :destroy]
  end
  

  root to: 'posts#index'
  
end

試したこと

ターミナルで見るとuser_idがnilだったのでsessionをcurrent_user.idに変えてみたのですが
変えると
User Load (0.4ms) SELECT users.* FROM users WHERE users.id = 4 ORDER BY users.id ASC LIMIT 1
が出てきてmember_idが表示されませんでした。

補足情報（FW/ツールのバージョンなど）

http://labyrinth-of-wisdom.hatenadiary.com/entry/2016/03/07/073000
この記事を参考にしながらコードを書きました。

Answer 1

@member_id = Member.find(params[:id]).id
ここがおかしいです。
logで送られてくるパラメータを確認してみて下さい、id というのが無いと思います。
idを送るには
add_member_path(id: member.id)
としないと、多分、、、

なお
@member_id = Member.find(params[:id]).id は　
@member_id = params[:id] でよいのでは？

Comments

[nasuk47]

add_member_path(id: member.id)
に変更して
@member_id = Member.find(params[:id]).idと
@member_id = params[:id]
両方試してみたのですが変化ありませんでした。

[winterboum]

log見せて下さい

[nasuk47]

Started GET "/posts" for ::1 at 2019-11-27 22:00:27 +0900
Processing by PostsController#index as HTML
User Load (0.3ms) SELECT `users`.* FROM `users` WHERE `users`.`id` IS NULL LIMIT 1
↳ app/controllers/application_controller.rb:10
Rendering posts/index.html.erb within layouts/application
User Load (0.4ms) SELECT `users`.* FROM `users` WHERE `users`.`id` = 4 ORDER BY `users`.`id` ASC LIMIT 1
↳ app/views/posts/index.html.erb:8
Rendered posts/index.html.erb within layouts/application (3.0ms)
Completed 200 OK in 65ms (Views: 58.8ms | ActiveRecord: 0.6ms)
これであっていますか？

[winterboum]

それは index を呼ぶときですね。「中間テーブルに値をしまう」時のlogを。

[nasuk47]

Processing by FavoritesController#create as HTML
Parameters: {"authenticity_token"=>"aJgvylCMZR+j4WmOXT+Zre75tEDgbM3EUDZQJy6P4hIiBUp0jCBfkFuOB0OCskDrd6nKBCFmSfSfksV683o1Aw==", "id"=>"6"}
User Load (0.6ms) SELECT `users`.* FROM `users` WHERE `users`.`id` IS NULL LIMIT 1
↳ app/controllers/application_controller.rb:10
User Load (0.6ms) SELECT `users`.* FROM `users` WHERE `users`.`id` = 4 ORDER BY `users`.`id` ASC LIMIT 1
↳ app/controllers/favorites_controller.rb:3
Member Load (0.4ms) SELECT `members`.* FROM `members` WHERE `members`.`id` = 6 LIMIT 1
↳ app/controllers/favorites_controller.rb:4
(0.2ms) BEGIN
↳ app/controllers/favorites_controller.rb:6
User Load (0.3ms) SELECT `users`.* FROM `users` WHERE `users`.`id` = 4 LIMIT 1
↳ app/controllers/favorites_controller.rb:6
Member Load (0.3ms) SELECT `members`.* FROM `members` WHERE `members`.`id` = 6 LIMIT 1
↳ app/controllers/favorites_controller.rb:6
Favorite Exists (0.3ms) SELECT 1 AS one FROM `favorites` WHERE `favorites`.`member_id` = 6 LIMIT 1
↳ app/controllers/favorites_controller.rb:6
Favorite Exists (0.4ms) SELECT 1 AS one FROM `favorites` WHERE `favorites`.`user_id` = 4 LIMIT 1
↳ app/controllers/favorites_controller.rb:6
(0.3ms) ROLLBACK
↳ app/controllers/favorites_controller.rb:6
Redirected to http://localhost:3000/posts
Completed 302 Found in 52ms (ActiveRecord: 6.6ms)
これを見るとデータが入っていると思うのですが
データベースを確認するとデータは入っていませんでした。

[winterboum]

いえ、失敗しています。ROLLBACK掛かってますよね。
失敗の原因が判るように、viewにエラーメッセージを表示するようにするのが普通ですが、renderでなくredirect_to使ってるから情報が蒸発してますね。

validates_uniqueness_of :member_id, :user_id　
が原因です。　これですと　member_idがユニーク、user_idもユニークになります。
組み合わせでユニークになるようにして下さい

[nasuk47]

validates_uniqueness_of :member_id, scope: :user_id　
に直すと無事データ保存できました！ありがとうございます。