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

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

ただいまの
回答率

89.06%

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

解決済

回答 2

投稿 編集

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

Yuukou

score 5

前提・実現したいこと

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
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • NCC1701

    2019/12/06 20:25

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

    キャンセル

  • Yuukou

    2019/12/06 21:26

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

    キャンセル

  • NCC1701

    2019/12/06 22:47

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

    キャンセル

  • Yuukou

    2019/12/06 23:32

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

    キャンセル

回答 2

checkベストアンサー

+1

エラーメッセージがすべて掲載されていないので、想像で答えます。が、ほぼ合っているはずです。
エラー発生箇所の
<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されているかを確認してみてください。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/12/07 23:08

    ありがとうございます。
    コンソールで確認してみたところ
    ```
    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のコントローラーを新たに追加しましたので、よろしければご覧ください。

    キャンセル

  • 2019/12/08 09:47

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

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

    キャンセル

  • 2019/12/08 21:27

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

    キャンセル

+1

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

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

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

同じタグがついた質問を見る