質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

645閲覧

if文をwhereメソッドに書き換えたい

tomtom1

総合スコア168

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2019/12/14 05:46

###やりたいこと・知りたいこと
if文をwhereメソッドに書き換えたいです。
現在Viewにif文を書いているのですが、Controller内にwhereメソッドを使って書き換えたいです。

view

1<% @posts.each do |post| %> 2 <% if @current_user.following?(post.user) || @current_user.id == post.user_id %> 3 <%= post.name %> 4<% else %><% end %>

こちらのif文では、「CurrentUserがフォローしている人のPost と CurrentUserのPost 」という定義です。つまりフォローしていない人のPostと自分以外のPostは非表示する仕様になっています。
しかし、ページネーション8件取り出した後に書いているため、表示件数が8より下回る場合があり、そのためにController内に移したく思っています。
###コード

Controller

1def index 2 @posts = Post.joins(:user).all.order(created_at: :desc).page(params[:page]).per(8) 3end

Model

1class User < ApplicationRecord 2 has_many :posts, dependent: :delete_all 3 has_many :active_friendships, class_name:"Friendship", foreign_key: "follower_id", dependent: :destroy 4 has_many :passive_friendships, class_name:"Friendship", foreign_key: "followed_id", dependent: :destroy 5 has_many :following, through: :active_friendships, source: :followed 6 has_many :followers, through: :passive_friendships, source: :follower 7 8 9 def follow(user) 10 active_friendships.create(followed_id: user.id) 11 end 12 def unfollow(user) 13 active_friendships.find_by(followed_id: user.id).destroy 14 end 15 def following?(user) 16 following.include?(user) 17 end 18end

Model

1class Post < ApplicationRecord 2 belongs_to :user 3end

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

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

一度に考えるとややこしいですが分けると楽ですね

ruby

1def index 2 users = User.where(id: current_user).or current_user.followings 3 @posts = Post.joins(:user). 4 where(user: users). 5 order(created_at: :desc).page(params[:page]).per(8) 6end

投稿2019/12/14 07:55

asm

総合スコア15147

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tomtom1

2019/12/14 09:24

ありがとうございます!そのようなやり方があるのですね。勉強になります! users = User.where(id: @current_user).or @current_user.followings としました! そこで、下記のエラーが出ました。 undefined method `followings' for #<User:0x00007f8f57d9e9d0> Did you mean? following following? following= following_ids followers .following?に変更した場合のエラーは、 wrong number of arguments (given 0, expected 1) .followingに変更した場合のエラーは、 Relation passed to #or must be structurally compatible. Incompatible values: [:joins] となります。 .following?( )の()内の値をちゃんと渡せてないからエラーが出ているのでしょうか> <?
asm

2019/12/14 09:33

users = User.where(id: current_user).or current_user.following @posts = Post. where(user: users). order(created_at: :desc).page(params[:page]).per(8) だと、どうでしょうか?
tomtom1

2019/12/14 11:24

早速ご回答ありがとうございます! .joins(:user)を抜いたものですね! その場合下記のエラーが出てしまいます! Relation passed to #or must be structurally compatible. Incompatible values: [:joins]
asm

2019/12/14 12:05

current_user.followingだと何かがjoinしているんですかね @posts = Post.where(user: current_user). or(Post.where(user: current_user.following)). order(created_at: :desc).page(params[:page]).per(8) だといけるんかな
tomtom1

2019/12/15 09:00

そちらで無事に解決しました!!ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問