Has_and_belongs_to_many, évitant les doublons dans la table de liaison

Question

J'ai un ensemble de modèles HABTM assez simple

class Tag < ActiveRecord::Base 
   has_and_belongs_to_many :posts
end 

class Post < ActiveRecord::Base 
   has_and_belongs_to_many :tags

   def tags= (tag_list) 
      self.tags.clear 
      tag_list.strip.split(' ').each do 
        self.tags.build(:name => tag) 
      end
   end 

Maintenant, tout fonctionne bien sauf que j'obtiens un tas de doublons dans la table des Tags.

Que dois-je faire pour éviter les doublons (basés sur le nom) dans la table des tags ?

Answer 1

En plus des suggestions ci-dessus:

1. ajouter :uniq à l'association has_and_belongs_to_many
2. ajouter un index unique sur la table de jointure

Je ferais une vérification explicite pour déterminer si la relation existe déjà. Par exemple:

post = Post.find(1)
tag = Tag.find(2)
post.tags << tag unless post.tags.include?(tag)

Answer 2

Vous pouvez passer l'option :uniq comme décrit dans la documentation. Notez également que l'option :uniq ne prévient pas la création de relations en double, elle assure seulement que les méthodes accessor/find les sélectionneront une fois.

Si vous voulez éviter des doublons dans la table d'association, vous devriez créer un index unique et gérer l'exception. De plus, la validation validates_uniqueness_of ne fonctionne pas comme prévu car vous pourriez vous retrouver dans le cas où une deuxième requête écrit dans la base de données entre le moment où la première requête vérifie les doublons et inscrit dans la base de données.

Answer 3

Dans Rails4:

class Post < ActiveRecord::Base 
  has_and_belongs_to_many :tags, -> { uniq }

(attention, le -> { uniq } doit être directement après le nom de la relation, avant les autres paramètres)

Documentation Rails

Answer 4

Définir l'option uniq :

class Tag < ActiveRecord::Base 
   has_and_belongs_to_many :posts , :uniq => true
end 

class Post < ActiveRecord::Base 
   has_and_belongs_to_many :tags , :uniq => true

Answer 5

J'ai contourné cela en créant un filtre before_save qui corrige les choses.

class Post < ActiveRecord::Base 
   has_and_belongs_to_many :tags
   before_save :fix_tags

   def tag_list= (tag_list) 
      self.tags.clear 
      tag_list.strip.split(' ').each do 
        self.tags.build(:name => tag) 
      end
   end  

    def fix_tags
      if self.tags.loaded?
        new_tags = [] 
        self.tags.each do |tag|
          if existing = Tag.find_by_name(tag.name) 
            new_tags << existing
          else 
            new_tags << tag
          end   
        end

        self.tags = new_tags 
      end
    end

end

Il pourrait être légèrement optimisé pour travailler par lots avec les tags, il pourrait également avoir besoin d'un meilleur support transactionnel.