wrong number of bind variables (1 for 2)

Question

###実現したいこと
if文をwhere条件に書き換えたいです。

###コード

<% if @current_user.following?(post.user) || @current_user.id == post.user_id %>

自分がフォローしているユーザーのポスト or 自分のポスト　というif文です。
そこで、where条件を使い、Controller内に書き換えようとしましたが、エラーでうまくいきません。

Post.joins(:user).where("@current_user.following?(users) or users.id =?",@current_user.id)

wrong number of bind variables (1 for 2) in: @current_user.following?(users) or users.id =?

調べると、?の個数と引数の個数が合っていない見たいでした。
そこで、このように書き換えて見ましたが、別のエラーが出てしまっています。

Post.joins(:user).where("@current_user.following?= ? or users.id =?",(users) ,@current_user.id)

undefined local variable or method `users' for #<PostsController:0x00007fa8b720cc60>

お分かりの方、ぜひ助言を宜しくお願いします。

追記

class User < ApplicationRecord
  has_many :posts, dependent: :delete_all
  has_many :active_friendships, class_name:"Friendship", foreign_key: "follower_id", dependent: :destroy
  has_many :passive_friendships, class_name:"Friendship", foreign_key: "followed_id", dependent: :destroy
  has_many :following, through: :active_friendships, source: :followed
  has_many :followers, through: :passive_friendships, source: :follower

  def follow(user)
    active_friendships.create(followed_id: user.id)
  end
  def unfollow(user)
    active_friendships.find_by(followed_id: user.id).destroy
  end
  def following?(user)
    following.include?(user)
  end
  def posts
    return Post.where(user_id: self.id)
  end
  def create_notification_follow!(current_user)
    temp = Notification.where(["visitor_id = ? and visited_id = ? and action = ? ",current_user.id, id, 'follow'])
    if temp.blank?
      notification = current_user.active_notifications.new(
        visited_id: id,
        action: 'follow'
      )
      notification.save if notification.valid?
    end
  end
end

class Post < ApplicationRecord
  belongs_to :user
  
end

Comments

[peperoncino000]

関連するモデルを追記してください

[tomtom1]

追記に加えました

Answer 1

contrllerに移そうというのは正しいです。
where("@current_user.following?(users) or users.id =?",@current_user.id)
railsでのwhere文の書き方はいろいろありますが、この形式の場合は””の中がそのまま SQL文としてRDBに送られます。ですので、？の数が合ったとしても検索文としては成り立ちません。

Post.joins(:user,:active_friendships).
  where("user_id = ? or active_friendships.user_id = ?" , @current_user.id, @current_user.id)

辺りかな。

Comments

[tomtom1]

ご回答ありがとうございます！
controllerに移そうというのは正しいです。
→ありがとうございます。参考までにぜひお聞きできればと思うのですが、
現状、Controllerに移した方が、スッキリするし、Post.allで取得した後にifで表示するか否か分けると全てを取得して、速度が落ちてしまうので、Controller時点で仕分けをするという意図でしたが、この考えが正しいという意味でしょうか？または他の理由の正しさがあるのでしょうか？

→お教え頂いた条件で下記のエラーが出てしまいました。
Can't join 'Post' to association named 'active_friendships'; perhaps you misspelled it?
もしかしたら、active_friendshipsはUserにbelongしているが、Postには関係性がないのでエラーと出てしまったのでしょうか..> <？