モデルに定義したメソッドが使えない

Question

前提・実現したいこと

今まで稼働していたタイムライン(/posts/index)が突如表示されなくなりました。どうやらpost.rbのuserメソッドの読み込みがうまくいってないようです。
どう解決すればいいのかわからないのでご教授ください。

発生している問題・エラーメッセージ

該当のソースコード

index.html.erb

<div class="main-content">
  <div class="container">
    <h1>TimeLine
      <%= link_to(posts_new_path) do %>
        <span>
          <button type="button" class="btn btn-primary rounded-circle p-0" style="width:2rem;height:2rem;">
            ＋
          </button>
        </span>
      <% end %>
    </h1>
    
    <% @posts.each do |post| %>
      <div class="posts-index-item mb-20">
        <div class="posts-index-user">
          <!--avatar-->
          <div class="posts-index-img d-inline">
            <%= link_to("/users/#{post.user.id}") do %>
              <% if post.user.avatar.attached? %>
                <%= image_tag post.user.avatar, class: "avatar-index rounded-circle mx-auto" %>
              <% else %>
                <img class="avatar-index rounded-circle mx-auto" src="<%= "/images/default_user.png" %>" alt="Userimage">
              <% end %>
            <% end %>
          </div>
          <!--username-->
          <div class="posts-index-username d-inline">
            <%= link_to post.user.username, "/users/#{post.user.id}", class: "text-dark font-weight-bold" %>
          </div>
          <span class="text-secondary">
          ・<%= distance_of_time_in_words_to_now(post.created_at) %>前
          </span>
        </div>
        <!--content-->
        <div class="post-content">
          <%= link_to(post.content, "/posts/#{post.id}", class: "text-dark no-deco") %>
        </div>
      </div>
    <% end %>
  </div>
</div>

post.rb

class Post < ApplicationRecord
  validates :content, {presence: true, length: {maximum: 140}}
  validates :user_id, {presence: true}
    
  has_one_attached :avatar
  
  def user
    return User.find_by(id: self.user_id)
  end
end

試したこと

• showアクションはうまくいっているのでやはりuserメソッドがおかしい
• userメソッドが使われているところだけ切り取るとうまく描画されるのでやはりuserメソッドがおかしい

補足情報（FW/ツールのバージョンなど）

Ruby 2.6.3
rails 5.2.4.4
devise,activestorage導入済

Comments

[winterboum]

エラーは全文載せてください。

[Puchimaru24]

失礼しました。画像を添付しました。

Answer 1

単に「user_idがnil、もしくはuserと対応しない値だった」だけではないでしょうか？

---

ご提示のPost#userのようなメソッドは、（あえて勉強としてやるなら別ですが）手で書くべきではありません。「belongs_to :user」と書くだけで、Post#userよりはるかに便利な機能性を手に入れられます。

Comments

[Puchimaru24]

回答ありがとうございます。
勉強として記述していましたが、これからはbelongs_toを使っていきたいと思います。belongs_toとユーザー削除時の投稿削除で解決できそうです。

Answer 2

user methodが動かないのではなく、指定したidのuserが居ないということでしょう。

Postのvalidationでuser_idの存在は確保していますが、それだけでは実在のidで無くても登録できてしまいます。

belongs_to を勉強してください

Comments

[Puchimaru24]

回答ありがとうございます。
おっしゃる通り、ユーザーを削除したのにそのユーザーの投稿が残っていたのが原因でした。belongs_toとユーザー削除時の投稿削除で解決できそうです。

[winterboum]

では、user にhas_many　を設定し、dependant: destory オプションしましょう