Rails 子モデルから重複しないデータを取得したい

Question

Rails ActiveRecordのデータ取得

モデルに従属している子モデルの重複なしリストを取得したいです。
現在は以下のようにしています。

    @posts = Post.includes([:tag]).where(user_id: current_user.id)
    @tags = @posts.map { |p| p.tag }.uniq

@postsも@detailsもview側で利用します。

モデルの設定

class Post < ApplicationRecord
  belongs_to :tag
end

class Tag < ApplicationRecord
  has_one :post
end

1つのPostに付けるTagは1つだけ。
そもそもmodelの関連付けがおかしいような・・・。

発生している問題

1. Rails初心者のため、データ取得のベストプラクティスがわからず。。お教えください。

補足情報（FW/ツールのバージョンなど）

Rails5 MySQL

Comments

[siruku6]

確認なのですが、現在どんな問題やエラーが発生しているのでしょうか？

それとも、問題は無いけれどももっときれいにソースコードを書きたいということなのでしょうか？？

[mongolia]

こっちです!
>それとも、問題は無いけれどももっときれいにソースコードを書きたいということなのでしょうか？？
周りでRuby/Railsを使う人がおらず、どのようなコード書くのが良いかわからないんです。

Answer 1

何をもってuniq {に|と} するかなのですが、

Post と Tag が 1:1 なのですから、
@tags = @posts.map { |p| p.tag }　でuniq無しでもuniqです
この場合なら @tags をわざわざ作らなくても、@tags.each{|tag| の代わりに@posts.each{|post| として　post.tag で扱えばよいと思う。

Post1のtag1とpist2のtag2が（Tagとしては別物ですが）その属性の値が一緒の場合もありえるわけで、その場合もひとつだけにしたい、というのだと
@tags = @posts.map { |p| p.tag.name }.uniq
にするわけですが、すると @tags がtagの集まりではなくなるからそれで良いものかどうか。