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

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

ただいまの
回答率

91.35%

  • Ruby

    5185questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 5

    86questions

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

解決済

回答 2

投稿 2017/11/30 11:29 ・編集 2017/12/01 00:19

  • 評価
  • クリップ 1
  • VIEW 63

ninpig04

score 8

前提・実現したいこと

ここに質問したいことを詳細に書いてください
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
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • scivola

    2017/11/30 12:45

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

    キャンセル

  • ninpig04

    2017/12/01 00:12

    修正しました

    キャンセル

回答 2

checkベストアンサー

0

現象としては,ある 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 というものを使えばそもそもこのメソッドは必要ありません。

投稿 2017/12/01 08:24

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/01 20:13

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

    キャンセル

  • 2017/12/05 03:27

    うまく動きました

    キャンセル

0

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

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

という意味です。

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

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

投稿 2017/11/30 12:44

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/12/01 00:19

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

    キャンセル

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

ただいまの回答率

91.35%

関連した質問

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

  • Ruby

    5185questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails 5

    86questions