Question

我有用户和付款模式。用户有很多付款，付款有一个用户。为了确定用户是否有有效的付款，我使用此：

User.rb

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

我想定义一个范围，让用户返回有效付款，我已经想出了这个：

User.rb

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

编写这样一个范围的合适，更有效的方法是什么？

Answer 1

正如GSP所说的ActiveRecord或对Mongoid使用以下查询：

User.where( :id.in => Payment.where(:made_at <= DateTime.now, :valid_until >= DateTime.now).distinct(:user_id) )

Answer 2

您可以执行以下操作：

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)（而不是包含）可以使范围返回非uniq用户列表
您可以将日期时间传递到此范围，而不是立即使用：User.with_active_payments(DateTime.current - 1.months)

Answer 3

您希望对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

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

3 个答案: