配列の中のハッシュのuniq処理

Question

RailsでTwitter風のSNSアプリを作っているのですが、

1.フォローしているアカウントのツイート
2.自分のツイート
3.フォローしているアカウントがリツイートしたツイート

をタイムラインに表示させるために一度それぞれでActiveRecodeRelationで取得したツイートをハッシュ化し、大元の1つの配列に入れているのですが、

1と3でツイートが重複する事があり、例えば自分がフォローしている二人がそれぞれのツイートをリツイートし合った場合に1にも3にも該当のツイートが格納される事になり、同じツイートが2回3回...と格納されてしまいます。

図で表すと

大元の配列[
-ハッシュ{
--id:1 , body:〇〇}
-ハッシュ{
--id:1 , body:〇〇 retweet:true(リツイートで流れたツイートか否かを判別するキー)}
-ハッシュ{
--id:2 , body:△△}
-ハッシュ{
--id:3 , body:□□}]

簡単に表すとこのような状況なのですが、これをそのままArray.uniqメソッドを使うと、
ハッシュの中の重複しているidの部分だけが削除されて、残りのbodyその他の部分は残ったままです。(ハッシュ自体が消されるわけではない)

理想の挙動としては、idが重複しているハッシュがある場合、retweetではない純粋なツイートのみを残すような書き方をしたいのですが、
いい方法が見つかりません。同じような経験をされている方いましたらご教授願いたいです。

現在のコード

		@user = current_user
		@users = @user.followings
		@following_user_id = []
		@users.each do |following|
			@following_user_id.push(following.id)
		end
		@following_user_id.push(current_user.id)
		@retweet_ids = Retweet.where(user_id: @following_user_id)
		@tweet_array = []
		@tweet_hash = Hash.new{[]}
		@retweet_ids.each do |retweet_id|
			@tweet_hash = Hash.new{[]}
			@tweet_hash["id"]=retweet_id.tweet.id
			@tweet_hash["body"]=retweet_id.tweet.body
			@tweet_hash["user_id"]=retweet_id.tweet.user_id
			@tweet_hash["status"]= 1
			@tweet_hash["status_by"]= retweet_id.user_id
			@tweet_hash["score"]= retweet_id.tweet.score
			@tweet_hash["created_at"]= retweet_id.tweet.created_at
			@tweet_hash["updated_at"]= retweet_id.tweet.updated_at
			@tweet_array.push(@tweet_hash)
		end
		
		@follow_tweets = Tweet.where(user_id: @following_user_id)

		@follow_tweets.each do |follow_tweet|
			@tweet_hash = Hash.new{[]}
			@tweet_hash["id"]=follow_tweet.id
			@tweet_hash["body"]=follow_tweet.body
			@tweet_hash["user_id"]=follow_tweet.user_id
			@tweet_hash["status"]= 0
			@tweet_hash["score"]= follow_tweet.score
			@tweet_hash["created_at"]= follow_tweet.created_at
			@tweet_hash["updated_at"]= follow_tweet.updated_at
			@tweet_array.push(@tweet_hash)
		end
		@tweets = @tweet_array.sort_by! {|h| h["updated_at"]}.reverse

Answer 1

retweetでないものが必ず含まれるのか、含まれない可能性があるのか、で変わります。
必ず含まれるなら
@tweet_array.reject{|tweet_hash| tweet_hash["body"].is_retweet?}

含まれない可能性があるなら
group_by_id = @tweet_array.group_by{|tweet_hash| tweet_hash["id"] }
としますと

{ id1 => [tweet_hash,tweet_hash,tweet_hash],
  id2 => [tweet_hash],
  id3 => [tweet_hash,tweet_hash]
}

の様になりますから、
group_by_id.values.map{|tweet_hashs| この中から一つ選ぶ }
とします

tweet_hash["body"].is_retweet? やこの中から一つ選ぶ部分は見分け方を元に工夫してください

Comments

[ryoya1122]

ありがとうございます。後者のコードで実装しました。

group_by_id.values.map{|a| a[0]} でとりあえずは実装しました。この中から一つ選ぶ方法は優先順位をもう少し細かく決めて後ほど細分化したいと思います。

これは質問とは直接関係ないのですが、この後配列をソートするつもりなのですが、
@tweets.sort_by! { |a| a[:created_at] }
@tweets.sort! { |x, y| x[:created_at] <=> y[:created_at] }
色々この辺りを試しているのですが、どう書いてもデフォルトの順番から全くソートされず困っています。
自分で考えた原因は[:created_at]のキーからバリューを呼び出せていないかと考えたのですが、カラムやドキュメントなどを見ると、同じやり方で配列の中に組み込まれたハッシュをソートしている記事があったので、原因が突き止められず困っています。
tweet_hashの中には確実に"created_at"のキーはあります。

が、書いてる途中に気づいたのですが、ハッシュがシンボルではなく文字列のキーで設定されているので、呼び出せてないのかもしれません。
キーをシンボルにしてもう一度試してみます。とりあえずご回答ありがとうございました。とても助かりました。

[ryoya1122]

シンボルに直したらうまくソートする事ができました。
文字列をキーにするよりやはりシンボルで管理する方がrubyとして綺麗ですし、メリットが多い事を自覚しました。
重ね重ねありがとうございます。

[winterboum]

ところで　@following_user_id　ですが
@following_user_id = @user.followings.map(&:id)
でとれません？

[ryoya1122]

確かにここeachで回すの凄い無駄な気がしてました。mapの理解がまだ不十分で、arrayをeachで回す処理をメソッド化したものだと思ってます。
複雑なメソッドが苦手で逃げているのですが、できる事の幅が広いメソッドなので、もっと勉強しようと思いました！！
該当の箇所はcurrent_userのidも込みで
@following_user_id = @user.followings.map(&:id).push(current_user.id)
のように修正させていただきました！多分コレで問題ない...？と思います。

なにからなにまでご指摘ありがたいです！！