通过数值迭代写入 ActiveRecord 列

Question

我有一个要在其中存储订单的数据库。我不想将每个订单项捆绑在一个字段中，除非有充分的理由这样做。有列item_1、item_2、item_3、item_4；我想遍历每个订单项直到完成。

我得到的错误是：

语法错误： /Users/chadsakonchick/Projects/restlessnapkin/charge.rb:47：语法 错误，意外的 tSTRING_BEG，期待 keyword_end @order.line_item = item["quantity"] 'x' item["item"] ^ /Users/chadsakonchick/Projects/restlessnapkin/charge.rb:47：语法 错误，意外的 tIDENTIFIER，期待 keyword_end @order.line_item = 项目[“数量”] 'x' 项目[“项目”] ^

/order.rb

# Write order to the database
@order = Order.new
@order.venue_id = @venue
@order.customer_id = @customer.id    

# Loop each line item
x = 1
@customer_order.each do |item|
  line_item = "item_#{x}"
  @order.line_item = item["quantity"] 'x' item["item"]
  x += 1
end

Answer 1

你的代码有两个问题：

创建订单项字符串

这会引发语法错误：

item["quantity"] 'x' item["item"]

改用字符串插值：

"#{item["quantity"]} x #{item["item"]}"

将字符串分配给属性

您的代码将字符串分配给@order.line_item，即分配给line_item 属性。 line_item 变量从未被使用过：

line_item = "item_#{x}"
@order.line_item = ...

改用ActiveRecord::AttributeMethods#[]=：

line_item = "item_#{x}"
@order[line_item] = ...

您可以使用Enumerator#with_index 摆脱临时变量x：

@customer_order.each.with_index(1) do |item, index|
  @order["item_#{index}"] = "#{item["quantity"]} x #{item["item"]}"
end

请注意，如果有超过 4 个项目，这将引发 ActiveModel::MissingAttributeError，因为没有 item_5 属性。因此，与其拥有固定数量的item_x 属性，不如创建一个单独的LineItem 模型并设置一个association：

class LineItem < ActiveRecord::Base
  belongs_to :order
end

class Order < ActiveRecord::Base
  has_many :line_items
end

这允许您分配任意数量的订单项。假设LineItem 具有属性position、quantity 和item，这样的事情会起作用：

@customer_order.each.with_index(1) do |item|
  @order.line_items.create(position: index, quantity: item['quantity'], item: item['item'])
end