简单的has_many：通过关联

Question

非常简单的设置。我想确保我对ORM的理解是正确的。

class User < ActiveRecord::Base 
    has_many :memberships 
    has_many :groups, through => memberships 
end 

class Group < ActiveRecord::Base 
    has_many :memberships 
    has_many :users, through => memberships 
end 

class Membership < ActiveRecord::Base 
    belongs_to :user 
    belongs_to :group 
end 

现在，当用户创建一个组时，我希望链接表中的成员记录得到填充。它应该是一个原子（交易）。

class GroupsController < ApplicationController 
    def create 

    @group = current_user.groups.build(params[:group]) 

    if @group.save 
     flash[:notice] = "Group has been created." 
     redirect_to @group 
    else 
     flash[:alert] = "Group has not been created." 
     render :action => "new" 
    end 
    end 
end 

这不起作用。该组被保存，但没有在链接表中创建成员记录。然而使用创建vs构建工程。这是它应该如何工作？

这里最好的办法是什么？

Answer 1

此行为是设计使然。正如你所提到的，你可以做@group = current_user.groups.create(params[:group])。

或者你可以添加一个额外的语句来创建的连接模型的表中的记录为：

@group = current_user.groups.build(params[:group]) 
if @group.save 
    @group.memberships.create(:user_id => current_user) 
    # redirect and notify 

好，原因是简单地建立@group并保存它不连接表中增加一个额外的记录。

事实上，在这种情况下，@group = current_user.groups.build(params[:group])有点类似于@group = Group.new(params[:group])。不同之处在于，在前一种情况下，current_user.groups将包含@group（您可以在重定向之前尝试在Groups#create之前），但在做current_user.reload后跟current_user.groups将产生[]。

做到这一点的最好方法有点类似于你的方法。有一个简单的创建行动：

def create 
    @group = Group.new(params[:group]) 
    # if else for save and redirect 

然而，这个工作提交Groups#create params哈希表应包括user_ids为：

"group"=>{"name"=>"new group", "user_ids"=>["1", "2", "3"]}, "commit"=>"Create Group" 

可能这就是为什么@ bruno077是问你的理由粘贴你的视图的代码，以获得关于传递的user_ids参数的想法。因此，如果新的组表单包含选择多个用户的字段，那么其简单的创建操作就像上面所示（因为user_ids参数）。但是如果有一个没有选择用户选项的新组表单，那么最好使用第一个选项（一个使用create）。