Railsでauto_incrementのidを挿入する際、特定のテーブルに存在しないidを挿入したい

Question

以前の質問。
https://teratail.com/questions/157540
https://teratail.com/questions/162431

現状以下のようなテーブルがあります。

article
id: integer
title: string

deleted_article
id: integer
article_id: integer
title: string

articleを削除後、deleted_articleに情報を退避する感じです。

ここで上記一つ目の質問に書いてあるid再利用がarticleテーブルで起きてしまうため以下のような状況になってしまいます。

articleテーブル

id	title
3	title3
4	title4
5	title5
1	title6

deleted_articleテーブル

id	article_id	title
1	1	title1
2	2	title2

ここでdeleted_articleのarticle_idはuniqueなので、title: "title6"の記事が削除できなくなっています。

ここでアプリケーション（Rails）側でdeleted_articlesのarticle_idに存在しないidのみをarticlesに挿入したいのですが、以下のような実装はどう思いますか？
(rollbackが起きるとauto incrementがプラスされるようなのでそれを利用しています。）

loop do
  begin
    Article.transaction do
      article = Article.create!(title: ’test’)
      raise ActiveRecord::Rollback if DeletedArticle.exists?(article_id: article.id)
      break
    end
  rescue 
    next 
  end
end

また、もし他に良い案があれば教えてください。

Answer 1

そもそもなぜ ID: 1 title6 というデータが作られているのでしょうか
Database の auto_increment 任せなら ID: 6 title6 が作られ ID: 1,2 は歯抜けのままだと思うのです
一番良いのは、その変なデータが作られている場所の修正や、 もし auto_increment 設定のミスならそれを直すとかの根本解決だと思います

とはいえ誤魔化さないといけない or 誤魔化せば良い程度の話で記載された処理を改善するなら以下でしょうか
auto increment -> rollback だとID状況によっては最悪何十何百回と繰り返すことになるので、最大値を指定する感じです

loop do
  begin
    Article.transaction do
      max_id = [DeletedArticle.maximum(:article_id), Article.maximum(:id)].max
      Article.create!(id: max_id + 1, title: ’test’)
    end
  rescue 
    next 
  end
end

Comments

[kuuritar]

ありがとうございます。
確かに最大値の設定は必要そうですね。
idが戻る理由ですが、先頭に置いたリンクに詳細を書いていますが、mysqlの仕様で、DB再起動や最適化等により起こるようです。

Answer 2

論理削除をしたいだけなら、articleモデルにstatusカラムをつけておいて、削除操作した場合はstatusをdeletedにするなどの設計が良いのではないでしょうか？

Comments

[kuuritar]

論理削除はやりたくなく、あくまでバックアップ目的で保存したいだけです。
article自体は物理削除したいです。

[退会済みユーザー]

なるほど、kuzugrさんのサービスの全体像を把握できないので、あまり有益なアドバイスとならず申し訳ございません。ちなみにですが、なぜ論理削除ではなく上のような方法を取ろうと考えられているのですか？

[kuuritar]

>なぜ論理削除ではなく上のような方法を取ろうと考えられているのですか？

これは私が動かしているサービスではなく、私が手伝っているサービスですでに稼働中だからですね・・・
設計を変える必要があるためです。

[退会済みユーザー]

なるほど、自分の好きなようにはできないためとういうことですね。
事情も知らず、お力になれず申し訳ございまでんでした。