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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

1329閲覧

NoMethodError in PostsController#show が解消できません

akairen

総合スコア18

Ruby

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2020/01/14 14:36

Ruby(Rails)初学者で初めてのオリジナルアプリを作成しているのですがNoMethodError in PostsController#showというエラーが出てしまい解消できずにいます

このエラーについて調べてみたところ、同じようなエラーでデータベースのデータを消去したら解消されたという方やbelongs_to :user やhas_many :users をmodelに入れたら解決したという方がいらっしゃったのでしたが、データベースのデータを消去しても解消できませんでしたし、下記のコードにある通りbelongs_to :user やhas_many :users はあるのでなにが原因で起きているエラーなのかすらわからずにいます

発生している問題・エラーメッセージ

NoMethodError in PostsController#show undefined method `user_id' for nil:NilClass Extracted source (around line #14): @post = Post.find_by(id: params[:id]) @user = User.find_by(id: @post.user_id) end

 #14というのは @user = User.find_by(id: @post.user_id)の部分です

該当のソースコード

**controllers - 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を定義してください @user = User.find_by(id: @post.user_id) end def new @post = Post.new end def create @post = Post.new( content:params[:content], user_id: @current_user.id ) if @post.save 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 redirect_to("/posts/index") else render("posts/edit") end end def destroy @post = Post.find_by(id: params[:id]) @post.destroy 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
**model - user.rb** class User < ApplicationRecord has_secure_password validates :name, {presence: true} validates :email, {presence: true, uniqueness: true} has_many :posts def posts return Post.where(user_id: self.id) end end
**model - post.rb** class Post < ApplicationRecord validates :content, {presence: true ,length:{maximum:140}} validates :user_id,{presence:true} belongs_to :user end

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

showメソッドそのものじゃなくて、showメソッド内のプロパティuser_idは何ぞと怒ってます。

createメソッドでは確かにuser_idは入ってますが、一度値を返しているため、@post内にuser_idは保持されません。その状態でshowメソッドを実行してもuser_idプロパティはありません。
そして値がないならないでnil指定してくれっていうエラーです。

def show @post = Post.find_by(id: params[:id]) #ここで@postに代入されるのは右の文 # 変数@userを定義してください @user = User.find_by(id: @post.user_id) #このuser_idが何なのか end def create @post = Post.new( content:params[:content], user_id: @current_user.id ) #ここで作っても一度返してしまえば保持はされない if @post.save redirect_to("/posts/index") else render("posts/new") end end

投稿2020/01/14 15:01

編集2020/01/14 15:20
FKM

総合スコア3608

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

akairen

2020/01/19 13:56

返信が遅くなってしまい申し訳ありません ご指摘の通りuser_idの部分をuserにした結果解決できました ありがとうございました
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問