Ruby on Rails中关联订单商品匹配SKU时增量的最优实现方式

咱们先聊聊你给出的这段代码的优化空间哈～

switch = false 
order.order_items.each do |item| 
  if item.sku == test.sku 
    item.increment!(:quantity) 
    switch = true 
  end 
end

这段代码能实现功能，但确实算不上最优方案，主要存在两个问题：

• 效率偏低：它会把当前订单下的所有order_items都加载到内存中遍历，如果订单条目很多，内存占用和执行速度都会受到明显影响。
• 代码冗余：switch变量完全是多余的，哪怕你在找到匹配项后加break跳出循环，也不如直接从数据库层面定位目标条目来得高效。

你最初尝试的where查询方向是对的，咱们可以把它优化得更简洁高效，推荐两种实用的实现方式：

方式一：用find_by精准定位（最直观高效） #

# 直接从数据库查询匹配SKU的条目
if existing_item = order.order_items.find_by(sku: test.sku)
  existing_item.increment!(:quantity)
else
  # 如果需要新增条目，这里可以补充创建逻辑
  order.order_items.create!(sku: test.sku, quantity: 1)
end

这种方式直接让数据库帮我们找到目标条目，无需加载所有order_items，性能上提升明显，代码逻辑也清晰易懂。

方式二：用find_or_initialize_by简化完整流程 #

如果需要同时处理“存在则递增，不存在则创建”的完整逻辑，可以用这个更简洁的写法：

order.order_items.find_or_initialize_by(sku: test.sku).tap do |item|
  item.quantity = item.persisted? ? item.quantity + 1 : 1
  item.save!
end

或者结合increment!进一步简化成一行逻辑：

item = order.order_items.find_or_initialize_by(sku: test.sku)
item.persisted? ? item.increment!(:quantity) : item.update!(quantity: 1)

总结来说，最优实现的核心思路是让数据库完成查询定位工作，而不是在内存里遍历所有条目，这样既提升了性能，也让代码更简洁易维护。

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