Rails has_many 通过避免重复

Question

我通过歌曲模型和艺术家模型之间的关联设置有一个 has_many。 我的代码看起来像这样

SongArtistMap 模型

class SongArtistMap < ActiveRecord::Base
 belongs_to :song
 belongs_to :artist
end

艺人模特

class Artist < ActiveRecord::Base
 has_many :song_artist_maps
 has_many :songs, :through => :song_artist_maps

 validates_presence_of :name
end

歌曲模型

class Song < ActiveRecord::Base
  has_many :song_artist_maps
  has_many :artists, :through => :song_artist_maps
  accepts_nested_attributes_for :artists
end

我有一个用户提交歌曲并输入歌曲标题和歌曲艺术家的表单。

因此，当用户提交歌曲并且我的 Artists 表还没有歌曲的艺术家时，我希望它创建该艺术家并在 SongArtistMap 中设置地图

如果用户提交的歌曲的艺术家已经在 Artists 表中，我只想创建 SongArtistMap 但不复制该艺术家。

目前，每次用户提交歌曲时，都会在我的艺术家表中创建一个新艺术家，即使相同的艺术家已经存在并且为该重复的艺术家创建了 SongArtistMap。

知道如何解决这个问题吗？我觉得 Rails 可能有一些简单的小技巧来修复这个已经内置的问题。谢谢！

Answer 1

好的，我前一阵子想通了，忘记发帖了。所以这就是我解决问题的方法。首先我意识到我不需要has_many through 关系。

我真正需要的是has_and_belongs_to_many 关系。我设置了它并为它制作了桌子。

然后在我的Artists 模型中添加了这个

def self.find_or_create_by_name(name)
  k = self.find_by_name(name)

  if k.nil?
    k = self.new(:name => name)
  end

  return k
end

在我的Song 模型中我添加了这个

before_save :get_artists
def get_artists
  self.artists.map! do |artist|
   Artist.find_or_create_by_name(artist.name)
  end
end

这正是我想要的。

Answer 2

我在其他两个经过的表模型中使用了一个方法，即使用 before_create 调用。不过，这可能会变得更整洁、更快。

before_create :ensure_only_one_instance_of_a_user_in_a_group

  private

  def ensure_only_one_instance_of_a_user_in_a_group
    user = User.find_by_id(self.user_id)
    unless user.groups.empty?
      user.groups.each do |g|
        if g.id == self.group_id
          return false
        end
      end
    end
    return true
  end

Answer 3

试试这个：

class Song < ActiveRecord::Base
  has_many :song_artist_maps
  has_many :artists, :through => :song_artist_maps
  accepts_nested_attributes_for :artists, :reject_if => :normalize_artist


  def normalize_artist(artist)
    return true if  artist['name'].blank?
    artist['id'] = Artist.find_or_create_by_name(artist['name']).id
    false # This is needed
  end
end

我们实际上是通过重载 reject_if 函数来欺骗 Rails（因为我们永远不会返回 true）。

您可以通过不区分大小写的查找来进一步优化这一点（如果您使用的是 MySQL，则不需要）

    artist['id'] = ( 
     Artist.where("LOWER(name) = ? ", artist['name'].downcase).first ||       
     Artist.create(:name => artist['name'])
    ).id