フォロー中のユーザーの全ツイートを新しい順に表示

Question

ご回答いただけるとありがたいです。よろしくお願いします！

実現したいこと

フォロー中のユーザーの全ツイートを新しい順にトップページに表示したいです。

現状

フォロー中のユーザーの全ツイートをトップページに表示させることはできていますが、順番がユーザー登録の順番です。
例えば
１、User1がtweet1を投稿
２、User2がtweet2を投稿
３、User1がtweet3を投稿

1から3の順で投稿していても、表示される順は上から
２、User2がtweet2を投稿
３、User1がtweet3を投稿
１、User1がtweet1を投稿
になります。

該当のソースコード

#relationship
class Relationship < ApplicationRecord
  belongs_to :user
  belongs_to :follow, class_name: 'User'

  validates :user_id, presence: true
  validates :follow_id, presence: true
end

#tweet
class Tweet < ApplicationRecord
  belongs_to :group
  belongs_to :user
  has_one_attached :image

  validates :content, presence: true, unless: :was_attached?

  def was_attached?
    self.image.attached?
  end
end

#user
class User < ApplicationRecord
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  validates :nickname, presence: true

  has_many :group_users
  has_many :groups, through: :group_users
  has_many :tweets
  has_many :owned_groups, class_name: "Group"

  has_many :relationships
  has_many :followings, through: :relationships, source: :follow
  has_many :reverse_of_relationships, class_name: 'Relationship', foreign_key: 'follow_id'
  has_many :followers, through: :reverse_of_relationships, source: :user

  def follow(other_user)
    unless self == other_user
      self.relationships.find_or_create_by(follow_id: other_user.id)
    end
  end

  def unfollow(other_user)
    relationship = self.relationships.find_by(follow_id: other_user.id)
    relationship.destroy if relationship
  end

  def following?(other_user)
    self.followings.include?(other_user)
  end
  
end

#relationships
class RelationshipsController < ApplicationController
  before_action :set_user, only: [:create, :destroy]

  def index
    @followings = current_user.try(:followings)
  end

  def create
    following = current_user.follow(@user)
    following.save
    redirect_to @user
  end

  def destroy
    following = current_user.unfollow(@user)
    following.destroy
    redirect_to @user
  end

  def following
    @user = User.find(params[:id])
    @followings = @user.followings 
  end

  def follower
    @user = User.find(params[:id])
    @followers = @user.followers
  end

  private

  def set_user
    @user = User.find(params[:follow_id])
  end

end

#relationships#index
(省略)
<% @followings.each do |following| %>
  <% following.tweets.order("created_at DESC").each do |tweet| %>
    <div class="tweet">
(省略)

###試したこと

#relationships
  def index
    @followings = current_user.try(:followings).order("created_at DESC")
  end

同じ結果になる　＋　ログアウトしたら以下のエラーが出る

NoMethodError in RelationshipsController#index
undefined method `order' for nil:NilClass

以上になります。ご助言いただけると幸いです。

Answer 1

tweet に user_id カラムがあり、current_user.followings で絞り込めるから

followings = current_user.followings
@tweets = Tweet.where(user: followings).order(created_at: "desc")

関連付けて取得する方法

user.rb
has_many :timeline, :through => :followings, source: "tweets"

controller
@tweets = current_user.timeline.order(created_at: "desc")

下のは何となく書いたら出来ました。どういう理屈で動くのかは説明できません。
小さいサンプルで確認しているので、間違ってたらすみません。

どちらも結果としては同じですが、実行するSQLが異なります。
データ量(や多分MySQLの設定でも)で変わってくるのでどちらが優れているとは言えません。

Comments

[shawn_709]

本当に毎回ありがとうございます！感謝しても感謝しきれません。
上の方で実装したら、いけました！ありがとうございます！
また、小さいサンプルまでわざわざ用意して確認してくださり、ありがとうございます！