多対多の関連付け（modelの書き方が正しいか）

Question

ユーザー（出品者）がサービス（商品）を出品し、別のユーザー（購入者）がそれを購入できるというケースを想定した場合、以下の考え方で正しいでしょうか？誤りがあればご指摘頂けますと幸甚です。

###前提
・ユーザー（出品者）とサービス （商品）： １対多の関連付け
・サービス（商品）とユーザー（購入者） ： 多対多の関連付け ＊１つのサービス（商品）に複数のユーザー（購入者）が購入できる前提。
・ユーザーは出品者にも購入者にもなれる : 出品者と購入者は同じmodel

##モデル（正しいでしょうか？）
・上記前提を元にすると、必要となるmodelはUserとProductとEntry（productとそれを買うuser（購入者））を多対多の関連付けで使用する中間テーブル）の３つ

・すなわち、以下のイメージ
user (1)<-> (多)product（１） <-> （多）entry（多） <->（１） user

###各モデルのテーブル（migration file) *単純化してます
User

#User
class CreateUsers < ActiveRecord::Migration[5.0]
  def change
    create_table :users do |t|
      t.string :email, null: false  # メールアドレス
      t.string :family_name  # 姓
      t.string :given_name   # 名
      t.string :address  # 住所
      t.text :user_profile   # プロフィール
      t.string :hashed_password   # パスワード

      t.timestamps
    end
  end
end

#Product
class CreateProducts < ActiveRecord::Migration[5.0]
  def change
    create_table :products do |t|
      t.references :user   #外部キー
      t.integer :price   #価格
      t.string :title   #タイトル
      t.datetime :posted_at #商品投稿日
      t.binary :data #写真
      t.timestamps null: false
    end
  end
end

#Entry
class CreateEntries < ActiveRecord::Migration[5.0]
  def change
    create_table :entries do |t|
      t.references :product, null: false
      t.references :user, null: false
      t.timestamps
    end
  end
end

###各モデルの書き方

#Userモデル
class User < ApplicationRecord
  has_many :products, dependent: :destroy
  has_many :applied_products, through: :entries, dependent: :destroy, source: :product
  has_many :entries, dependent: :destroy
end

#Productモデル
class Product < ApplicationRecord
  belongs_to :user
  has_many :entries, dependent: :destroy
  has_many :applicants, through: :entries, source: :user
end

#Entryモデル
class Entry < ApplicationRecord
belongs_to :applied_products, class_name: "Product", foreign_key: "product_id"
belongs_to :applicants, class_name: "User", foreign_key: "user_id"
end

Comments

[IPU]

modelの紐付けの前に、どんなテーブルで作る予定かを書かないと伝わりづらいと思います。

[innjera]

情報かけており申し訳ありません。今しがた編集致しました。宜しくお願い致します。

Answer 1

同じ種類の「商品」が複数あった場合、それらの商品の集まりを"１つの物"としてとらえる考え方と、個々の商品をそれぞれ"１つの物"としてとらえる考え方がありますが、個々の商品を"１つの物"として扱う方が考えやすいです。
というのは、複数の商品をひとまとめで考えると、購入者ごとの購入数の合計が商品数を超えないように常に状態を管理する必要があるので、複雑になるからです。

個々の品物をそれぞれ"１つの物"とするなら、
・ユーザー（出品者）と商品 ： １対多の関連付け （ある品物を出品するのは一人だけ、但し一人が複数の品物を出品できる）
・商品とユーザー（購入者） ： 多対１の関連付け （ある品物を購入するのは一人だけ、但し一人が複数の品物を購入できる）
になります。

ある人は出品者にも購入者にもなれる。
しかし売買に関する役割(role)が異なりますから、出品者と購入者のモデルは別々にしたほうが素直だと思います。
そうすれば１つの商品に関した関係が、下記のように単純なものになるからです。
（購入者が居ない状態では、buyerが存在しない関係になります）

seller(出品者) - item(個別の商品) - buyer(購入者）

Comments

[innjera]

コメント有難う御座います！
初歩的な質問で恐縮ですが、「出品者と購入者のモデルは別々にしたほうが素直」という点が、良く理解できませんでした。
Aというユーザーが、出品もするし購入もする場合に、モデルを分けてしまうと、（例えばですが）ユーザー情報(氏名やパスワード等)をもつデーターベースを作成する為に、①seller modelのmigrationファイルと②buyer modelのmigrationの２つを作成（ほぼ同一のcolumnを持つmodel/data baseを2つ作る）ことにならないでしょうか？

Answer 2

・上記前提を元にすると、必要となるmodelはUserとProductとEntry（productとそれを買うuser（購入者））を多対多の関連付けで使用する中間テーブル）の３つ

やりたい事に対して、この構成の作り方は、正しいと思います。
１番シンプルですね。

ただし、テーブルを見ると、Entryのカラムが足りないですね。
このままだと、ユーザと商品の紐付けはわかりますが、ユーザの立場（出品or購入）がわかりません。
Entryテーブルに、出品or購入がわかるようなflag用のカラムを増やしてあげると、やりたい事が出来る構成になるはずです。

本題のmodelの紐付け部分ですが、

・すなわち、以下のイメージ

user (1)<-> (多)product（１） <-> （多）entry（多） <->（１） user

ここも少し違いますね。
書き方を合わせて書くと、
product（１） <-> （多）entry（多） <->（１） user
だけですね。

では、これをどうコード化するかと言うと、

# app/models/user.rb
class User < ActiveRecord::Base
  has_many :entries
  has_many :products, through: :entries
end

# app/models/entry.rb
class Entry < ActiveRecord::Base
  belongs_to :user
  belongs_to :product
end

# app/models/product.rb
class Product < ActiveRecord::Base
  has_many :entries
  has_many :users, through: :entries
end

こうですね。
以下のサイトがほとんど同じことをやっているので、参考に読んでみてください。
Rails4で多対多のリレーションをモデルに実装する

あと、余談ですが、dependent: :destroyは今回のケースでは使わない方が良いと思いますよ。
ユーザを削除したりしても、関連する商品を消されては困るはずなので。

Comments

[innjera]

種々アドバイス有難うございます！よく理解できました。

[innjera]

いただいたアドバイスを踏まえ、最終的にこうしてみました。
#User model
has_many :entries_of_adviser, :class_name => 'Entry', :foreign_key => 'adviser_id'
has_many :entries_of_applicant, :class_name => 'Entry', :foreign_key => 'applicant_id'
has_many :products_of_adviser, :through => :entries_of_adviser, :source => 'product'
has_many :products_of_applicant, :through => :entries_of_applicant, :source => 'product'

#Product モデル
has_many :advisers, :through => :entries
has_many :applicants, :through => :entries
has_many :entries

#Entryモデル
belongs_to :adviser, :class_name => 'User'
belongs_to :applicant, :class_name => 'User'
belongs_to :product