記事削除後のCouldn't find Article with 'id'=1を解消したい

Question

前提・実現したいこと

記事を投稿したり削除できるwebサービスを制作しています。

トップページにはいいねの数によって記事を並べるランキング機能を実装しており、
管理者はこれらの記事を削除できる権限を持っています。

問題なく機能していたと思ったのですが削除機能を改めて確かめてみたところ、
下記のエラーが発生しました。

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

ActiveRecord::RecordNotFound in HomesController#index
Couldn't find Article with 'id'=1

該当のソースコード

class HomesController < ApplicationController
  # before_action :set_article, only: [:index]
  
  def index
    @manga = Article.where(category: "マンガ")
    @illust = Article.where(category: "イラスト")
    @anime = Article.where(category: "アニメ")
    
    @all_ranks = Article.find(Like.group(:article_id).order('count(article_id) desc').limit(4).pluck(:article_id))
    @new_article = Article.order(created_at: :desc).limit(5)
  end
  
  # def set_article 
  #   @article = Article.find(params[:article_id]) 
  # end
end

 <div class="lanking-article">  
   <% @all_ranks.each.with_index(1) do |article, i| %>
     <div class="ranking-card">
       <td class="text-center">
         <% case i when 1 %>
           <i class="fas fa-crown" style='color: gold;'></i>
         <% when 2 %>
           <i class="fas fa-crown" style='color: silver;'></i>
         <% when 3 %>
           <i class="fas fa-crown" style='color: orange;'></i>
         <% else %>
           <%= i %>位
         <% end %>
       </td>
              
       <p><%= link_to image_tag(article.image), article_path(article) %></p>
       <p><%= link_to truncate(article.title, length:30), article_path(article) %></P>
     </div>
   <% end %>
 </div>

試したこと

現状は削除した記事のID（今回の場合はid=1）を一覧画面で表示しようとしたときも参照
してしまっているため"Couldn't find Article" = （削除したので）記事が見つからない
となっていると考えました。

そこで上記のコントローラでコメントアウトされているset_articleメソッドを定義し、
まず全体の記事の確認してから記事を表示しようと考えたのですがうまくいかず悩んでします。

考え方の方向性としてはあっていますでしょうか？
もし最適な考え方や解決策などがあればぜひ助言を頂けると嬉しいです。

よろしくお願いいたします。

Answer 1

考え方の方向性としてはあっていますでしょうか？

まずは、記事を削除したときに、すでに付いたいいねをどうしたいかから考えないといけません。

いちばんスッキリするのは、「記事を消したときに、その記事へのいいねもまとめて削除する」ことです。

Comments

[AkiDatsugoku26]

maisumakunさん助言ありがとうございます。

「記事を消したときに、その記事へのいいねもまとめて削除する」
→こちらを頼りに記事(article)モデルに
　 has_many :likes, dependent: :destroyを追記してみました。

狙いとしては記事が消されればLikeも消されるようにしたかったのですが、
どうやらキャッシュが残っているようでエラーは解消できていません。

調べるとafter_destroyの記事がヒットしたのでもう少し粘ります。

[maisumakun]

> どうやらキャッシュが残っているようでエラーは解消できていません。

dependent: :destroyは、「これから削除する分」にだけ効果があります。すでにArticleが削除されてしまったLikeはそのままになっています（削除したいなら手動で削除してください）。

[AkiDatsugoku26]

すぐにご返信頂きありがとうございます。
一旦DBをリセットしてから記事投稿→削除→リロードしてみると問題なく表示されました！

結局after_destoryでキャッシュをクリアはできなかったのですが実務だとよく使うものなのでしょうか。
助言のおかげで解決できました。ベストアンサーにしておきますー！

Answer 2

こちらベストアンサーの助言を頼りに解決できましたので共有します。
まず結果からですが記事(article)モデルに以下のコードを追記しました。

has_many :likes, dependent: :destroy

エラー時にはdependent: :destroyが記述されていませんでした。このコードによって記事が消されたタイミングでいいね！（Like）も消すことができます。
ここまでで助言にあった「記事を消したときに、その記事へのいいねもまとめて削除する」をクリアすることができました。

しかし肝心の下記のエラーは解消できません

ActiveRecord::RecordNotFound in HomesController#index
Couldn't find Article with 'id'=1

なぜか？それは記事を削除したのはコメントにもある通り「これから削除する分」にだけ効果がある為、
既に削除してしまった記事に対しては有効ではないからです。
そしてキャッシュが残っている為このままでは消された記事（今回の場合id:1）を探し続けてしまいエラーが解消されないのだと思います。

いろいろ調べてみましたが僕の場合は結局dbをリセットしテストデータを入れなおしました。

rails db:reset

rake db:seed

これで次回以降記事を削除した場合はいいねも一緒に消されるため、
存在しない記事を探してエラーが出ることもなくなるはずです。

追記
キャッシュを削除する方法としてはafter_destroyなどいろいろヒットしましたがどれも実装することができませんでした。今回の解決策以外で効果的な方法があれば是非教えてください。