0

我在这里有一段代码，我真的可以在重构方面使用一些帮助。我需要不同的方法在 rails 的表单中添加关系数据。代码取自http://railscasts.com/episodes/75-complex-forms-part-3，我的问题是我需要同时拥有 Material 模型和 Answer 模型的方法。所以我需要两次完全相同的代码，用“答案”替换“材料”。

看来这应该通过一些动态编程来解决？但我对此完全没有经验。

这是如何解决的？

after_update :save_materials
after_update :save_answers  

def new_material_attributes=(material_attributes)
  material_attributes.each do |attributes|
    materials.build(attributes)
  end
end

def existing_material_attributes=(material_attributes)
  materials.reject(&:new_record?).each do |material|
    attributes = material_attributes[material.id.to_s]
    if attributes
      material.attributes = attributes
    else
      materials.delete(material)
    end
  end
end

def save_materials
  materials.each do |material|
    material.save(false)
  end
end
4

2 回答 2

5

您可能还想看看这个网站：

http://refactormycode.com/

于 2008-12-26T14:43:33.387 回答
1

如果我理解正确，您希望使用与 for 相同的方法answers，materials但复制最少的代码。做到这一点的方法是抽象一些私有方法，这些私有方法适用于任何一个，answers或者materials从这些模型特定的方法中用适当的模型调用它们。我在下面给出了一个示例。请注意，我没有对这些save_方法做任何事情，因为我觉得它们足够短，抽象它们并没有节省多少。

after_update :save_materials
after_update :save_answers  

// Public methods

def new_material_attributes=(material_attributes)
  self.new_with_attributes(materials, material_attributes)
end

def new_answer_attributes=(answer_attributes)
  self.new_with_attributes(answers, answer_attributes)
end

def existing_material_attributes=(material_attributes)
  self.existing_with_attributes(materials, material_attributes)
end

def existing_answer_attributes=(answer_attributes)
  self.existing_with_attributes(answers, answer_attributes)
end

def save_materials
  materials.each do |material|
    material.save(false)
  end
end

def save_answers
  answers.each do |answer|
     answer.save(false)
  end
end

// Private methods    

private
def new_with_atttributes(thing,attributes)
    attributes.each do |attribute|
       thing.build(attribute)
    end
end

def existing_with_attributes=(things, attributes)
  things.reject(&:new_record?).each do |thing|
    attrs = attributes[thing.id.to_s]
    if attrs
      thing.attributes = attrs
    else
      things.delete(thing)
    end
  end
end
于 2008-12-26T15:11:43.173 回答