如何为Cartitem模型编写替代控制器逻辑的高效关联查询方法？

现有模型关联关系 #

class Promoitem 
  belongs_to :shop
  belongs_to :article, optional: true

class Cartitem 
  belongs_to :article
  belongs_to :cart

class Cart 
  belongs_to :shop

class Article
  has_many   :cartitems
  has_many   :promoitems

原控制器冗余逻辑 #

控制器中存在一段判断有效促销并获取api_price的代码，因重复使用导致冗余，希望迁移至Cartitem模型中：

valid_promo = Promoitem.where('shop_id = ? AND date_start <= ? AND date_end >= ? AND article_id = ?', @shop.id, Date.today, Date.today, cartitem.article_id ).first
if valid_promo
  api_price = valid_promo.price_promo
else
  api_price = cartitem.article.sell_price
end

当前错误实现尝试 #

现有尝试的模型方法因关联逻辑错误无法正常工作，错误代码如下：

class Cartitem 
# scope 定义错误
  scope :price_promo, lambda { joins(:promoitems).where("date_start <= ? AND date_end >= ? AND article_id = ?", Date.today, Date.today, self.article_id) }

def valid_promo
  self.promoitems.price_promo
end

def api_price
  if self.valid_promo
    self.promoitems.price_promo
  else
    self.article.sell_price
  end 
end

考虑通过Article为Cartitem添加关联，但关联写法有误：

has_many :promoitems, through: :articles 
  has_many :cartitems, through: :articles

提问 #

如何编写与原控制器逻辑等效的正确scope？或是否有更优的实现方案？

第一步：修正关联关系 #

先给Cartitem添加正确的跨模型关联（删除多余的cartitems关联）：

class Cartitem
  belongs_to :article
  belongs_to :cart
  # 关联到当前购物项对应商品的所有促销，注意是单数article，不是复数articles
  has_many :promoitems, through: :article
end

第二步：实现等效的模型方法 #

推荐直接用实例方法实现，比scope更直观，且完全匹配原控制器逻辑：

class Cartitem
  belongs_to :article
  belongs_to :cart
  has_many :promoitems, through: :article

  # 获取当前购物项所属店铺的有效促销
  def valid_promo
    promoitems.where(
      shop_id: cart.shop_id,
      date_start: ..Date.today,
      date_end: Date.today..
    ).first
  end

  # 计算最终API价格
  def api_price
    valid_promo&.price_promo || article.sell_price
  end
end

方案细节说明 #

1. 关联修正：through: :article才是正确写法，因为Cartitem关联的是单个Article实例，而非集合
2. 店铺ID匹配：通过cart.shop_id获取当前购物项所属店铺，和原控制器的@shop.id逻辑一致
3. 日期查询优化：使用Rails内置的范围查询语法替代字符串SQL，更安全、易维护
4. 逻辑简化：用&.安全调用避免空指针，||短路判断替代if-else，代码更简洁
5. 性能优化：批量处理购物项时，可通过includes(cart: :shop, article: :promoitems)预加载关联，避免N+1查询

如果需要批量筛选带有效促销的购物项，可添加如下scope：

class Cartitem
  scope :with_valid_promo, -> {
    joins(cart: :shop, article: :promoitems)
      .where(
        promoitems: {
          date_start: ..Date.today,
          date_end: Date.today..,
          shop_id: Cart.arel_table[:shop_id]
        }
      )
  }
end

但针对单个Cartitem实例，直接使用valid_promo和api_price实例方法是更优的选择，代码更清晰且完全复用原逻辑。

内容的提问来源于stack exchange，提问作者Jerome