rails NoMethodError解決したいです。

Question

rails初学者で投稿型webアプリを作成中です。
以下のエラーが発生し困っています。
image_nameが定義されていないと言われているのは分かるのですが、前にも同じようなエラーが発生し、エラー文の指摘されている箇所ではないところを修正して解決に至りました。
今回は調査してもわからないのご教授願います。

●合わせてお聞きしたいのがrailsのエラー文の指摘とは関係ないところを指してしまうこともあるのでしょうか？

ActionView::Template::Error (undefined method `image_name' for nil:NilClass):
    1: <div class="main user-show">
    2:   <div class="container">
    3:     <div class="user">
    4:       <img src="<%= "/user_images/#{@user.image_name}" %>">
    5:       <h2><%= @user.name %></h2>
    6:       <p><%= @user.email %></p>
    7:       <% if @user.id == @current_user.id %>
  
app/views/users/likes.html.erb:4

エラーが発生しているソースコード

<div class="main user-show">
  <div class="container">
    <div class="user">
      <img src="<%= "/user_images/#{@user.image_name}" %>">
      <h2><%= @user.name %></h2>
      <p><%= @user.email %></p>
      <% if @user.id == @current_user.id %>
        <%= link_to("編集", "/users/#{@user.id}/edit") %>
      <% end %>
    </div>
    <ul class="user-tabs">
      <li><%= link_to("投稿", "/users/#{@user.id}") %></li>
      <li class="active"><%= link_to("いいね!", "/users/#{@user.id}/likes") %></li>
    </ul>
    <% @likes.each do |like| %>
      <% post = Post.find_by(id: like.post_id) %>

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

class UsersController < ApplicationController
  before_action :authenticate_user, only: %i[index show edit update] # ログインユーザーのみ閲覧
  before_action :forbid_login_user, only: %i[new create login_form login] # ログアウトユーザーが閲覧
  before_action :ensure_correct_user, only: %i[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],
      password: params[:password],
      image_name: 'default_user.jpg'
    )
    if @user.save
      session[:user_id] = @user.id
      flash[:notice] = 'ユーザー登録が完了しました'
      redirect_to("/users/#{@user.id}")
    else
      render('users/new')
  end
end

  def edit
    @user = User.find_by(id: params[:id])
    # @user =User.find_by(name: params[:name], email: params[:email])
  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 # 情報を保持する
      flash[:noice] = 'ログインしました'
      redirect_to('/posts/index')
    else
      @error_message = 'メールアドレスまたはパスワードが間違っています'
      @email = params[:email]
      @password = params[:password]
      render('users/login_form')
    end
  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
    def likes
      @user = User.find_by(id: params[:id])
      @likes = Like.where(user_id: @user.id)
    end
end
  # ユーザーの削除
  # def destroy
  #   @user = User.find_by(:id params[:id])
  #   @user.destroy
  #   flash[:notice]="削除しました"
  #   redirect_to("/users/index")
  # end
end

class Like < ApplicationRecord
  validates :user_id, {presence: true}
  validates :post_id, {presence: true}
end

よろしくお願い致します。

Answer 1

エラー文を読みとくと
(undefined method `image_name' for nil:NilClass)＝「image_nameというメソッドはnilクラスには定義されていません」と、いう意味になるので。
おそらくこのimage_nameを呼び出している@userにユーザーオブジェクトが代入されてない可能性があります。

このテンプレートを呼び出しているlikeアクション側では、@userに、
User.find_by(id: params[:id])と、find_byメソッドに引数（id:params[:id]）を渡す事でユーザーオブジェクトを生成し、代入しているようですが、何らかの原因でユーザーオブジェクトが生成されていない為に、nilクラス（中身が空のオブジェクト）が生成されその結果エラーになっているように思われます。
恐らく原因としては引数である（parmas[:id]）にうまく値が渡ってない事が原因かなと思いますので、一度このアクションを呼び出すメソッドを調べ、params[:id]に値が伝わっているかどうかを調べてみてください。

Comments

[munequn]

丁寧かつ迅速な回答ありがとうございます！
質問なのですが調べるとはテストコードを書くという意味でしょうか？

[kingyo0802]

サーバーを起動させた状態でlikeアクションを反応させるリクエストを発行すれば、サーバーのログに各種情報の動きが表示されると思いますので、それを読んで見てちゃんとparams[:id]に値が入っているを確認するのが一番早くて正確ですね！

もしくは超アナログな方法ですが、@userを一度消して、代わりに適当な＠変数を作ってそこにparms[:id]を代入してその変数をテンプレートで表示させてみて、ちゃんと整数が出てくるかどうかを見てみたりするなんて事もあります。

[munequn]

ありがとうございます！
試してみます！

[munequn]

def likes
@user = User.find_by(id: params[:id])
@likes = Like.where(user_id: @user.id)
end
def ensure_correct_user
if @current_user.id != params[:id].to_i
flash[:notice] = '権限がありません'
redirect_to('/posts/index')
end
の順番にしたらエラーがでなくなったのですが関係ありますか？

[kingyo0802]

アクション元がどういった作りになっているか分からないのでハッキリとは言えませんが、likesというアクションとensure_correct_userというアクションに関連性がなければ関係はない筈です。

ただ、もしかすると元からこのエラーの原因が文法的ミスではなく、記述的なミス（余分なスペースが混ざっていた等）で起こっていたものとすると、アクションを入れ替えた際に、偶然記述ミスがなくなり、正常に機能するようになったのかしれません。