コントローラーにモデルから絞込みの結果を呼び出す

Question

HomeController

@pass = Pass.where("start_date < ?", Time.zone.now).where("end_date > ?", Time.zone.now).last
    if !@pass.nil?
      @discount = @pass.discount_price
    else
      @discount = 500
    end

上記のコードの絞込みを今はcontrollerでやってるのですが、これをPassModel内で行いたいです。

@pass = Pass.where("start_date < ?", Time.zone.now).where("end_date > ?", Time.zone.now).last

この部分をModelで処理し、Controllerに渡すにはどうすればいいですか？

Answer 1

Pass モデルのクラスメソッドを定義すればいいですね。

既存のコードをそのまんま生かすなら

class Pass < ApplicationRecord
  def self.xxx
    where("start_date < ?", Time.zone.now).where("end_date > ?", Time.zone.now).last
  end
end

となります。メソッド名は思いつかなかったのでとりあえず xxx にしてあります。

コントローラーでは，

@pass = Pass.xxx

とすればいいわけです。

ただ，

where("start_date < ?", Time.zone.now).where("end_date > ?", Time.zone.now)

の部分はスコープを定義するのが見通しがよいかと思います。つまり，

class Pass < ApplicationRecord
  scope :within_period,
    ->{ where "start_date < :now AND end_date > :now", now: Time.zone.now }

  def self.xxx
    within_period.last
  end
end

みたいな。スコープ名がこれで適切かは自信がありませんが。

これで回答にはなっているかと思いますが，二つ疑問があります。

start_date や end_date は，名前からすると時刻じゃなくて date 型ですよね。それを時刻と比較して，意図どおりの検索になっているのかどうか。

また，last を取っていますが，これが意味を持つためには，where の絞り込みで得られるレコードが確実に何かの順序になっている必要があります。
しかし，order を使っていないので，レコードの追加が何かの順序になっているのでないかぎり，last には意味が無いことになります。