Railsでの画像とリンク表示について

Question

Railsでの画像表示についてお聞きしたいです。
以下のように書いて、user_id:1でログインしている状態です。
画像については、user = User.allでimage_name: "1.jpg"と保存されているのを確認しています。

index.html.erb

<div class="main posts-index">
  <div class="container">
    <% @posts.each do |post| %>
      <div class="posts-index-item">
        <div class="post-left">
          <img src="<%= "/user_images/#{post.user.image_name}" %>">
        </div>
        <div class="post-right">
          <div class="post-user-name">
            <%= link_to(post.user.name, "/users/#{post.user.id}") %>
          </div>
          <%= link_to(post.content, "/posts/#{post.id}") %>
        </div>
      </div>
    <% end %>
  </div>
</div>

post_controller.rb

  def index
    @posts = Post.all.order(created_at: :desc)
  end

post.rb

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

  def user
    return User.find_by(id: self.user_id)
  end

end

しかし、表示を試みると２点に対して以下のエラーが表示されてしまいます。

<img src="<%= "/user_images/#{post.user.image_name}" %>">

=> undefined method `image_name' for nil:NilClass

<%= link_to(post.user.name, "/users/#{post.user.id}") %>

=> undefined method `name' for nil:NilClass

これはどのように解決すべきか、ご教授いただければ幸いです。

Answer 1

Progateですよね

app/models/post.rbにこれを書いてないのではないでしょうか

class Post < ApplicationRecode
  def user
    return User.find_by(id: self.user_id)
  end
end

Comments

[退会済みユーザー]

はい、Progateです。回答ありがとうございます。
それは書いてます。それでもエラーが出すんですよね。。。

[story_aniki]

念のためにapp/models/post.rbのコードを張っていただけないでしょうか

[退会済みユーザー]

ありがとうございます。
post.rbを追加しました。

[story_aniki]

うーん、異状ないですね、あとはuser.rbにnameやimage_nameのpresence: trueのバリデーションが抜けていてたまたまuserのnameとimage_nameが同時に抜けているデータがあるとか‥
データベースのほうのデータは異状ないのですよね？名前が抜けているデータがあったり画像名が抜けているデータがあったり

[退会済みユーザー]

user = User.all で以下のようにデータベースが出力されます。
User Load (2.1ms) SELECT "users".* FROM "users" LIMIT ? [["LIMIT", 11]]
=> #<ActiveRecord::Relation [#<User id: 1, name: "1stUser", email: "[e-mail]", created_at: "2019-12-23 02:28:22", updated_at: "2019-12-23 21:21:54", image_name: "1.jpg", password: "1234">]>
たぶん抜けやミススペルはないと思うのですが。。。

Answer 2

エラーメッセージのとおりで、どちらもnilに対してメソッドを呼び出そうとしています。

推測するに、Userと紐付かないPostがあったのではないでしょうか。

Comments

[退会済みユーザー]

回答ありがとうございます。
#{post.user.image_name}のpostは
<% @posts.each do |post| %>のpostにかかっていると思います。
post_controllerで
@posts = Post.all.order(created_at: :desc)
となっているので、投稿を1つずつ出力する形になっていると思います。

なので#{post.user.image_name}は
投稿したユーザーの画像を示していると思うのですが、どうでしょうか？
（image_name）はデータベースの画像コラムを指しています。

[maisumakun]

> 投稿したユーザーの画像を示していると思うのですが、どうでしょうか？

その「投稿したユーザー」のデータがないPostのレコードが紛れ込んでいるのではないか、という話です。

[退会済みユーザー]

仰る通り、1つユーザーIDがついてないレコードがありました。
ユーザーIDをつける前の投稿で消し忘れがあったようです。
ユーザーIDがついてない投稿を消しましたら、エラーがでなくなりました。
ありがとうございました！