タグのついた投稿記事を削除したい

Question

前提

画像投稿のwebアプリをRuby on Rails6にて作成中です。
画像にはタグが付いており、削除時にエラーが出ます。

実現したいこと

投稿を削除したい

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

NoMethodError in DesignsController#destroy
undefined method `find' for PostForm:Class

該当のソースコード

def destroy
    post_form = PostForm.find(params[:id])
    if post_form.destroy
      redirect_to root_path
    end

[app > controller.rb]

class DesignsController < ApplicationController
  before_action :authenticate_user!, only:[:edit, :new, :destroy]
  before_action :move_to_index, except: [:index, :show]
  before_action :set_design, only: [:show, :edit]

  def index
    @designs = Design.includes(:user).order("created_at DESC")
  end

  def new
    @post_form = PostForm.new
  end

  def create
    @post_form = PostForm.new(design_params)
    
    if @post_form.save
      redirect_to root_path
      else
        render :new
    end
  end

  def show

  end

  def destroy
    post_form = PostForm.find(params[:id])
    if post_form.destroy
      redirect_to root_path
    end
  end

  def update
    design = Design.find(params[:id])
    if
    design.update(design_params)
      redirect_to design_path(design.id)
    else
      redirect_to request.referer
    end
  end

  def edit
    unless user_signed_in? && current_user.id == @design.user_id
      redirect_to action: :index
    end
  end

  def search
    @designs = Design.search(params[:keyword]).order("created_at DESC")
  end

  private

  def design_params
    params.require(:post_form).permit(:title, :file_name, :tag_name, :image).merge(user_id: current_user.id)
  end

  def set_design
    @design = Design.find(params[:id])
  end

  def move_to_index
    unless user_signed_in?
      redirect_to action: :index
    end
  end

end

[app > views > designs > show.erb]

<main class="main">
  <div class="inner">
    <div class="prototype__wrapper">
      <p class="prototype__hedding">
        <%= @design.title %>
      </p>
      <% if user_signed_in? && current_user.id == @design.user_id %>
      <div class="prototype__detail"><br>
          <p class="detail__title">ファイル場所</p>
          <p class="detail__message">
            <%= @design.file_name %>
          </p>
        </div>
        <div class="prototype__manage">
          <%# link_to "編集する", edit_design_path, method: :get, class: :prototype__btn %>
          <%= link_to "削除する", design_path, method: :delete, class: :prototype__btn %>
        </div>
      <% end %>

      <span class="tag_style">#<%=link_to @design.tags.first&.tag_name, search_designs_path(tag_id: tag.id)%></span>

      <div class="prototype__image">
        <%= image_tag @design.image %>
      </div>

    </div>
  </div>
</main>

[app > models > post_form.rb]

class PostForm
  include ActiveModel::Model

  attr_accessor(
    :title, :file_name, :image,
    :id, :user_id,
    :tag_name
   )

   with_options presence: true do
    validates :title
    validates :file_name
    validates :image
  end


  ActiveRecord::Base.transaction do

    def save
      design = Design.create!(title: title, file_name: file_name, image: image, user_id: user_id)
      tag = Tag.where(tag_name: tag_name).first_or_initialize
      tag.save!
      PostTagRelation.create!(design_id: design.id, tag_id: tag.id)
    end



    return true

    rescue false
  end
end

試したこと

def destroyをpost_formに書いたり、＠をつけたり
def destroy
design = Design.find(params[:id])
if design.destroy
redirect_to root_path
end
end
に書き換えてみたり、

find(params[:id])　を　PostForm.(design_params)　に変えてみても
param is missing or the value is empty: post_form　のエラーが出るだけでした。

Answer 1

PostForm クラスには find メソッドも destroy メソッドも存在しないのに、その存在しないメソッドを呼び出そうとしているため、エラーが発生しています。

「試したこと」の

def destroy
  design = Design.find(params[:id])
  if design.destroy
    redirect_to root_path
  end
end

というコードが大正解なので、それで良いはずですが、動きませんか？
改めて実行してみて、エラーが出るならメッセージを教えてください。

--

追伸:
PostForm のトランザクションの書き方がおかしいので、ついでにそっちも直した方が良いと思います。

  # ↓ こう書くのが正しいです。
  def save
    ActiveRecord::Base.transaction do
      design = Design.create!(title: title, file_name: file_name, image: image, user_id: user_id)
      tag = Tag.where(tag_name: tag_name).first_or_initialize
      tag.save!
      PostTagRelation.create!(design_id: design.id, tag_id: tag.id)
    end

    return true
  rescue
    return false
  end

--

追記:

エラー原因の解説

以下のエラーは「外部キー制約」に失敗しているために表示されます。

ActiveRecord::InvalidForeignKey in DesignsController#destroy
Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails

例えば、「デザインA」に「Tag: Ruby」と「Tag: Java」の2つが紐づいている場合、DBでは以下のようなデータ構造になります。

[designs]
| id | title |
| 11 | デザインA |

[post_tag_relations]
| id | design_id | tag_id |
| 1 | 　　　11 | 　　21 |
| 2 | 　　　11 | 　　22 |

[tags]
| id | tag_name |
| 21 | Ruby |
| 22 | Java |

ここで単純に「デザインAだけ」を消すと、 post_tag_relations テーブルに「存在しない design_id 」が発生してしまいます。

[post_tag_relations]
| id | design_id | tag_id |
| 1 | 　　⚠️11 | 　　21 | ← 「存在しない ID を指し示すのはおかしくね？」とMySQL君が気をきかせる
| 2 | 　　⚠️11 | 　　22 |

なので、そうならないようにエラーを発生させて、データの整合性を保とうとするわけです。
それが、エラーメッセージ「Cannot delete or update a parent row: a foreign key constraint fails」の意味です。

修正方法

単純に designs だけを消すのではなく、関連する post_tag_relations も一緒に消す必要があります。
コードの修正はすごく簡単で、モデル側に dependent: :destroy のオプションを付けるだけです。

  class Design < ApplicationRecord
    belongs_to :user
-   has_many :post_tag_relations
+   has_many :post_tag_relations, dependent: :destroy
    has_many :tags, through: :post_tag_relations
    has_one_attached :image

    ・・・略・・・

Comments

[tamtamtime]

ご回答ありがとうございます！！

def destroy
design = Design.find(params[:id])
if design.destroy
redirect_to root_path
end
end

ですと以下のエラーが出ます。

ActiveRecord::InvalidForeignKey in DesignsController#destroy

Mysql2::Error: Cannot delete or update a parent row: a foreign key constraint fails (`abestdesign_development`.`post_tag_relations`, CONSTRAINT `fk_rails_c3aa21b013` FOREIGN KEY (`design_id`) REFERENCES `designs` (`id`))
Extracted source (around line #30):

28 def destroy
29 design = Design.find(params[:id])
30 if design.destroy
31 redirect_to root_path
32 end
33 end

IDが変だという事まではわかるのですが、記載が間違っているのかと右往左往してしまいました。

[tamtamtime]

モデルに「dependent: :destroy」を追記で削除できるようになりました。
難しく考え過ぎたかもしれません。

[shinoharat]

先ほど回答を更新したのですが、既に自己解決されてたみたいですね。
「dependent: :destroy」で大正解です。