掲載期間が過ぎたら自動削除されるようにしたいです。

Question

・質問したい点
掲載したものに掲載期間を登録できるようにしました。
そこから掲載期間が過ぎたら自動で削除されるようにしたいのですが、controllerからifを使って自動削除させることはできるのでしょうか？

・plan.rb (モデル)

class Plan < ApplicationRecord
  belongs_to :user
  validates :title, presence: true, length: { maximum: 50 }
  validates :detail, presence: true, length: { maximum: 5000 }
  validates :start_on, presence: true
  validates :end_on, presence: true
end

・plans_controller(コントローラー)

class PlansController < ApplicationController
  before_action :require_user_logged_in
  
  def new
    @plan = Plan.new
    if @plan.user == current_user
      redirect_to '/'
    end
  end
  
  def create
    @plan = current_user.plans.build(plan_params)
    if @plan.save
      flash[:success] = '企画掲載を設定しました'
      redirect_to root_url
    else
      @pagy, @plans = pagy(current_user.plans.order(id: :desc))
      flash.now[:danger] = '企画掲載に失敗しました'
      render 'toppages/index'
    end
  end

  def destroy
    @plan.destroy
    flash[:success] = '掲載を削除しました。'
    redirect_back(fallback_location: root_path)
  end

  private

  def plan_params
    params
      .require(:plan)
      .permit(:content)
  end
  
  def correct_user
    @plan = current_user.plans.find_by(id: params[:id])
    unless @plan
      redirect_to root_url
    end
  end
  
end

↑このコントローラーに下記のコードを追加してみようと思うのですが、この記述は意味がないですか？

def self.plan_destroy
    @date = Date.today
    @plan = Plan.all
    if plan.end_on < date
      @plan.destroy
    end
  end

この上記の記述で掲載期間後に自動削除されますでしょうか。
ご意見を頂けると幸いです。

Answer 1

該当するものを探し出して削除する という機能は１行です。

問題はそれをいつどの様に召喚するか、です。

一番簡単なのは kaniclub さんの回答のようになにかのアクションにくっつける。
この方法は実装は簡単ですが、indexが呼ばれるたびに実行される。無駄が増えます。
この方法を採る場合は 「関係画面(Plan参照)に行くには必ず通るが、頻度がすくない」actionを選ぶのが良いでしょう。
例えば、ユーザのlogin

バッチといって定期的に裏で動かす仕組みで起動する 方法もあります。忙しいサイトではこちらが良いかも。
興味あれば whenever で検索。

なお、削除すべきPlanが沢山ある場合、eachで回すとDBの負荷が上がります。
Plan.where('end_on < ?', Date.today).destroy_all
が良いでしょう。

Comments

[退会済みユーザー]

回答していただきありがとうございます！

Answer 2

似たような機能を付けたことがあるので、当時はこのような感じで実装したと思います。
参考になれば幸いです

実際には動かしていないので、エラーチェックなどは出来ていません。

・plans_controller(コントローラー)

before_action :plan_destroy, only: [:show, :index]
.
.
.
.

def plan_destroy
    Plan.where('end_on < ?', Date.today).each do |plan|
        plan.destroy
    end
end

Comments

[退会済みユーザー]

回答していただき、ありがとうございます！