0

postモデルのレコードを削除した際に、それに紐づくcommentモデルのレコードを削除したいのですが、以下のエラーになります。
現状コメントがなければ正常に削除ができる状態です。

わかる方がいましたらよろしくお願いします。

エラーメッセージ:

Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails (`anipho_development`.`comments`, CONSTRAINT `fk_rails_2fd19c0db7` FOREIGN KEY (`post_id`) REFERENCES `posts` (`id`))

現状のコードはこんな感じです

マイグレーションファイル

class CreateComments < ActiveRecord::Migration[6.0]
  def change
    create_table :comments do |t|
      t.references :user, foreign_key: true
      t.references :post, foreign_key: true
      t.string     :content, null: false
      t.timestamps
    end
  end
end

comment.rb

class Comment < ApplicationRecord
  belongs_to :user
  belongs_to :post
  validates :content, presence: true
end

post.rb

class Post < ApplicationRecord
  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to_active_hash :category
  belongs_to :user
  has_one_attached :image
  has_many :comments, dependent: :destroy

  with_options presence: true do
   validates :image
   validates :title
   validates :category_id, numericality: { other_than: 1 , message: "は--以外から選んでください"} 
  end
end

user.rb

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 :posts
  has_many :comments
  
  PASSWORD_REGEX = /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]+\z/i.freeze
  validates :nickname, presence: true
  validates :password, format: { with: PASSWORD_REGEX}
  
end

2 件の回答 2

1

見た感じちゃんと動きそうなコードですね。。
postモデルの削除はどうやって行っていますか?

レコードの削除はいくつかの方法がありますが、RailsではおおむねActiveRecordの4つのメソッドのうちどれかを使うことになります。
delete, delete_all, destroy, destroy_allです。
4つもあって混乱すると思いますが、そのエラーを出さないように削除するためには、

@post = Post.find(params[:id]
@post.destory

という感じで消す必要があります。

上に揚げた4つのメソッド、大別して delete系とdestory系の二つに分かれているんですが、

  • delete系...実行するとActiveRecordが該当するDELETE文のSQLを生成し、そのまま実行する。ヴァリデーションやコールバックなど、ActiveRecordが更新系処理に付随して実行する処理は全てスキップされる。
  • destroy系...上と違い、ActiveRecordが更新系処理に付随して実行する処理を実行後に削除処理が実行される。

という違いがあります。

Postモデルに定義されている

 has_many :comments, dependent: :destroy

これ。この依存関係の定義はコールバックに相当するので、delete系メソッドで削除した場合は実行されません。
なのでメソッド実行時にすぐ下記のSQLが発行されます。

DELETE FROM posts where id=xxxx

CreateComments のマイグレーションファイルにFKの定義がある通り、

t.references :post, foreign_key: true

commentsテーブルのレコードは、常に親であるpostsテーブルのレコードのidを必ず持つように設定されています。
コメントが残った状態でpostsレコードを消してしまうとコメントが孤児状態で浮いてしまうので、それを防止するためのFKのエラーが発生している訳ですね。
このエラーを発生させないように、先にcommentsのレコードを消す必要がある訳ですが、そのためには dependent: :destroy の定義だけでなく、destroyメソッドを使用する必要があります。

-2

私は ruby-on-rails については知識がありません。
下記のサイトの回答を参考にしてみたらどうでしょうか?

stack overflow
Why are my add_foreign_key on_delete: :cascade designations not being transferred to the schema
https://stackoverflow.com/questions/49439591/why-are-my-add-foreign-key-on-delete-cascade-designations-not-being-transferre

この回答に外部キー制約(foreign_key)で on_delete: :cascade の指定をしています。

  t.references :document, foreign_key: { on_delete: :cascade }

なお、外部キー制約とカスケード操作については下記のドキュメントの参照アクションの項目に記述があります。

MySQLの CREATE TABLE 文ですが FOREIGN KEY句の ON DELETE ON UPDATE等のサンプルがあります。

13.1.20.5 FOREIGN KEY の制約
https://dev.mysql.com/doc/refman/8.0/ja/create-table-foreign-keys.html

ご参考になれば幸いです。

追記)
上記の stack overflow の回答は正しく動作しなかったようです。

下記の記事も参考にしてください。

Qiita
Railsアプリケーションにおける外部キー制約エラーの解決法
https://qiita.com/KM9973/items/39db3300770ccb71600d

Railsで外部キー制約の参照制約の書き方
https://qiita.com/na-777/items/cf17f8c79cc76af2898d

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。