Rails いいねの取り消しボタンが機能せず再度いいねされてしまう

Question

実現したいこと

• いいねの取り消し機能を動作するようにする

いいねの取り消しができず、逆に再度いいねをしたことになってしまい、自分では解決方法が見いだせないので解決策やヒントをご教授いただきたいです。

前提

Rails7でアルバイト求人サービスで作っており、こちらの記事を参考に「掲載されたお仕事(job)へのいいね(like)」機能を実装しています。
ログイン中のユーザーが、そのお仕事をいいねしていなければ「いいね！ボタン」を表示し、既にいいねしていれば「いいねを取り消すボタン」を表示し、いいねを取り消せるようにしたいです。
userテーブル、jobテーブル、likeテーブルがあり、どのuserがどのjobをいいねしたかがlikeテーブルに保存されるようになっています。

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

app/views/jobs/show.html.erbに「いいねを取り消すボタン」を配置しましたが、押下しても再度いいねしたことになってしまいます(エラーは出ていません)。
例えば、以下のように現在ユーザー5(user_id:5)でログインしており、お仕事7(job_id:7)のお仕事をいいねしている状態で、

+----+---------+--------+----------------------------+----------------------------+
| id | user_id | job_id | created_at                 | updated_at                 |
+----+---------+--------+----------------------------+----------------------------+
|  9 |       5 |      7 | 2023-05-08 04:23:19.496488 | 2023-05-08 04:23:19.496488 |

お仕事7(job_id:7)のいいねを取り消そうと思い「いいね済ボタン」を押すと以下のように再度いいねした情報が再度likeテーブルに保存されてしまいます。

+----+---------+--------+----------------------------+----------------------------+
| id | user_id | job_id | created_at                 | updated_at                 |
+----+---------+--------+----------------------------+----------------------------+
|  9 |       5 |      7 | 2023-05-08 04:23:19.496488 | 2023-05-08 04:23:19.496488 |
| 10 |       5 |      7 | 2023-05-08 04:52:42.012388 | 2023-05-08 04:52:42.012388 |

該当のソースコード

class ApplicationController < ActionController::Base
  helper_method :current_user
  before_action :set_current_user

  def set_current_user
    @current_user = User.find_by(id: session[:user_id])
  end
  
  private

  def current_user
    # @current_user ||= User.find_by(id: session[:user_id]) if session[:user_id]
  end
end

class Like < ApplicationRecord
  belongs_to :user
  belongs_to :job
end

class User < ApplicationRecord
  validates :name, presence: true, length: { maximum: 50 }
  validates :email, presence: true, length: { maximum: 100 }, uniqueness: true
  # validates :password_digest, presence: true
  # has_secure_password
  # validates :password, length: { minimum: 6 }
  has_many :likes, dependent: :destroy
end

class Job < ApplicationRecord
  validates :salary, presence: true, length: { maximum: 5 }
  has_many :likes, dependent: :destroy

  def liked_by?(user)
    likes.exists?(user_id: user.id)
  end
end

Rails.application.routes.draw do
  # resources :jobs
  root "home#top"
  resources :users, only: [:new, :create, :edit, :destroy, :update]
  resources :jobs, only: [:new, :create, :edit, :index, :show, :destroy]  do
    resource :likes, only: [:create, :destroy]
  end
  # post "users/create" => "users#create", as: "create_users"
  # get "users/:id/edit" => "users#edit", as:"edit_users"
  # post "users/:id/update" => "users#update", as:"update_users"
  get "signup", to: "users#new", as: "new_users"
  resources :guest_sessions, only: [:create]
  # post "guest_login", to: "guest_sessions#guest_login"
  # get "guest_login", to: "guest_sessions#guest_login"
  post "logout" => "sessions#destroy"
  # get "logout" => "sessions#destroy"
  get "login", to: "sessions#new", as: "new_sessions"
  post  "login", to: "sessions#create", as: "create_sessions"
  delete "logout", to: "sessions#destroy", as: "destroy_sessions"
  # get "users/:id/destroy", to: "users#destroy"
  # delete "users/:id/destroy", to: "users#destroy", as: "destroy_users"
end

class LikesController < ApplicationController
  # before_action :authenticate_user

  def create
    # @like = Like.new(user_id: @current_user.id, job_id: params[:job_id])
    # @like = Like.new(like_params)
    # @like.save
    job = Job.find(params[:job_id])
    @current_user.likes.create(job_id: job.id)
    redirect_to jobs_path
  end

  def destroy
    # @like = Like.find_by(like_params)
    # @like.destroy
    job = Job.find(params[:job_id])
    like = @current_user.likes.find_by(job_id: job.id)
    like.destroy
    redirect_to jobs_path
  end

  private
  def like_params
    { user_id: @current_user.id, job_id: params[:job_id] }
  end
end

<%= @job.salary %>
<%= link_to("お仕事を編集する", edit_job_path) %>
<%= button_to "お仕事を削除する", job_path(@job), method: :delete %>

<% if @job.liked_by?(@current_user) %>
    <%= button_to "いいねを取り消す", job_likes_path(@job), data: { turbo_method: :delete } %>
<% else %>
    <%= button_to "いいね！", job_likes_path(@job), data: { turbo_method: :post } %>
<% end %>

試したこと

参考記事とルーティングを見比べましたが、
参考記事では
resources :posts, only: [:new, :create, :index, :show, :destroy] doとしており、
私は
resources :jobs, only: [:new, :create, :edit, :index, :show, :destroy] doというようにeditアクションを加えているだけなので特に問題につながるところではないと思います。
その他では問題の発生箇所となるようなところはわかりませんでした。

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

Ruby3.1.3
Rails7

Answer 1

likeテーブルにカラム(例：good_flg)を追加するのはいかがでしょうか。

likeテーブルにカラム(列)を追加します。
このカラムはいいね押した状態 or いいね押していない状態 のどちらなのか管理するものです。
一般的には0:押していない、1:押している 状態と定義することが多いです。
私なら、int型のgood_flg(NOT NULL、DEFAULT 0)でカラム追加します。

このテーブルの利用方法の例として、
1.ユーザ(user_id:5)がお仕事(job_id:7)に初めていいねする
→likeテーブルにINSERTでgood_flgが1のレコードが登録される
2.ユーザ(user_id:5)がお仕事(job_id:7)のいいねを取り消す
→likeテーブルのUPDATE(Edit)で該当のレコードのgood_flgを0にする
3.ユーザ(user_id:5)がお仕事(job_id:7)に再びいいねする
→likeテーブルのUPDATE(Edit)で該当のレコードのgood_flgを1にする

また、ユーザ(user_id:5)がお仕事(job_id:7)にいいねしているか否かの状態はlikeテーブルを参照できます。
したがって、View(HTML)側にていいねの状態によってアイコンや動作を変更したい際は上記を利用可能です。

ちなみにですが、お仕事ごとで合計いいね数をカウントしたい際は、WHEREでjob_id=[該当のお仕事のjob_id] AND good_flg=1で取得可能です。

Comments

[hono112]

Mashさん、ご回答いただきありがとうございます。
こちらの質問ですが、別の機能を実装したときに同じようなコードを書いたところうまく行ったので、とりあえず解決済みにしたいと思います。