Rails最佳实践：模型中的关联对象创建

Question

我大张旗鼓地有3个表：用户（电子邮件，密码），联系人（名称，电话），关系（user_id，contact_id，级别）。

当用户创建新联系人时，我希望他与之关联。 该协会的“友谊”级别为1到3。

我使用一种形式在我的contacts＃create控制器中输入级别。

现在，我有这个很棒

  def create
    @contact = Contact.new(params[:contact])
    if @contact.save
      #@relation = Relation.new(:user_id => current_user.id, :contact_id => @contact.id, :level => params[:relation])
      #@relation.save
      redirect_to root_url, :notice => "ok!"
    else
      render "new"
    end
  end

我当时正在考虑将关系创建移到我的联系人模型中，以执行以下操作：

  after_create { Relation.create(user_id: current_user.id, contact_id: self.id, level: params[:relation]) }

当然，这是行不通的，但是您知道了。 可以在模型中做到这一点还是可以像现在一样保留它

干杯

Answer 1

像这样吗 基本上，只需创建关系并联系与current_user关联的所有对象。

current_user.relations.create（contact：Contact.new（params [：contact]），级别：params [：relation]）

不要将其移至after_create。 如果有的话，可以在某个地方创建一个接受用户，联系人和关系的功能。

Answer 2

我宁愿将它保留在控制器中。 为了进行测试（以及可能用于其他目的），您可能不希望将用户和联系人紧密地绑在一起。 我看到的方式是，控制器是将创建逻辑联系在一起的地方，而模型中的after_create之类的方法更多地是设置某些参数，而不是创建新的关联，而在将来您可能并不需要。

tl; dr-在控制器中放置类似这样的内容会使两个模型紧密结合在一起。

Answer 3

contact.rb

has_one :relation
accepts_nested_attributes_for :relation

关系

belongs_to :contact
belongs_to :user

像这样的意见

= for_form @contact do |f|
  = f.fields_for :relation do |r|
    = r.text_field :level
  = f.submit 'create'

控制器新动作

  @contact = Contact.new
  @contact.build_relation # create new relation object for the contact

控制器创建动作

  @contact = Contact.new(params[:contact])
  @contact.relation.user = current_user
  @contact.save