質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.10%

rails NoMethodError解決したいです。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 306

munequn

score 17

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

エラー文を読みとくと
(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]に値が伝わっているかどうかを調べてみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/09/25 12:48

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

    キャンセル

  • 2019/09/26 23:22

    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
    の順番にしたらエラーがでなくなったのですが関係ありますか?

    キャンセル

  • 2019/09/26 23:51 編集

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

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.10%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる