为什么我的每个帖子都有相同集合名称的重复项

Question

您好，我有一个帖子模型和一个集合模型，通过集合模型连接在一起。当用户发布帖子时，他会将帖子添加到集合中，例如“音乐”。然而，当我列出所有用户的收藏时，每个帖子都有多个“音乐”条目，而不是只有 1 个。 我正在使用 @collections = @user.posts.map(&:collections).flatten 获取集合，如果我在末尾添加 .uniq 则不会出现重复项 (@collections = @user.posts.map(& ：collections）.flatten.uniq）但是有人可以解释为什么我必须这样做吗？？？多谢。

UsersController

  def show
    @user = User.find(params[:id]) rescue nil
    @posts = @user.posts.paginate(:per_page => "10",:page => params[:page])
    @title = @user.name
    @collections = @user.posts.map(&:collections).flatten
  end

views/users/show.html.erb

 <h1>Collections</h1>

  <% @collections.each do |collection| %>
    <%= link_to collection.name, user_collection_posts_path(@user, link_name(collection)) %><br />
  <% end %>

集合模型

class Collection < ActiveRecord::Base
  mount_uploader :image, CollectionUploader
  attr_accessible :name, :image, :user_id
  has_many :collectionships
  has_many :users, :through => :posts
  has_many :posts, :through => :collectionships
end

collectionship 模型

class Collectionship < ActiveRecord::Base
  belongs_to :post
  belongs_to :collection
  has_one :user, :through => :post
  attr_accessible :post_id, :collection_id
end

帖子模型

  belongs_to :user
  has_many :collectionships
  has_many :collections, :through => :collectionships

user mdoel

has_many :posts, :dependent => :destroy
has_many :collections, :through => :posts

Answer 1

你已经找到了导致这种情况的线路。以下是我对为什么您看到自己所做的事情的看法（只是该行评估的每个步骤的扩展）：

@user.posts #=>
[
    <Post1:
        id: 1492
        collections: ['history', 'spain']
    >,
    <Post2:
        id: 1912
        collections: ['history', 'shipwrecks']
    >
]

@user.posts.map(&:collections) #=>
[
    ['history', 'spain'],
    ['history', 'shipwrecks']
]

@user.posts.map(&:collections).flatten #=>
[
    'history',
    'spain',
    'history',
    'shipwrecks'
]

因此您可以看到，对于每个帖子， post.collections 返回 发布的所有集合（理应如此）。 flatten 方法并不关心是否存在重复项 - 它只关心返回单个一维数组。因此，除非您对最终产品调用 uniq ，否则这些重复项将在整个操作过程中保留下来。

我相信也有一种 ActiveRecord 方法可以避免这种情况：如果用户 has_many :collections，则 @user.collections 不应该有任何重复项。不过，可能是一个丑陋的 AR 宏，具有如此多的继承级别。

无论如何，希望有帮助！