我有一个用户和一个支付模型。用户有多个付款,而一个付款有一个用户。为了弄清楚用户是否有有效的付款,我使用这个:
用户.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
编写这样一个范围的合适、更有效的方法是什么?
您可以执行以下操作:
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)
(而不是includes)可以使作用域返回非uniq用户列表 - 您可以将日期时间传递到此作用域,而不是现在使用:
User.with_active_payments(DateTime.current - 1.months)
正如GSP对ActiveRecord所说,或者对Mongoid使用以下查询:
User.where( :id.in => Payment.where(:made_at <= DateTime.now, :valid_until >= DateTime.now).distinct(:user_id) )
您想要对payments
表使用内部联接。这可能有效:
User.joins(:payments).where('payments.made_at <= ? and payments.valid_until >= ?', DateTime.now, DateTime.now)
作为一个范围,它可能看起来像这样:
class User < ActiveRecord::Base
scope :active_users, -> {
joins(:payments).where('payments.made_at <= ? and payments.valid_until >= ?', DateTime.now, DateTime.now)
}
end