我觉得这需要重构 - 有帮助吗？红宝石建模

Question

因此，假设您有

line_items

和 line_items 属于一个品牌和一个模型

一个品牌有许多模型和行项目

一个模型属于一个模型

对于简单的示例想法 LineItem.new(:make => "Apple", :model = >“Mac Book Pro”）

创建 LinteItem 时，您需要一个用于品牌和型号的文本字段框。品牌和型号不应多次存在。

所以我使用了以下实现：

before_save :find_or_create_make, :if => Proc.new {|line_item| line_item.make_title.present? }
before_save :find_or_create_model

def find_or_create_make
  make = Make.find_or_create_by_title(self.make_title)
  self.make = make
end

def find_or_create_model
  model = Model.find_or_create_by_title(self.model_title) {|u| u.make = self.make}
  self.model = model
end

但是使用此方法意味着我必须运行自定义验证而不是 #validates_presence_of :make 由于关联发生在虚拟属性

validate :require_make_or_make_title, :require_model_or_model_title

def require_make_or_make_title
  errors.add_to_base("Must enter a make") unless (self.make || self.make_title)
end

def require_model_or_model_title
  errors.add_to_base("Must enter a model") unless (self.model || self.model_title)
end

Meh 上，这开始很糟糕。现在，真正糟糕的是使用表单进行编辑。考虑到我的表单字段是部分的，我的编辑正在呈现与新表单相同的表单。这意味着 :make_title 和 :model_title 在表单上为空白。

我不太确定纠正上述问题的最佳方法是什么，这是我认为需要完全重构的最后转折点。

如果有人可以提供任何反馈，那就太好了。

谢谢！

Answer 1

我认为 line_items 不应该属于某个品牌，它们应该只属于一个型号。一个模型应该有很多行项目。一个品牌可以通过一个模型拥有许多行项目。您缺少一些显示字段的方法。

class LineItem
  belongs_to :model
  after_save :connect_model_and_make
  def model_title
     self.model.title
  end
  def model_title=(value)
     self.model = Model.find_or_create_by_title(value)
  end
  def make_title
     self.model.make.title
  end
  def make_title=(value)
     @make = Make.find_or_create_by_title(value)  
  end
  def connect_model_and_make
     self.model.make = @make
  end
end
class Model
  has_many :line_items
  belongs_to :make
end
class Make
  has_many :models
  has_many :line_items, :through => :models
end

这实际上并没有那么糟糕，只是没有超级简单的方法来做到这一点。我希望您在某个时候在这些文本字段上添加自动完成功能。