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

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

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

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

Ruby on Rails

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

Q&A

解決済

1回答

695閲覧

投稿した物を表示できません

akairen

総合スコア18

Ruby

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

Ruby on Rails

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

0グッド

0クリップ

投稿2020/01/21 08:53

rails初学者で初めてオリジナルアプリを作っているのですが、投稿したもの(データベースに保存したもの)を表示できず下記のようなエラーが発生してしまっています

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

NoMethodError in Users#show Showing /Users/kuritahajime/pile_up_app/app/views/users/show.html.erb where line #37 raised: undefined method `time' for nil:NilClass

というようなエラーメッセージが出ているのですが下記のコードにある通り、timeというのはデータベースのカラム名なのにも関わらずNilClassが出ているのがなぜなのかがどうしてもわかりません

該当のソースコード

**contorollers - users/contoroller.rb** 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 show @user = User.find_by(id: params[:id]) @post = Post.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: "pile-up3.jpg", password: params[:password] ) if @user.save session[:user_id] = @user.id 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 redirect_to("/users/#{@user.id}") else render("users/edit") end end def login_form end def login @user = User.find_by(email: params[:email]) if @user && @user.authenticate(params[:password]) session[:user_id] = @user.id redirect_to("/users/#{@user.id}") else @error_message = "メールアドレスまたはパスワードが間違っています" @email = params[:email] @password = params[:password] render("users/login_form") end end def logout session[:user_id] = nil 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
**users - show.html.erb** <div class="user-wrapper"> <div class="cont"> <div class="user-left"> <div class="user-img"> <%= image_tag '/images/pile-up3.jpg' %> </div> <div class="user-profile"> <div class="user-name"> <h3><%= @user.name %></h3> </div> <% if @user.id == @current_user.id %> <%= link_to("編集", "/#{@user.id}/edit") %> <% end %> <div class="user-time"> <p>積み上げ総時間</p> <h2>40分</h2> </div> </div> </div> <div class="user-right"> <div class="user-contents"> <div class="user-title"> <h2>あなたの積み上げ</h2> </div> <% @user.posts.each do |post| %> <div class="user-content"> <p class="time"><%= @post.time %>分</p> <p class="content"><%= @post.type %></p> <p class="matter"><%= @post.content %></p> <% if @post.user_id == @current_user.id %> <%= link_to("編集", "/posts/#{@post.id}/edit") %> <%= link_to("削除", "/posts/#{@post.id}/destroy",{method:"post"}) %> <% end %> </div> <% end %> </div> </div> </div> </div>
**route.rb** Rails.application.routes.draw do post "login" => "users#login" #OK get "login" => "users#login_form" #OK post "logout" => "users#logout" post "users/:id/update" => "users#update" get "users/:id/edit" => "users#edit" post "users/create" => "users#create" #OK get "signup" => "users#new" #OK get "users/:id" => "users#show" get "posts/index" => "posts#index" get "posts/new" => "posts#new" get "posts/:id" => "posts#show" post "posts/create" => "posts#create" get "posts/:id/edit" => "posts#edit" post "posts/:id/update" => "posts#update" post "posts/:id/destroy" =>"posts#destroy" get "/" => "home#top" #OK get "about" => "home#about" #OK # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html end
**migrate - add_time_to_post.rb** class AddTimeToPosts < ActiveRecord::Migration[5.2] def change add_column :posts, :time, :integer end end

試したこと

このエラーについて調べてみたら関連付け(belongs_to, has_many)で解決できるというような記事が出てきたので以下の通りに試してみたのですが上手くいきませんでした(そもそも関連付けのやり方を間違えていたらすいません)

**models - post.rb** class Post < ApplicationRecord validates :content, {presence: true ,length:{maximum:140}} validates :user_id,{presence:true} validates :time, {presence: true ,length:{maximum:4}} validates :type, {presence: true ,length:{maximum:30}} self.inheritance_column = :_type_disabled belongs_to :user def user return User.find_by(id: self.user_id) end end
**models - user.rb** class User < ApplicationRecord # has_secure_passwordメソッドを追加してください 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

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

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

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

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

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

guest

回答1

0

ベストアンサー

UserControllerのshowメソッド内の記述

def show @user = User.find_by(id: params[:id]) @post = Post.find_by(id: params[:id]) end

となっていますがuserのshowページのルーティングは

get "users/:id" => "users#show"

となっておりユーザーのID情報だけしかshowページでは受け取れていないと思います。
(例:http://〜/users/1 にアクセスするとUserテーブルもPostテーブルもURLに記載されたID1で検索しちゃっているはず、、)

おそらくですが、

@post = Post.find_by(id: params[:id])

@post = Post.where(user_id: @user.id])

上記の記述でユーザーIDをキーに投稿内容を複数件取得できた気がします。

久しくRailsを触っていないのに回答してしまい恐縮ですがご確認いただければと思います。

投稿2020/01/21 10:01

GenkiSugiyama

総合スコア86

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

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

akairen

2020/01/21 10:16

ご回答ありがとうございます アドバイスいただいと通り修正してみたのですが、エラーは解消できませんでした
GenkiSugiyama

2020/01/21 10:28

<% @user.posts.each do |post| %> の @user.posts→@post に変更した上で中身の@post.timeなどの「@post」をただの「post」に変更してみて試していただけますでしょうか?
akairen

2020/01/21 10:54

アドバイスいただいた通り修正してみた結果、質問文冒頭のエラーは解消されました(また別のエラーが発生したのでまだちゃんとと表示できているかを確認できてはいませんが...) 本当にありがとうございました
GenkiSugiyama

2020/01/21 11:10

投稿されていたエラーが解消されたようでよかったです! 初めのうちはエラーが出ると嫌な気持ちになってしまいがちですが、発想を変えてみるとエラーは「何がどうおかしいのか教えてくれている」と考えることもできます。 まずはエラーの英文を読み解いたりエラー文をそのままググったりして「どういう意味のエラーなのか」「どういう場合に発生するエラーなのか」を理解した上でご自分のソースと向き合ってみると今よりスムーズにエラーが解消できるようになると思います。 偉そうに長文失礼いたしました。Railsの勉強頑張ってください!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問