railsで投稿一覧にユーザー名を表示する際に発生したNoMethodError in Posts#indexの原因を知りたい

Question

前提・実現したいこと

railsにて簡単なSNSサービスを作っています。
その中の機能の一つとして投稿一覧にユーザー名の画像とユーザー名を表示させる機能を実装中に以下のエラーメッセージが発生しました。

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

NoMethodError in Posts#index

/app/views/posts/index.html.erb where line #7 raised:

undefined method `image_name' for nil:NilClass

該当のソースコード

     <img src="<%= "/user_images/#{post.user.image_name}" %>">

試したこと

コードの解法がある教材(Progate)を進めていたので、該当するファイルの答えをコピーして見ましたがエラーになってしまいました。

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

問題のposts/index.html.erbのコードはこちらです。

<div class="main posts-index">
  <div class="container">
    <% @posts.each do |post| %>
      <div class="posts-index-item">
        <div class="post-left">

          <img src="<%= "/user_images/#{post.user.image_name}" %>">
        </div>
        <div class="post-right">
          <div class="post-user-name">

            <%= link_to(post.user.name, "/users/#{post.user.id}") %>
          </div>
          <%= link_to(post.content, "/posts/#{post.id}") %>
        </div>
      </div>
    <% end %>
  </div>
</div>

post.rbは以下のようになります。

class Post < ApplicationRecord
  validates :content, {presence: true, length: {maximum: 140}}
  validates :user_id, {presence: true}
  belongs_to :user
  def user
    return User.find_by(id: self.user_id)
  end
  # self.user_idのユーザーを戻り値として返す
end

コントローラー内でのアクションは以下のようになります。

class PostsController < ApplicationController
  before_action :authenticate_user
  
  def index
    @posts = Post.all.order(created_at: :desc)
  end
  
  def show
    @post = Post.find_by(id: params[:id])
    @user = @post.user
  end
  
  def new
    @post = Post.new
  end
  
  def create
    @post = Post.new(
      content: params[:content],
      user_id: @current_user.id
    )
    if @post.save
      flash[:notice] = "投稿を作成しました"
      redirect_to("/posts/index")
    else
      render("posts/new")
    end
  end
  
  def edit
    @post = Post.find_by(id: params[:id])
  end
  
  def update
    @post = Post.find_by(id: params[:id])
    @post.content = params[:content]
    if @post.save
      flash[:notice] = "投稿を編集しました"
      redirect_to("/posts/index")
    else
      render("posts/edit")
    end
  end
  
  def destroy
    @post = Post.find_by(id: params[:id])
    @post.destroy
    flash[:notice] = "投稿を削除しました"
    redirect_to("/posts/index")
  end
  
end

usersのコントローラーはこちらになります。

class UsersController < ApplicationController

  before_action :authenticate_user,{only: [:index,:show,:edit,:update]}
  before_action  :forbid_login_user,{only:[:new,:create,:login_form,:login]}
  before_action  :ensure_correct_user, {only: [:edit, :update]}

  def index
    @users = User.all
  end
  def show
    @user = User.find_by(id: params[:id])
  end
  def new
    @user = User.new
  end
  def create
    @user = User.new(
      name: params[:name],
      email: params[:email],
      image_name: "default_user.jpg",
      password: params[:password]
      )
    if @user.save
      session[:user_id] = @user.id       # 登録されたユーザーのidを変数sessionに代入
    flash[:notice] = "新規登録しました。"
     redirect_to("/users/#{@user.id}")
    else
      render("users/new")
    end
  end
  def edit
    @user = User.find_by(id: params[:id])
  end
  def update
    @user = User.find_by(id: params[:id])
    @user.name = params[:name]
    @user.email = params[:email]

    if params[:image]
      @user.image_name = "#{@user.id}.jpg"
      image = params[:image]
      File.binwrite("public/user_images/#{@user.image_name}", image.read)
    end

    if @user.save
      flash[:notice] = "登録内容を編集しました。"
      redirect_to("/users/#{@user.id}")
    else
      render("users/edit")
    end
  end

  def login_form
  end
  def login 
    @user = User.find_by(
      email: params[:email],
      password: params[:password]
    )    #フォームに入力されたメールアドレスとパスワードを取得
    if @user
      session[:user_id] = @user.id # 変数sessionに、ログインに成功したユーザーのidを代入
      flash[:notice] ="ログインしました"
      redirect_to("/posts/index")
    else
      @error_message = "メールアドレスまたはパスワードが間違っています"
      @email = params[:email]
      @password = params[:password]
      render("users/login_form")
    end #@userが存在するかどうかを判定するif文作成
  end

    def logout
      session[:user_id] = nil
      flash[:notice] = "ログアウトしました"
      redirect_to("/login")
    end

    def ensure_correct_user
      if @current_user.id != params[:id].to_i
        flash[:notice] = "権限がありません"
        redirect_to("/posts/index")
      end
    end    # ログイン中のユーザー」と「編集しようとしているユーザー」が正しくない場合の処理

end

Comments

[NCC1701]

コントローラーは全て記載しましょう

[Yuukou]

ご指摘いただきありがとうございます。コントローラを追記しました。
もしまだ記載の足りない箇所がありましたらよろしくお願いします。

[NCC1701]

`NoMethodError in Posts#index'のエラーはpost_controller.rb（PostsController）に`index’メソッドが無い、とのことです。掲載されたコントローラーはUsersContorollerなので、分析できません。

[Yuukou]

申し訳ありません。コントローラーを変更しました。

Answer 1

class Post に
belongs_to :user
を追加して下さい。

それでも駄目だった時は、Yuukouさんの指摘です。
user_idにvalidatinが入ってるので、そこに値は入っていることは保証されますが、belongs_toがないので、そのidにあたるUserがいないということかも。

belongs_toをつけた後、rails c にて
Post.first.user
してみるとわかります

Answer 2

エラーメッセージがすべて掲載されていないので、想像で答えます。が、ほぼ合っているはずです。
エラー発生箇所の
<img src="<%= "/user_images/#{post.user.image_name}" %>">
で、post.user.image_nameが原因でしょう。
すなわち、
「postインスタンスのuserにはimage_nameというメソッドがないよ」のはずです。
多くの場合、これはpostインスタンスのuserが「nil」のケースが多いです。
（理屈上では、userにはimage_nameという属性（カラム）が無いよというパターンもありえますが、モデル設計の問題で発見しやすいので稀でしょう））

つまり、postとuserの関連付けのモデル設計はしたものの、具体的なインスタンスにおいて、保存がされていないケースです。
コンソール等で@post.userを確かめるとnil（か空配列）が返ってくると思います。
なので、post.userにuserインスタンス（@user）を代入しているか、代入ごsaveされているかを確認してみてください。

Comments

[Yuukou]

ありがとうございます。
コンソールで確認してみたところ
```
Traceback (most recent call last):
1: from (irb):4
NoMethodError (undefined method `user' for nil:NilClass)
Did you mean? super
```
という表示をされました。

そのため、post.userにuserインスタンス（@user）を代入しているか、代入後saveされているか、usersのコントローラーを確認したのですが、異常はないと思われます...。(見落としがあったら、すみません...。)

usersのコントローラーを新たに追加しましたので、よろしければご覧ください。

[winterboum]

それは、post=Post,なにがし　を行わずにpost.user.image_nameをやってるからです。

で、
正しい答えを得るには正しい質問が必要です。
まず、エラーメッセージを全文載せましょう

[Yuukou]

申し訳ございません。
エラーメッセージを編集しました。よろしければご覧ください。