2

我正在尝试使用三个主要模型构建一个配方管理员应用程序:

食谱- 特定菜肴的食谱
成分- 成分列表,通过唯一性验证
数量- 成分和食谱之间的连接表,也反映了特定食谱所需的特定成分的数量。

我正在使用嵌套表单(见下文),我使用嵌套表单(第 1部分,第 2 部分)上的出色 Railscast 构建以获取灵感。(由于这个特定模式的需要,我的表单在某些方面比教程更复杂,但我能够使其以类似的方式工作。)

然而,当我的表单被提交时,所有列出的成分都会重新创建——如果该成分已经存在于数据库中,它将无法通过唯一性验证并阻止创建配方。总阻力。

所以我的问题是:有没有办法提交此表格,以便如果存在名称与我的成分名称字段之一匹配的成分,它会引用现有成分而不是尝试创建具有相同名称的新成分?

下面的代码细节...


Recipe.rb

class Recipe < ActiveRecord::Base
  attr_accessible :name, :description, :directions, :quantities_attributes,
                  :ingredient_attributes

  has_many :quantities, dependent: :destroy
  has_many :ingredients, through: :quantities
  accepts_nested_attributes_for :quantities, allow_destroy: true

Quantity.rb

class Quantity < ActiveRecord::Base
  attr_accessible :recipe_id, :ingredient_id, :amount, :ingredient_attributes

  belongs_to :recipe
  belongs_to :ingredient
  accepts_nested_attributes_for :ingredient

并在Ingredient.rb

class Ingredient < ActiveRecord::Base
  attr_accessible :name
  validates :name, :uniqueness => { :case_sensitive => false }

  has_many :quantities
  has_many :recipes, through: :quantities

这是我的嵌套表单,显示在Recipe#new

<%= form_for @recipe do |f| %>
  <%= render 'recipe_form_errors' %>

  <%= f.label :name %><br>
  <%= f.text_field :name %><br>
  <h3>Ingredients</h3>

  <div id='ingredients'>
    <%= f.fields_for :quantities do |ff| %>
      <div class='ingredient_fields'>
        <%= ff.fields_for :ingredient_attributes do |fff| %>
          <%= fff.label :name %>
          <%= fff.text_field :name %> 
        <% end %>
        <%= ff.label :amount %>
        <%= ff.text_field :amount, size: "10" %>
        <%= ff.hidden_field :_destroy %>
        <%= link_to_function "remove", "remove_fields(this)" %><br>
      </div>
    <% end %>
    <%= link_to 'Add ingredient', "new_ingredient_button", id: 'new_ingredient' %> 
  </div><br>

  <%= f.label :description %><br>
  <%= f.text_area :description, rows: 4, columns: 100 %><br>
  <%= f.label :directions %><br>
  <%= f.text_area :directions, rows: 4, columns: 100 %><br>
  <%= f.submit %>
<% end %>

link_toand允许动态添加和删除数量/成分对,link_to_function并且改编自前面提到的 Railscast。他们可以使用一些重构,但或多或​​少地工作。


更新:根据 Leger 的要求,这是来自recipes_controller.rb. 在Recipes#new路线中,3.times { @recipe.quantities.build }为任何给定的配方设置三个空白数量/配料对;这些可以使用上面提到的“添加成分”和“删除”链接即时删除或添加。

class RecipesController < ApplicationController

  def new
    @recipe = Recipe.new
    3.times { @recipe.quantities.build }
    @quantity = Quantity.new
  end

  def create
    @recipe = Recipe.new(params[:recipe])

    if @recipe.save
      redirect_to @recipe
    else
      render :action => 'new'
    end
  end
4

1 回答 1

2

您不应该将成分匹配的逻辑放在视图中 -在将它们Recipe#create传递给模型之前创建适当的对象是有责任的。请分享控制器的相关代码

开始编码之前的几点注意事项:

  1. 我使用 Rails4@ruby2.0,但尝试编写兼容 Rails3 的代码。
  2. attr_acessible在 Rails 4 中已弃用,因此使用了强参数。如果您想升级您的应用程序,请从一开始就使用强大的参数。
  3. 建议在Ingredient不区分大小写的基础上制作小写以提供统一的外观

好的,我们开始:

删除,和中attr_accessible的字符串。Recipe.rbQuantity.rbIngredient.rb

不区分大小写,小写Ingredient.rb

class Ingredient < ActiveRecord::Base
  before_save { self.name.downcase! } # to simplify search and unified view
  validates :name, :uniqueness => { :case_sensitive => false }

  has_many :quantities
  has_many :recipes, through: :quantities
end


<div id='ingredients'>调整表格的一部分以创建/更新配方:

<%= f.fields_for :quantities do |ff| %>
  <div class='ingredient_fields'>
    <%= ff.fields_for :ingredient do |fff| %>
      <%= fff.label :name %>
      <%= fff.text_field :name, size: "10" %>
    <% end %>
    ...
  </div>
<% end %>
<%= link_to 'Add ingredient', "new_ingredient_button", id: 'new_ingredient' %> 

我们应该使用:ingredientfrom Quantitynested_attributes 和 Rails 将添加_attributes-part 同时创建params-hash 以进行进一步的质量分配。它允许在新操作和更新操作中使用相同的表单。为了使这部分正常工作,应提前定义关联。见下文调整Recipe#new

最后recipes_controller.rb

def new
  @recipe = Recipe.new
  3.times do
    @recipe.quantities.build #initialize recipe -> quantities association
    @recipe.quantities.last.build_ingredient #initialize quantities -> ingredient association
  end
end

def create
  @recipe = Recipe.new(recipe_params)    
  prepare_recipe

  if @recipe.save ... #now all saved in proper way
end

def update
  @recipe = Recipe.find(params[:id])
  @recipe.attributes = recipe_params
  prepare_recipe    

  if @recipe.save ... #now all saved in proper way
end

private 
def prepare_recipe
  @recipe.quantities.each do |quantity|
    # do case-insensitive search via 'where' and building SQL-request
    if ingredient = Ingredient.where('LOWER(name) = ?', quantity.ingredient.name.downcase).first
      quantity.ingredient_id = quantity.ingredient.id = ingredient.id
    end
  end
end

def recipe_params
  params.require(:recipe).permit(
    :name,
    :description,
    :directions,
    :quantities_attributes => [
      :id,
      :amount,
      :_destroy,
      :ingredient_attributes => [
        #:id commented bc we pick 'id' for existing ingredients manually and for new we create it
        :name
  ]])
end

prepare_recipe我们做以下事情:

  1. 查找具有给定名称的成分 ID
  2. 将 foreign_key 设置quantity.ingredient_id为 ID
  3. 设置quantity.ingredient.id为 ID(想想如果您不这样做并更改配方中的成分名称会发生​​什么)

享受!

于 2013-10-24T21:36:13.883 回答