Rails控制器中根据商品ID查找生成含商品对象与HEX色值的新数组

现有存储Product Id和Color HEX Code的数组数据源：

• 数据源变量：

@campaign.selectedproducts

• 数据样例（样例中第三个元素存在引号遗漏的格式笔误，实际应为字符串类型）：

["2,333333","1,333333","4,444444"] 

原有实现代码无法正常运行：

@selectedgifts = @campaign.selectedproducts.collect [{|i| Product.find(i) }, |i| i.split(',').last]

目标输出为每个元素包含对应Product对象、匹配HEX色值的数组，格式参考：

["Product Object, HEX code", "Product Object, HEX code"]

原有代码存在两处语法和逻辑错误：

• collect（Ruby中map为同名别名）的块语法错误，块不需要包裹在方括号[]内
• 未对每个元素做拆分处理，直接将id,hex格式的完整字符串传入Product.find，会触发ID匹配错误

基础修正版本（直接满足字符串拼接输出要求）：

@selectedgifts = @campaign.selectedproducts.map do |entry|
  # 按逗号拆分单条数据，限制拆分长度为2，避免HEX值本身带逗号时拆分异常
  product_id, hex_color = entry.to_s.split(',', 2)
  # 拼接为目标格式的字符串
  "#{Product.find(product_id.to_i)}, #{hex_color}"
end

实际业务使用更推荐返回二元数组结构，避免后续使用时还要拆分字符串取对象和色值：

@selectedgifts = @campaign.selectedproducts.map do |entry|
  product_id, hex_color = entry.to_s.split(',', 2)
  [Product.find(product_id.to_i), hex_color]
end

性能优化提示：上述基础版本存在N+1查询问题，选中商品数量较多时循环单条查询数据库性能很差，推荐使用批量查询方案：

# 第一步：批量拆分所有条目，提取商品ID和对应色值
parsed_entries = @campaign.selectedproducts.map { |e| e.to_s.split(',', 2) }
all_product_ids = parsed_entries.map { |pid, _| pid.to_i }
# 第二步：一次性查询所有关联商品，转为ID索引的哈希表
product_index = Product.where(id: all_product_ids).index_by(&:id)
# 第三步：组装最终结果
@selectedgifts = parsed_entries.map do |pid, hex|
  [product_index[pid.to_i], hex]
end

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