NoMethodErrorが出てしまって困っています。

Question

###前提・実現したいこと
ここに質問したいことを詳細に書いてください
progateのRoRの練習問題をコピペしてツイートアプリを自分の開発環境上で動かそうとしています。しかしながら、一つひとつ着実にコピーしているはずが、途中で次のようなエラーがでました。

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

undefined method `image_name' for nil:NilClass

<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">

###該当のソースコード

<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>

###試したこと
/public/user_images...などと書き変えてもダメでした
以前におこなったrails db:migrateがうまくいっていないのかと思ってもう一度実行しましたが、正常に実行されているようで、追加のメッセージはやはり出ませんでした。

###補足情報(言語/FW/ツール等のバージョンなど)
Rubyは2.3.1,RoRは5.0.0を使っています。
よろしくおねがいします。

追加情報になります

user.rbのコード

class User < ApplicationRecord
    validates :name, {presence: true}
    validates :email, {presence: true, uniqueness: true}
    validates :password, {presence: true}
    
    def posts
        return Post.where(user_id: self.id)
    end
    
end

post.rbのコード

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

posts_controller.rbのコード

class PostsController < ApplicationController
    before_action :authenticate_user
    
    before_action :ensure_correct_user, {only: [:edit, :update, :destroy]}
    
    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
    
   
    def ensure_correct_user
        @post = Post.find_by(id: params[:id])
        if @post.user_id != @current_user.id
            flash[:notice] = "権限がありません"
            redirect_to("/posts/index")
        end
    end
    
end

Comments

[scivola]

質問のコード部分のマークアップが正しくなされていない（``` の入れ方が間違っている？）ので，直していただけるとありがたいです。

[ninpig04]

修正しました

Answer 1

現象としては，ある Post に対して User が紐付いていない（対応する User が存在しない）ということなのですが，その原因は私にはわかりませんでした。

開発中に何かのはずみで user_id 無し（nil）の Post が作られてしまったか，当該の Post を投稿した User がのちに destroy されたか。
（後者かな？）

とりあえず，

post.user.image_name

を

post.user&.image_name

に変えればエラーは出なくなるでしょう。

&. は safe navigation operator と呼ばれるものですが，俗に「ぼっちオペレーター」「ぼっち演算子」ともいいます。Ruby 2.3 で導入されました。

以下の写真を見ると，「ぼっちオペレーター」と呼ばれる理由がわかります。
https://www.instagram.com/p/-M9l6mRPLR/

これはメソッド呼び出しの . の仲間です。
https://docs.ruby-lang.org/ja/2.4.0/doc/spec=2fcall.html

. との違いは，「レシーバーが nil なら nil を返す」という点です。

今の場合，post.user が nil を返しているので post.user&.image_name も nil を返します。

これでエラーなく表示されるでしょうから，その表示を見れば「ああ，この User は削除したんだった」とか分かるかもしれません。

ところで，「progateのRoRの練習問題」というのを見たことが無いのですが，ここに挙げられたコードはそこに載っていたものですか？
全体的にあまりよくない書き方だと思います。

例えば Post クラスの

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

などは

def user
  User.find(self.user_id)
end

と書けます。
Ruby のメソッドは最後に評価した式の値がメソッドの返り値となるので，return は要らないんです。
また，id で探すのは find メソッドの出番です。

それどころか belongs_to というものを使えばそもそもこのメソッドは必要ありません。

Comments

[ninpig04]

なるほど
今別のところでエラーが出ているので、それが解決したら試してみます

[ninpig04]

うまく動きました

Answer 2

エラーメッセージの意味は，

（NilClass のインスタンスである）nil image_name っていうメソッドは見当たらないけど？

という意味です。

何らかの原因で post.user が nil になっているため，エラーになったのですね。

post.user はおそらく User クラスのインスタンスが返るはずなのでしょうが，なぜ nil なのかは，もう少し情報が無いとわかりません。

Comments

[ninpig04]

情報を追加しました。まだ不足の情報がありましたらお伝えください。よろしくお願いいたします。