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

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

ただいまの
回答率

87.79%

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

解決済

回答 1

投稿

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

score 18

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

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 19:16

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

    キャンセル

  • 2020/01/21 19:28

    <% @user.posts.each do |post| %>

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

    キャンセル

  • 2020/01/21 19:54

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

    本当にありがとうございました

    キャンセル

  • 2020/01/21 20:10

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

    偉そうに長文失礼いたしました。Railsの勉強頑張ってください!

    キャンセル

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

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

関連した質問

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