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

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

ただいまの
回答率

89.09%

undefined method `image_name' for nil:NilClass といエラーがでています

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 2,800

SatoYuka

score 8

 前提・実現したいこと

postとuserのuser_idの関連付けがうまくできずにいます。
progateのRuby on Rails5 学習コース Ⅸ 「 新規投稿をログインユーザーに紐付けよう」 のところで、投稿一覧にユーザー情報を表示ができずindex.html.erb でundefined method `image_name' for nil:NilClassというエラーが発生しています。

postのuser_idは存在しているのですが、それに紐づいているはずのuser_idがnilとなっており、postに対しuser_idがうまく紐づいてくれません。また、postでuser_idを保存しようとしても保存できず、falseが返ってきます。 以下のpost.rbとuser.erbのコードでは関連づけられていないのでしょうか?

 発生している問題・エラーメッセージ``NoMethodError in Posts#indexundefined method image_name' for nil:NilClass

エラーメッセージ

### 該当のソースコード  
---post.rb---  
class Post < ApplicationRecord  
validates :content, {presence: true, length: {maximum: 140}}  
validates :user_id, {presence: true}  

belongs_to :user  

end  
---user.erb----  
class User < ApplicationRecord  
validates :name, {presence: true}  
validates :email, {presence: true, uniqueness: true}  
validates :password, {presence: true}  


has_many :posts  
end  
---posts/index.html----  
<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>  
ソースコード   
---posts_controller.erb----  
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  

ここに言語を入力 コード

 試したこと```ここに言語を入力

コード

 補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

コードは正しいはずなのに post.user が nil であるとき,原因の一つとして考えられるのは,開発の中途段階で,posts テーブルに user_id が空のレコードができちゃってることです。
現在のコードではそういうレコードはできないはずなんだけど,コードがそうなる前にできちゃったレコードってことですね。

ただ,

また、postでuser_idを保存しようとしても保存できず、falseが返ってきます。 

とあるので,現在のコントローラーのコードが正しくない可能性が高いと思います。
そこのコードを見せてください。

Rails のバージョンはいくつですか。Rails 5 以降の場合,

belongs_to :user  

の記述では,user_id を空(nil)にするのは許されないはずなんですが。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/03/15 23:38

    回答いただきありがとうございます。
    バージョンはRails 5.1.5でした。
    Userのuser_idはカラ(nil)にはならないので、Postテーブルを作った時にuser_idがカラの物を作ってしまったということになるのでしょうか。
    そうした場合は、Postテーブルのuser_idだけ作り直すことが必要になってくるのでしょうか?

    postsコントローラーのソースを追加いたしましたので、お手数ですがご確認いただけたら幸いです。

    キャンセル

  • 2018/03/15 23:56

    ソースコードの提示で,インデントを無くされると読みづらいです。

    「Userのuser_idはカラ(nil)にはならないので」とありますが,User モデルにも user_id ってカラムがあるのですか???

    キャンセル

  • 2018/03/16 12:59

    インデントがうまく表示できなかったです。
    User にはidでした。すみません間違えました。

    キャンセル

  • 2018/03/21 17:13

    だいぶ時間が空いてしまいました。

    やはり現在のコードで,`user_id` が `nil` の `Post` はできそうな気がしないですね。
    今データベースに入っているデータを捨ててよければ,いったん
    rails db:drop
    でデータベースを無くして,改めて
    rails db:migrate
    してもう一度やってみては。
    たぶん,こんどは index で同じエラーは出ないと思います。

    それで,もし Post を投稿するときにエラーが出るようならそのエラーを見せてください。

    キャンセル

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

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