Railsで掲示板投稿機能での画像表示ができない

Question

Rails初心者です。
post.user.image_nameの方は表示されるのですが、
post.post.post_imageはメソッドエラーが出てしまい表示できません。
どこがおかしいのでしょうか？

index.html.erb

<div class="main posts-index">
  <div class="container">
    <% @posts.each do |post| %>
      <div class="posts-index-item">
        <div class="post-left">
          <!-- ユーザーの画像が表示されるように、以下のsrcに値を追加してください -->

    <img src="<%= "/#{post.user.image_name}" %>">
        </div>
        <div class="post-right">
   <img src="<%= "/post_images/#{post.post.post_image}" %>">


          <div class="post-user-name">

            <!-- link_toメソッドを用いて、ユーザー詳細ページへのリンクを作成してください -->

          </div>
            <%= link_to(post.content, "/posts/#{post.id}") %>

        </div>

      </div>

    <% end %>
  </div>

</div>

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 = @post.user
    # 変数@likes_countを定義してください
    @likes_count = Like.where(post_id: @post.id).count
  end

  def new
    @post = Post.new
  end

  def create
    @post = Post.new(
      content: params[:content],
      user_id: @current_user.id)
      #,post_image: nil)
    @post.save

    if @post.save

      if params[:post_image]
     @post.post_image = "#{@post.id}.jpg"
     image = params[:post_image]
     File.binwrite("public/post_images/#{@post.post_image}",image.read)
   end

      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

  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

users_controller.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 index
    @users = User.all
  end

  def show
    @user = User.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: "054.jpg",
      password: params[:password]
    )
    if @user.save
      session[:user_id] = @user.id
      flash[:notice] = "ユーザー登録が完了しました"
      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.image_name}", image.read)
    end

    if @user.save
      flash[:notice] = "ユーザー情報を編集しました"
      redirect_to("/users/#{@user.id}")
    else
      render("users/edit")
    end
  end

  def login_form
  end

  def login
    @user = User.find_by(email: params[:email], password: params[:password])
    if @user
      session[:user_id] = @user.id
      flash[:notice] = "ログインしました"
      redirect_to("/posts/index")
    else
      @error_message = "メールアドレスまたはパスワードが間違っています"
      @email = params[:email]
      @password = params[:password]
      render("users/login_form")
    end
  end

  def logout
    session[:user_id] = nil
    flash[:notice] = "ログアウトしました"
    redirect_to("/login")
  end

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

    # 変数@likesを定義してください
    @likes = Like.where(user_id: @user.id)
  end

  def ensure_correct_user
    if @current_user.id != params[:id].to_i
      flash[:notice] = "権限がありません"
      redirect_to("/posts/index")
    end

application_record.rb

class ApplicationRecord < ActiveRecord::Base
  self.abstract_class = true
end

like.rb

class Like < ApplicationRecord
    validates :user_id, {presence: true}
    validates :post_id, {presence: true}
  end
  

post.rb

class Post < ApplicationRecord
  validates :content, {presence: true, length: {maximum: 140}}
  validates :user_id, {presence: true}
  
  # インスタンスメソッドuserを定義してください
  def user
    return User.find_by(id: self.user_id)
  end
  
end

user.rb

class User < ApplicationRecord
  validates :name, {presence: true}
  validates :email, {presence: true, uniqueness: true}
  validates :password, {presence: true}
  
  # インスタンスメソッドpostsを定義してください
  def posts
    return Post.where(user_id: self.id)
  end
  
end

schema.rb

# This file is auto-generated from the current state of the database. Instead
# of editing this file, please use the migrations feature of Active Record to
# incrementally modify your database, and then regenerate this schema definition.
#
# Note that this schema.rb definition is the authoritative source for your
# database schema. If you need to create the application database on another
# system, you should be using db:schema:load, not running all the migrations
# from scratch. The latter is a flawed and unsustainable approach (the more migrations
# you'll amass, the slower it'll run and the greater likelihood for issues).
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 2020_10_30_055415) do

  create_table "likes", force: :cascade do |t|
    t.integer "user_id"
    t.integer "post_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "posts", force: :cascade do |t|
    t.text "content"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "user_id"
    t.string "post_image"
  end

  create_table "users", force: :cascade do |t|
    t.string "name"
    t.string "email"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.string "image_name"
    t.string "password"
  end

end

追記

index.html.erbの中の
<img src="<%= "/post_images/#{post.post.post_image}" %>">を
<img src="<%= "/post_images/#{post.post_image}" %>">
に変更し、エラーは消えましたが画像がうまく表示できませんでした。

posts_controllerのif @post.saveを変更。

Comments

[m.ts10806]

モデルのリレーションはどうなっているのでしょうか。

[ocs_R]

こうなっております。

application_record.rb
class ApplicationRecord < ActiveRecord::Base
self.abstract_class = true
end

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

# インスタンスメソッドuserを定義してください
def user
return User.find_by(id: self.user_id)
end

end

post.rb

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

# インスタンスメソッドuserを定義してください
def user
return User.find_by(id: self.user_id)
end

end

user.rb

class User < ApplicationRecord
validates :name, {presence: true}
validates :email, {presence: true, uniqueness: true}
validates :password, {presence: true}

# インスタンスメソッドpostsを定義してください
def posts
return Post.where(user_id: self.id)
end

end

[m.ts10806]

質問は編集できますし、こちらのコメント欄ではマークダウン使えませんので原則質問本文を更新してください

[ocs_R]

追加いたしました、よろしくお願いします。

[punchan36]

schema.rbを追記して頂けませんか？

[ocs_R]

追加いたしました。よろしくお願いします。

Answer 1

posts_controller.rbを拝見する限り、投稿用の画像のカラム名は恐らくpost_imageになっているかと思います。
posts/index.html.erbにある該当のコードを、下記の様に変えてみてはいかがでしょうか。

   <img src="<%= "/post_images/#{post.post.post_image}" %>">
　　↓
   <img src="<%= "/post_images/#{post.post_image}" %>">

【追記】
posts_controller.rbのif @post.save以下をこの様に変更してみてはいかかでしょうか。

    if @post.save
      if params[:post_image]
     　 @post.post_image = "#{@post.id}.jpg"
     　 post_image = params[:post_image]
     　 File.binwrite("public/post_images/#{@post.post_image}", post_image.read)
      end
      @post.save
      flash[:notice] = "投稿を作成しました"
      redirect_to("/posts/index")
    else
      render("posts/new")
    end

Comments

[ocs_R]

回答ありがとうございます。
コードを変更したのですが、画像表示がうまくできませんでした。
追記したので確認してもらってもよろしいでしょうか。
宜しくお願い致します。

[punchan36]

コード変更後に再度新規で「画像選択+投稿」をしても、新しい投稿もスクリーンショットの様に上手く表示されませんでしょうか？

[ocs_R]

はい、一応フォルダの方には保存されているのですが、画像表示はできませんでした。

[punchan36]

posts_controller.rbに関して回答に追記しました。そちらの方法ではどうでしょうか。

[ocs_R]

コードを入力をしてみましたが、メソッドエラーがでました。

[ocs_R]

全角の空白のスペースがあったみたいなので、修正したら動きました。
ありがとうございました！！！！！！！！！！！！！！！！！！

[punchan36]

良かったです！
コードを拝見させて頂いた感じ、progateでrailsを勉強しながらご自身でも試行錯誤されている所でしょうか？私もそうです。以前似たような所で躓いた事がありましたので、こちらもご参考になればと思います。
https://teratail.com/questions/300186
お互い頑張りましょう！