ruby-on-rails - 如何尽可能高效地编写 Rails 模型关系范围

Question

我有一个用户和一个付款模型。用户有多个付款，一次付款有一个用户。为了确定用户是否有有效的付款，我使用以下方法:

用户.rb

def payment_active?
  payments.where("? >= made_at and ? <= valid_until", DateTime.now, DateTime.now).any?
end

我想定义一个返回活跃付款用户的范围，我想出了这个:

用户.rb

def self.active_users
  User.all.select {|u| u.payment_active?}
end

编写这样一个作用域的合适、更有效的方法是什么？

Answer 1

您可以执行以下操作:

scope :with_active_payments, ->(datetime = nil) {
  includes(:payments).where('payments.made_at >= :now AND :now <= payments.valid_until', now: datetime || DateTime.current)
}

• 使用 DateTime.current 正确使用您当前的时区。
• 使用 .joins(: payments)(而不是 include)可以使作用域返回非唯一的用户列表
• 您可以将日期时间传递到此范围，而不是立即使用:User.with_active_ payments(DateTime.current - 1.months)