シンプルな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 

これは機能しません。グループは保存されますが、リンクテーブルに作成されたメンバーシップレコードはありません。ただし、create vs buildを使用すると動作します。それはどうやって動作するのでしょうか？

ここで最善のアプローチは何ですか？

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を実行すると[]が得られます。

これを行う最善の方法は、あなたのアプローチに多少似ています。これはとしてuser_idsを含める必要がありGroups#createに提出paramsハッシュを動作させるために、

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

しかし：：などの簡単な作成の作用を有する

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

ことがbruno077 @あなたを求めていた理由だったかもしれませビューのコードを貼り付けることで、渡されるuser_idsパラメータのアイデアを得ることができます。

したがって、新しいグループフォームに複数のユーザーを選択するフィールドが含まれている場合は、（user_idsパラメータのために）上記のような簡単な作成アクションがあります。しかし、ユーザーを選択するオプションを持たない新しいグループフォームを使用している場合は、最初のオプション（createを使用したもの）を使用する方がよいでしょう。