20

ご挨拶、

メンバーシップに関する追加情報（具体的には、ユーザーが会社の管理者であるかどうか、ブール値を介して）を含むモデルを介して相互に属する必要があるアプリケーションがCompaniesあります。コードの単純なバージョン：UsersCompanyMembershipadmin

class CompanyMembership < ActiveRecord::Base
  belongs_to :company
  belongs_to :user
end

class Company < ActiveRecord::Base
  has_many :company_memberships
  has_many :users, :through => :company_memberships
end

class User < ActiveRecord::Base
  has_many :company_memberships
  has_many :companies, :through => :company_memberships
end

もちろん、これにより、などを介して会社のすべてのメンバーを簡単に取得できますcompany.users.all。ただし、その会社の管理者である会社のすべてのユーザーのリストを取得しようとしています（また、ユーザーが特定の会社の管理者であるかどうかをテストしようとしています）。私の最初の解決策は次のとおりでしたcompany.rb：

def admins
  company_memberships.where(:admin => true).collect do |membership|
    membership.user
  end
end

def is_admin?(user)
    admins.include? user
end

これは機能しますが、何かが非効率に感じられ（各メンバーシップを反復処理し、毎回SQLを実行しますか？それともRelationはそれよりも賢いですか？）、これを実行するためのより良い方法があるかどうかはわかりません（おそらくRails 3が使用するスコープまたは派手な新しいRelationオブジェクト？）

続行するための最良の方法（できればRails 3のベストプラクティスを使用する）に関するアドバイスをいただければ幸いです。

4

5 に答える 5

17

私はこれを間違った方法で行っていたと思います。実際に必要だったcompany_membershipsのではなく、の条件を指定しました（のリストではなく、のリスト）。私が探していた解決策は次のとおりです。usersUsersCompanyMemberships

users.where(:company_memberships => {:admin => true})

これにより、次のSQLが生成されます（IDが1の会社の場合）。

SELECT "users".* FROM "users"
  INNER JOIN "company_memberships"
    ON "users".id = "company_memberships".user_id
  WHERE (("company_memberships".company_id = 1))
    AND ("company_memberships"."admin" = 't')

必要かどうかはまだわかりませんが、includes()メソッドは必要に応じてSQLクエリの数を抑えるために積極的な読み込みを実行します。

Active Recordを使用すると、ロードされるすべての関連付けを事前に指定できます。これは、呼び出しのincludesメソッドを指定することで可能になります。Model.findインクルードを使用すると、Active Recordは、指定されたすべての関連付けが、可能な限り最小限のquerys.queriesを使用してロードされるようにします。RoRガイド：ActiveRecordクエリ

（これが最善/最も効果的/正しい方法ではないと考える人からの提案はまだ受け付けています。）

于 2010-09-21T14:56:59.590 に答える
8

さらにクリーンな方法は、次のような関連付けを会社モデルに追加することです。

has_many :admins, :through => :company_memberships, :class_name => :user, :conditions => {:admin => true}

正確な構文を正しく理解するには、 railsのドキュメントを詳しく調べる必要があるかもしれません。

ビューで参照する可能性のある：userに関連付けられた他のクラスがない限り、：includeは必要ありません。

于 2010-09-28T00:42:39.797 に答える
4

このようなものはどうですか：

Company.find(:id).company_memberships.where(:admin => true).joins(:user)
于 2010-09-20T20:53:06.687 に答える
3

私はこの答えに出くわし、今日ではhas_many association scopes（has_manyドキュメント）を使用するより良い方法があると信じています：

has_many :admins, -> { where(admin: true) }, through: :company_memberships, class_name: :user

has_manyアソシエーションの2番目のパラメーターは、フィルターを含むprocまたはlambdaにすることができます。

于 2019-02-14T18:38:35.607 に答える
0

これを行うためにactiverecord_where_assocgemを作成しました。

これにより、結合やインクルードは不要になります。積極的な読み込みを行うことは、義務ではなく、あなたの選択のままです。

users.where_assoc_exists(:company_memberships, admin: true, company_id: @company.id)

会社をどのようにフィルタリングするかについての質問は不明確なので、私はそれを自分で追加しました。これがないと、ユーザーが複数の会社に所属している可能性がある場合、ある会社の管理者であるということは、別の会社の管理者として扱われることを意味する可能性があります。

このgemはRails3では機能しませんが、9年後、Rails4.1以降がサポートされています。

ここに紹介と例があります。詳細については、ドキュメントをご覧ください。

于 2020-02-25T13:31:36.280 に答える