Блог

У меня есть следующая настройка ассоциаций моделей, когда потребитель может выбрать продукт или вариант. Я просто хотел спросить, верен ли мой подход?

 class Favorite < ApplicationRecord
  belongs_to :consumer
  belongs_to :favorited, polymorphic: true
  belongs_to :product, optional: true
  belongs_to :variant, optional: true
end

class Consumer < ApplicationRecord
  has_many :favorites
  has_many :favorite_products, through: :favorites, source: :favorited, source_type: 'Product'
  has_many :favorite_variants, through: :favorites, source: :favorited, source_type: 'Variant'
end

class Product < ApplicationRecord
  has_many :favorites, dependent: :destroy
end

class Variant < ApplicationRecord
  has_many :favorites, dependent: :destroy
end

class CreateFavorites < ActiveRecord::Migration[6.0]
  def change
    create_table :favorites do |t|
      t.references :consumer, index: true
      t.references :favorited, polymorphic: true, index: true
      t.integer :product_id
      t.integer :variant_id

      t.timestamps
    end
  end
end
 

Непонятно, почему у вас есть как полиморфное favorited , так и отдельное отношение product_id/variant_id . Обычно можно использовать один из этих подходов. Полиморфные ассоциации тоже могут быть однонаправленными — has_many :favorites, as: :favorited, dependent: :destroy

Если вы используете отдельные столбцы для каждого отношения — индексы включены product_id , а variant_id также могут быть полезны для предотвращения полного сканирования таблицы при удалении продукта / варианта (включено dependent: :destroy , также, если вы не ожидаете обратных вызовов / вложенных отношений там — delete_all быстрее).