掲載開始期間になったら表示、閲覧できるようにしたいです。

Question

・問題点
掲載開始期間を設定できるようにしたのですが、その開始期間が始まったらトップページなどに表示、閲覧(show)できるようにしたいのですが、コントローラーにその機能を書くのか、viewページにそのような縛りで表示させるかなど、迷って困っています。
現状、コントローラーにdef plan_showを作成しましたので拝見して頂き、アドバイスを頂けたらと思っております、

・plans_controller

class PlansController < ApplicationController
  before_action :set_plan, only: [:show, :edit, :update, :destroy ]
  before_action :require_user_logged_in
  before_action :current_user, only: [:destroy, :edit, :new]
  before_action :require_user_logged_in, only: [:edit]
  before_action :plan_destroy, only: [:show, :index]
  
  def index
    @pagy, @plans = pagy(Plan.order(id: :desc))
  end
  
  def show
    
  end
  
  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 'plans/new'
    end
  end

  def destroy
    @plan = Plan.find_by(id: params[:id])
    @plan.destroy
    flash[:success] = '掲載を中止しました。'
    redirect_to("/")
  end


  private
  def set_plan
      @plan = Plan.find(params[:id])
  end
    
  def plan_params
    params.require(:plan).permit(:title, :photo, :detail, :start_on, :end_on)
  end
  
  def plan_destroy
    Plan.where('end_on < ?', Date.today).destroy_all
  end
  
  def plan_show
    Plan.where('start_on < ?', Date.today)
  end
  
  def correct_user
    @plan = current_user.plans.find_by(id: params[:id])
    unless @plan
      redirect_to root_url
    end
  end
  
end

・view/toppages/index

class ToppagesController < ApplicationController
  def index
    @all_plan = Plan.all
    @user = User.all
    @plans = Plan.order(created_at: :desc).limit(5)
    
    if logged_in?
       @my_plans = current_user.plans.order(created_at: :desc).limit(5)
    end
  end
end

・toppages_controller

class ToppagesController < ApplicationController
  def index
    @all_plan = Plan.all
    @user = User.all
    @plans = Plan.order(created_at: :desc).limit(5)
    
    if logged_in?
       @my_plans = current_user.plans.order(created_at: :desc).limit(5)
    end
  end
end

・views/plans/show.html.erb

<% if logged_in? %>
<% if current_user == @plan.user %>
<div class='plan-content'>
    <h4><%= @plan.title  %></h4>
    <p><%= @plan.start_on %>～<%= @plan.end_on %></p>
    <p><%= image_tag @plan.photo.url, :alt => '企画書イメージ図' if @plan.photo? %></p>
    <p><%= @plan.detail %></p>
    <div class="plan-content-btn">
      <%= link_to "削除", @plan, method: :delete, data: { confirm: "削除しますか？" } %>
      <%= link_to '戻る', '/' %>
    </div>
</div>
<% else %>
<div class='plan-content'>
    <h4><%= @plan.title  %></h4>
    <p><%= @plan.start_on %>～<%= @plan.end_on %></p>
    <p><%= image_tag @plan.photo.url, :alt => '企画書イメージ図' if @plan.photo? %></p>
    <p><%= @plan.detail %></p>
    <div class="plan-content-btn">
      <%= link_to '応募する', '/' %>
      <%= link_to '戻る', '/' %>
    </div>
</div>
<% end %>
<% else %>
<div class='plan-content'>
    <h4><%= @plan.title  %></h4>
    <p><%= @plan.start_on %>～<%= @plan.end_on %></p>
    <p><%= image_tag @plan.photo.url, :alt => '企画書イメージ図' if @plan.photo? %></p>
    <p><%= @plan.detail %></p>
    <div class="plan-content-btn">
      <%= link_to '応募する', login_path   %>
      <%= link_to '戻る', '/' %>
    </div>
</div>

<% end %>

情報が物足りない箇所がありましたら、また回答から聞いていただければ幸いです。
お手数をお掛けしてしまい、大変申し訳ありませんが、拝見して頂いて、アドバイスを頂けたらなと思っております。

Answer 1

取得時に期間内のデータだけ取得すれば良いと思います

https://techracho.bpsinc.jp/hachi8833/2021_08_26/24876
https://qiita.com/umanoda/items/b7e2eeee52712f289f6e

Comments

[退会済みユーザー]

こちらはどうでしょうか？
def plan_new
@plans = Plan.where('start_on < ?', Date.today)
end

[m.ts10806]

あの「どうでしょうか」と言われましても、実際どうだったのでしょう。
私としては「期間の予定を」と記載しているのですが、提示のコードだと終了日がないような。

[m.ts10806]

＞低評価された方

理由をコメント願います。
無言低評価があまりにも多くて大変迷惑しています。
回答に対する低評価の理由や改善の指摘をいただければ、内容吟味して回答の修正は検討します。

[退会済みユーザー]

すみません。
設定した掲載開始日に関係なく、掲載されていたので、間違いだとわかりました。
また終了日は終了日で、終了日を超えると、自動で削除される機能を記述していたのですが、そちらに掲載開始日を迎えると表示される機能を追加するのは可能でしょうか？
ちなみにそのコードは下記です。
def plan_destroy
Plan.where('end_on < ?', Date.today).destroy_all
end
こちらの機能に関しましては正常に作動しました。

また、低評価に関しましてはm.ts10806様に大変、ご迷惑をかけてしまい、申し訳ありません。
こちらも、もっと質問の仕方などに気を付けるべきでした。
どこかご不明な点がありましたら、解決したいので足りない情報をこちらからも可能な限り
記載させていただきたと思っております。
今回は誠に申し訳ありません。

[m.ts10806]

解決されたようで何よりです。

低評価については質問者さんではなく、
低評価をした人に向けて発したコメントです。質問者さんは無関係なのでお気になさらず。
質問に答えていないわけでもふざけているわけでもないのに
理由もなく低評価をつける不純な人がいるので（おそらく私怨）
そういう人に向けて理由をコメントするように催促しています。
嫌がらせ目的ならそれはそれで運営対処依頼しないといけませんしね。