Railsモデルのaffords HABTMの動作に 'インクルード'位置があるのはなぜですか？

Question

非常に奇妙なレールの動作をデバッグするのに1時間を費やしました。 考える：

アプリ/モデル/ user.rb

class User < ApplicationRecord 
    ... 
    has_many :images 
    has_many :videos 
    ... 
    has_many :tags 
    ... 
end 

アプリ/モデル/ image.rb

class Image < ApplicationRecord 
    ... 
    belongs_to :user 
    ... 
    has_and_belongs_to_many :tags 
    ... 
    include TagsFunctions 
    ... 
end 

アプリ/モデル/ video.rb

class Video < ApplicationRecord 
    ... 
    include TagsFunctions 
    ... 
    belongs_to :user 
    ... 
    has_and_belongs_to_many :tags 
    ... 
end 

アプリ/モデル/tag.rb

class Tag < ApplicationRecord 
    belongs_to :user 

    validates :text, uniqueness: {scope: :user}, presence: true 

    before_create :set_code 

    def set_code 
    return if self[:code].present? 

    loop do 
     self[:code] = [*'A'..'Z'].sample(8).join 
     break if Tag.find_by(code: self[:code]).nil? 
    end 
    end 
end 

アプリ/モデル/懸念/ tags_functions.rb

module TagsFunctions 
    extend ActiveSupport::Concern 

    # hack for new models 
    included do 
    attr_accessor :tags_after_creation 

    after_create -> { self.tags_string = tags_after_creation if tags_after_creation.present? } 
    end 

    def tags_string 
    tags.pluck(:text).join(',') 
    end 

    def tags_string=(value) 
    unless user 
     @tags_after_creation = value 
     return 
    end 

    @tags_after_creation = '' 
    self.tags = [] 
    value.to_s.split(',').map(&:strip).each do |tag_text| 
     tag = user.tags.find_or_create_by(text: tag_text) 
     self.tags << tag        
    end 
    end 
end 

私は、このようなコードを実行した場合：

user = User.first 
tags_string = 'test' 
image = user.images.create(tags_string: tags_string) 
video = user.videos.create(tags_string: tags_string) 

それはで1つのアイテムを与えますimage.tagsですが、重複する項目は2つあります。video.tags

しかし、我々はこのようコードを変更した場合：

user = User.first 
tags_string = 'test' 
image = Image.create(user: user, tags_string: tags_string) 
video = Video.create(user: user, tags_string: tags_string) 

すべてが正常に動作し、画像や映像

、さらに1個のタグ1個のタグ... 我々は以下のinclude TagsFunctionsを移動した場合has_and_belongs_to_many :tags、video.rbファイルでは、両方のコード例が正常に動作します。

私はレールをかなりよく知っていると思っていましたが、この動作は私にとっては本当に不明です。
Railsのバージョン：5.1.1

Answer 1

あなたは何のタグで、ここで起こっているようだが、ユーザーだけでなく、実際のビデオ記録のために作成されているこれらの2行 tag = user.tags.find_or_create_by(text: tag_text) self.tags << tag

をチェックすることをお勧めしますように思えます。しかし、タグモデルに何かがあるかどうかを見ずに知るのは難しいです。 tags関数内のタグ関連付けをユーザーと関連付けることを避けることは良いことかもしれません。

Answer 2

私は何を持っていることは、ドメインが最初の場所でより良いモデル化することができるので、XとYの問題だと思う：

# rails g model tag name:string:uniq 
class Tag < ApplicationRecord 
    has_many :taggings 
    has_many :tagged_items, through: :taggings, source: :resource 
    has_many :videos, through: :taggings, source: :resource, source_type: 'Video' 
    has_many :images, through: :taggings, source: :resource, source_type: 'Image' 
end 

# rails g model tagging tag:belongs_to tagger:belongs_to resource:belongs_to:polymorphic 
class Tagging < ApplicationRecord 
    belongs_to :tag 
    belongs_to :tagger, class_name: 'User' 
    belongs_to :resource, polymorpic: true 
end 

class User < ApplicationRecord 
    has_many :taggings, foreign_key: 'tagger_id' 
    has_many :tagged_items, through: :taggings, source: :resource 
    has_many :tagged_videos, through: :taggings, source: :resource, source_type: 'Video' 
    has_many :tagged_images, through: :taggings, source: :resource, source_type: 'Image' 
end 

module Taggable 
    extend ActiveSupport::Concern 

    included do 
    has_many :taggings, as: :resource 
    has_many :tags, through: :taggings 
    end 

    # example 
    # @video.tag!('#amazeballs', '#cooking', tagger: current_user) 
    def tag!(*names, tagger:) 
    names.each do |name| 
     tag = Tag.find_or_create_by(name: name) 
     taggnings.create(tag: tag, tagger: tagger) 
    end 
    end 
end 

これは私たちが代わりに文字列を比較するの検索に使用することができ、正規化タグテーブルを作成し、値。 has_and_belongs_to_manyは、2つのテーブルを結合するだけで、結合テーブルを直接クエリする必要がない場合（つまり、HABTMを使用すると90％の時間が間違っている）の最も単純なケースで本当に便利です。

HABTMを「ハックする」ためにコールバックを使用すると、それが悪化します。

さまざまなタグ付け可能なクラスを設定するときに、メタプログラミングを使用して複製をカットすることができます。

class Video < ApplicationRecord 
    include Taggable 
end 

class Image< ApplicationRecord 
    include Taggable 
end