投稿とユーザー画像の紐付けが上手くできません

Question

Ruby(Rails)初学者で初めてのオリジナルアプリを作成しているのですが投稿とユーザー画像の紐付けで下記のようなエラーが出てしまい解決できずにいます

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

NoMethodError in Posts#show
Showing /Users/kuritahajime/pile_up_app/app/views/posts/show.html.erb where line #6 raised:

undefined method `image_name' for nil:NilClass

該当のソースコード

**controllers - posts_controller.rb**


def show
    @post = Post.find_by(id: params[:id])
    # 変数@userを定義してください
    @user = User.find_by(id: @post.user_id)
    
  end

**posts - show.html.erb**

<div class="user-wrapper">
      <div class="cont">

        <div class="user-left">
          <div class="user-img">
            <img src="<%= "/user_images/#{@user.image_name}" %>">
          </div>

          <div class="user-profile">
            <div class="user-name">
              <h3>赤井れん</h3>
            </div>

            <% if @user.id == @current_user.id %>
              <%= link_to("編集", "/#{@user.id}/edit") %>
            <% end %>

            <div class="user-time">
              <p>積み上げ総時間</p>
              <h2>62時間40分</h2>
            </div>
            <div class="user-day">
              <p>積み上げ日数</p>
              <h2>42日</h2>
            </div>

          </div>
        </div>


    <div class="user-right">
         <div class="user-contents">

            <div class="user-title">
              <h2>あなたの積み上げ</h2>
            </div>

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

            <div class="user-content">
              <p class="time">30分</p>
              <p class="content">Rails学習</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>

**controllers - users_controller.rb**

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

試したこと

過去にProgateで同じようなことをやった記憶があり、確認してみたところProgateの9章で投稿(posts)とユーザー(users)を紐付けるというものがあったので、それを参考にしつつ所々コピペしてみたのですが上手くできませんでした

↓Progate9章　投稿とユーザーを紐付けよう↓

https://prog-8.com/rails5/study/9/4#/11

追記（modelのコード）

**model - 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

**model - post.rb**

class Post < ApplicationRecord
    validates :content, {presence: true ,length:{maximum:140}}
    validates :user_id,{presence:true}

    belongs_to :user
end

postの編集機能の部分のコードです↓

**contorollers - post_controller**

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

**new.html**
<div class="edit-wrapper">
      <div class="cont">
        <div class="edit-wrapper-main">
          <div class="edit-title">
           <h1>今日の積み上げ</h1>
          </div>
          <div class="edit-contents">

            <div class="edit-content edit-1">
              <p>今日の積み上げ時間</p>
              <p><textarea class="edit-time" type="text" name="" value="">   </textarea>分</P>
            </div>

            <div class="edit-content edit-2">
              <p>今日やったこと</p>
              <textarea class="edit-thing" type="text" name="" value="">   </textarea>
            </div>

            <div class="edit-content">
              <p>具体的な内容</p>
              <%= form_tag("/posts/create") do %>
              <textarea name="content" rows="8" cols="80"> <%= @post.content %> </textarea>
              <% end  %>
            </div>



          </div>
          <input class="edit-submit" type="submit" name="" value="投稿">
        </div>
      </div>
    </div>

Answer 1

@userがnil(該当するUserが居なかった）ためのエラーです。

def show
    @post = Post.find_by(id: params[:id])
    # 変数@userを定義してください
    @user = User.find_by(id: @post.user_id)
end

の @post.user_id が実在のUser id になっていないためでしょう。
rails 5 からはそうならないようになっているのですが、なにかしてしまってますね。
User と Psot の関連の定義を見せてください。
belongs_to とか has_many とかのことです

Comments

[akairen]

User と Psot の関連の定義というのはmodelのことでよろしいでしょうか？（間違っていたらすいません）
質問文にmodelのコードを加筆しておきましたがbelongs_to や has_manyなどはありませんでした
それが問題なのでしょうか？

[winterboum]

まぁ、問題です。
Postにはuser_idが必須だと思うのですが、それが
belongs_to :user
としておくだけで、user_idのないPost、実在しないuser_id の場合、Postのとうろくができなくなります

[akairen]

アドバイスいただいた通りpost.rbにbelongs_to :user入れたのですがエラーが解消できず、belongs_to について調べてみたところ、参照元のモデルにhas_manyを入れるとあったのでuser.rbにhas_many　:postsを入れました（修正した部分は質問文の方に加筆してあります）

それでもエラーは解消されず当初のエラー文がまだ出てきてしまうのですが私の解釈のどこかがおかしいのでしょうか？

それとも別の部分に問題があるのでしょうか？

[winterboum]

それらの呪文はこれから作成するものに対して有効です。
既に有るものはそのままなので。
Postの修正ってありますか？それを行えば呪文が効きます。

あと、
私の仮説が合っているかどうかを調べるには、rails c して
User.ids の結果と
Post.pluck(:user_id).uniq の結果を比べてください。

[akairen]

rails cを起動してアドバイスしていただいた通りにしたところ

irb(main):001:0> User.ids
(0.4ms) SELECT "users"."id" FROM "users"
=> [1, 2, 3]
irb(main):002:0> Post.pluck(:user_id).uniq
(0.7ms) SELECT "posts"."user_id" FROM "posts"
=> [nil]

となりました

Postの修正とはどの部分をどうすれば良いのでしょう？

[winterboum]

Postのプログラムの修正ではなく、Postの内容の書き換えです。
Postの編集機能はない？
もしそうでしたらPostを作りなおしましょう。
rails c して　Post.delete_all すると全部消えます。
そこでPostしなおしてください。
多分失敗しますが。
そうしたら、PostControllerやPostのnew.htmlやらを見せていただくことになります

[akairen]

postの編集機能の部分(createアクション、updateアクション)を含むpost_contorollerとpostのnew.htmlのコードを追記しておきました

[winterboum]

で、既に有る Postを修正してみましたか？

[winterboum]

aa,一覧を出すところでエラーになるから修正しようにも出来ないか。
ではPostを全削除して作りなおしましょう。
いまあるcodeなら一覧表示できるデータができると思います

[akairen]

すいません、そもそもなにがどう悪くてこのような状態になっているのかをよく理解できていないのでその部分をご教授いただけないでしょうか？

[winterboum]

class Postのvaridationや関連定義を設定する前に作られたPostがあるのではないでしょうか。
そのPostにuser_idがないため、
@user = User.find_by(id: @post.user_id)　で@userがnilとなり
<img src="<%= "/user_images/#{@user.image_name}" %>">
で、Nilにはimage_nameがないというえらーとなった

[akairen]

つまり、なんらかの手違いでPostが複数あり、それを修正するためにPostを削除し作り直すという解釈でよろしいでしょうか？

[winterboum]

「なんらかの手違いでPostが複数あり」→「なんらかの手違いで情報不十分なPostがあり」
ですね

[akairen]

なるほど、これを削除してまた作り直すとして前回と同じように　rails g controllerで作っても大丈夫なのでしょうか？
それとも同じように作るとまた情報不十分なPostなってしまうのでなんらかの別の方法で作るべきなのでしょうか？

[winterboum]

作り直しというのは Post　のデータです。
rails c で入って、 Post.delete_all すると全部消えますので、
それから新たに Post のデータを作ってください

[akairen]

Post.delete_allした結果、エラーが解消されました。ありがとうございます。
私の知識や理解力が不足していてかなりお手間を取らせてしまい申し訳ありませんでした

[winterboum]

良かったです。
同じ内容の二つの質問も解決済みにしておいてｋださい