コントローラーで、def userの呼び出し(rails)

Question

###実現したいこと
View内で条件分岐されたif文を、コントローラー内に記述したいです。
###コード

<% @posts.each do |post| %>
  <% if post.user.id != @current_user.id && post.user.status == 2 %>
  <% else %>
    <%= post.name %>
  <% end %>

postのuser.id がログインユーザーではない　且つ　post保持者ステータスが2出ない　というif文。

def user
    return User.find_by(id: self.user_id)
end

def index
    user = User.find_by(id: self.user_id)
    @mypostpost.user.id != @current_user.id
    @pstatus = Post.where.user(status: 2)
    @posts = Post.all.order(created_at: :desc).where.not(id: @mypost.id, id: @pstatus.id)
end

###エラー

undefined method `user_id' for #<PostsController:0x00007fba5e431b40> Did you mean? user_url

modelに記載のdef userがコントローラー内で反映されないため、エラーが出ます。
また、コントローラー内で記述してみましたが、反応が得られません。
どうぞ宜しくお願い致します。

Answer 1

modelで定義しているのは user でエラーが出ているのは user_id です。

「View内で条件分岐されたif文を、コントローラー内に記述」ということは、viewに渡す @posts　が「postのuser がログインユーザーではない　且つ　post保持者ステータスが2出ない」ものに絞り込んでおきたい
という意味でしょうか

まず、　user, user.id, user_id が混乱して使われてます。そこ注意です。

@posts = Post.joins(:user). where.not(users: {id: current_user.id}). where(users: {status: 2}). 　order(created_at: :desc)
かな、と思うのですが

@mypostpost.user.id != @current_user.id
@pstatus = Post.where.user(status: 2)

の部分が理解できていないので、後半の条件where.not(id: @mypost.id, id: @pstatus.idは省いたので、必要なら追加して下さい

追記
def user return User.find_by(id: self.user_id) end
が気になったので念の為.
これ class Post に定義したのではないかと思うのですが、belongs_to :user って定義してますか？
それがあればこのmethodは不要です。
ないと上の回答がうまく動きません

Comments

[tomtom1]

ご回答ありがとうございます。.joins(:user)について、大変勉強になりました。
また、追記の内容も、belongs_to :userがあれば記載は不要とのこと、ありがとうございます。
両方記載しておりました。

さて、ご質問させて頂いた件についてですが、言葉足らずで申し訳ないです。
A = postのuser.id がログインユーザーではない
B = post保持者ステータスが2出ない　
(A 且つ B )の条件を持ったというif文にしたいのですが、現状AとBという2つの条件になっていると思います。
A或いはBいずれかの条件が揃った場合でも、その条件には該当させたくなく、両方の条件が揃った場合、該当としたいのですが、いかがでしょうか。
事前の質問が言葉足らずで申し訳ありません。

[winterboum]

whereのchainはandになります

[tomtom1]

ありがとうございます。下記で試したのですが、全てエラーが出てしまい、未だうまくいきません。
.join(:user)メソッドを活用した場合のandでチェインする方法は、どのような書き方が良いのでしょうか？

.where.not(users: {id: @current_user.id}, users: {status: 1})
.where.not("users: {id: @current_user.id} and users: {status: 1}")
.where.not(users: {id: @current_user.id} and users: {status: 1})
.where.not('users: {id = ?} and users: {status = ?}', @current_user.id, 1)
.where.not('users: {id} = ? and users: {status} = ?', @current_user.id, 1)
.where.not(['users: {id = ?} and users: {status = ?}', @current_user.id, 1])
.where.not(users: {id: @current_user.id}).and(Post.where(users: {status: 1}))

[winterboum]

「かな、と思うのですが」のパターンは試してくれました？

[tomtom1]

はい、試しましてエラーなくできます。
しかし、その回答にご返信しました通り、
A と Bの条件と分かれた２つのif文ではなく、
(A 且つ B )の条件を持ったというif文にしたいという部分が叶えられずにおります> <

[winterboum]

え、
where.not(users: {id: current_user.id}). where(users: {status: 2}) のchainで
and の結果になっていませんか？
この部分実行時の log に SQLが出ているはずです。そこ見せて下さい

[tomtom1]

Post Load (0.3ms) SELECT "posts".* FROM "posts" INNER JOIN "users" ON "users"."id" = "posts"."user_id" WHERE "users"."id" != ? AND "users"."status" = ? ORDER BY "posts"."created_at" DESC [["id", 1], ["status", 1]]
このように記載されています。

[tomtom1]

求めているのは、
if post.user.id != @current_user.id && post.user.status == 2
の条件を求めていますが、現状
if post.user.id != @current_user.id || post.user.status == 2
という条件になっており。どちらかが該当された場合でも適応されPostが表示されなくなります。

[winterboum]

WHERE "users"."id" != ? AND "users"."status" = ? [["id", 1], ["status", 1]]
って
WHERE "users"."id" != 1 AND "users"."status" = 1
ってことですから、ANDになってるでしょ？　id が１でなくかつstatusが１になってますね
ん？
status が 2 になってないな。where(users: {status: 2}) 　になってます？

[tomtom1]

ご確認ありがとうございます。
ANDとなっていますが、その条件を別々に分けたくないのです。
一番上のindexに記載の通り、<% if post.user.id != @current_user.id && post.user.status == 2 %>に該当する場合は、スキップとしたいです。
しかし、今回の場合、最初のwhere.not(users: {id: current_user.id})の記載で、ログインユーザーのpostが全て表示されなくなります。

今回行いたいのは、
user.statusが2の場合、表示されなくなる。しかし、statusが2であっても、ログインユーザーの投稿の場合、表示する。です。

[winterboum]

それは
「postのuser.id がログインユーザーではない　且つ　post保持者ステータスが2出ない　というif文」
とは違いますね

[tomtom1]

そうですね！そのことに気づきましたので、言葉足らずで申し訳ないです。と前置きした上で、
次のコメントに詳しくしたいことを記載したつもりでした！
改めて、下記に記載しますが、いかがでしょうか。

A = postのuser.id がログインユーザーではない
B = post保持者ステータスが2出ない　
(A 且つ B )の条件を持ったというif文にしたいのですが、現状AとBという2つの条件になっていると思います。
A或いはBいずれかの条件が揃った場合でも、その条件には該当させたくなく、両方の条件が揃った場合、該当としたいのですが、いかがでしょうか。
事前の質問が言葉足らずで申し訳ありません。

[winterboum]

なってません。A 且つ Bになってます。

言葉足らずではないです、不正確なんです.違うことを言ってるんです
やりたいこと「user.statusが2の場合、表示されなくなる。しかし、statusが2であっても、ログインユーザーの投稿の場合、表示する」
と
条件が合っていないんです
しかもどうしたいかがくるくる変わる

「statusが２でなく、または user_が ログインユーザ」の時表示
すなわち
「(A 且つ B )の条件」では無いとき表示

ですよね？
status 　ユーザ
==２　　login 　　　　表示
==2 not login　　　非表示
!=2 login　　　　　表示
!= 2 not login　　　表示
ですね？

[tomtom1]

ご理解頂きありがとうございます！その通りです！
不正確でしたね、申し訳ないです！
この場合、どのようなコードが適切でしょうか。

[winterboum]

joins(:user).where("users.status != 2 or users.id =?",current_user.id)

[tomtom1]

そのように囲むのですね。解決できました！
ありがとうございます！

[winterboum]

# 正しい質問をしないと正しい答えは得られません