Question

我的表格包含行中的产品和列中的供应商。用户可以输入每个产品/供应商组合的订单数量。

enter image description here

对于每个供应商，创建1个订单。每个订单都包含OrderItems。 OrderItems是为用户输入数量的每个字段（产品/供应商组合）创建的。

有很多手写代码可以处理提交的表单。

有没有办法干掉下面的任何代码？或者是否有更好的方法？

我查看了嵌套表格Railscast，但我没有看到如何在这里使用accepts_nested_attributes_for，因为输入是二维的（供应商和产品的组合）。

class Product < ActiveRecord::Base    
  has_many :order_items
end

class Supplier < ActiveRecord::Base    
  has_many :orders
end

# groups all OrderItems for 1 Supplier
class Order < ActiveRecord::Base    
  has_many :order_items
  belongs_to :supplier

  def self.create_orders_and_order_items(orders)            
    orders.each do |supplier_id, order_items|
      if order_has_order_item?(order_items)
        order = create!(
          :total => 0,
          :supplier_id => supplier_id,
          :order_group_id => order_group.id
        )        
        OrderItem.create_order_items(order, order_items)                
        # update attributes
        order.update(:total => order.order_items.sum(:total))        
      end      
    end
  end

  def self.order_has_order_item?(order_items)
    sum = 0
    order_items.each do |product_id, quantity|
      sum += quantity.to_i
    end
    sum > 0 ? true : false    
  end    
end

# 1 OrderItem per product / supplier combination
class OrderItem < ActiveRecord::Base    
  belongs_to :order
  belongs_to :supplier
  belongs_to :product

  def self.create_order_items(order, order_items)        
    order_items.each do |product_id, quantity|
      if quantity.to_i > 0                        
        order_item = create!(
          :quantity => quantity,
          :product_id => product_id,
          :order_id => order.id,
        )

        # update after creating, because methods called below are only available once object has been instantiated
        order_item.udpate(:total => order_item.calculate_total)
      end
    end
  end  
end

class OrdersController < ApplicationController
  def create
    Order.create_orders_and_order_items(params[:orders])
    respond_to do |format|
      format.html { redirect_to :action => "index" }
    end
  end
end

# view: _form.html.erb
<table>
  <tr>
    <td>Name</td>
    <% @suppliers.each do |supplier| %>
      <td COLSPAN=2><%= supplier.name %></td>
    <% end %>
  </tr>
  <% @products.each do |product| %>
    <tr>
      <td><%= product.name %></td>
      <td><%#= some price %></td>
      <td><%= f.text_field "#{supplier.id}[#{product.id}]", :value => "" %></td>
    </tr>
</table>  

<%= f.submit %>

# params (from memory)
{"orders" => {
  "4" => # supplier_id, 1 Order for each Supplier
    { "13" => "2" } # product_id => quantity, = 1 OrderItem
  }
}

Answer 1

回答

或者是否有更好的方法？

我强烈建议使用表单对象，它基本上提取了在单独的类中使用多个AR模型处理表单的复杂性。

检查7 patterns to decompose fat AR models第3节。

Answer 2

在这种情况下，循环浏览所有产品似乎不应该做什么。我会在订单模型中添加accepts_nested_attributes_for。

这会让您删除create_orders_and_order_items和create_order_items。

另外，我会在OrderItem模型中使用validation for quantity。

我不确定您的代码是否像这样：您进入您的页面并查看所有产品的列表，然后您可以输入每种产品的数量。

而不是这个，你应该有可添加/可移动的条目，并且在每个条目中允许用户选择产品和数量。这是通过accepts_nested_attributes_for完成的，nested_form可以促进您的方式。

处理复杂嵌套表单的最佳方法

2 个答案: