undefined method `user' for nil:NilClass Did you mean? superエラーを解決したい

Question

Ruby on Railsで画像を投稿し、お気に入り(like)ボタンを押した画像一覧を表示する機能を作成しています。
お気に入り画像一覧を表示するところまではできたのですが、画像のリンクをクリックした後、画像の詳細ページに飛ぶところでエラーが発生しています。

エラー内容

NoMethodError in PostsController#show
undefined method `user' for nil:NilClass Did you mean? super

def show
    @post = Post.find_by(id: params[:id])
    @user = @post.user ←ここがエラーになっています！
  end

  def destroy

コントローラー posts_controler.rb

class PostsController < ApplicationController
  before_action :authenticate_user!, only: [:new]

  def new
    @post = Post.new
    @post.photos.build
  end

  def create
    @post = Post.new(post_params)
    if @post.photos.present?
      @post.save
      redirect_to root_path
      flash[:notice] = "投稿が保存されました"
    else
      redirect_to root_path
      flash[:alert] = "投稿に失敗しました"
    end
  end

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

  def destroy
    @post = Post.find_by(id: params[:id])
    if @post.user == current_user
      flash[:notice] = "投稿が削除されました" if @post.destroy
    else
      flash[:alert] = "投稿の削除に失敗しました"
    end
    redirect_to root_path
  end

  private

  def post_params
    params.require(:post).permit(:caption, photos_attributes: [:image]).merge(user_id: current_user.id)
  end

  def user_params
    params.require(:user).permit(:image)
  end
end

コントローラー users_controler.rb

class UsersController < ApplicationController
  def show
    @user = User.find_by(id: params[:id])
    @post = @user.posts.order(id: "DESC").page(params[:page]).per(9)
    @like = @user.likes.order(id: "DESC").page(params[:page]).per(9)
  end

  def edit
    @user = User.find(params[:id])
  end

  def update
    @user = User.find(params[:id])

    if current_user == @user
      if @user.update(user_params)
        flash[:notice] = 'プロフィール画像を変更しました'
      else
        flash.now[:danger] = 'プロフィール画像の変更に失敗しました'
      end
      @post = @user.posts.order(id: "DESC").page(params[:page]).per(9)
      render :show
    else
      redirect_to user_path
    end
  end

  def destroy
    @user = User.find_by(id: params[:id])
    @posts = Post.find_by(user_id: params[:id])
    flash[:notice] = "ユーザーを削除しました"

    if @posts.nil?
      @user.destroy
      redirect_to root_path
    else
      @posts.destroy
      @user.destroy
      redirect_to root_path
    end
  end

  private

  def user_params
    params.require(:user).permit(:image)
  end
end

ビュー

<div class="container">
  <h3 class="text-center text-secondary mb-4">ユーザー情報</h3>

  <div class="offset-sm-3 col-sm-6">
    <div class="offset-sm-3 col-sm-6 text-center">
      <h4 class="mb-3"><%= @user.name %></h4>
      <%= form_for(@user) do |f| %>
        <% if @user.image? %>
          <%= image_tag @user.image.thumb.url, class: "round-img" %>
        <% else %>
          <%= image_tag "/assets/default.jpg", class: "round-img" %>
        <% end %>
    </div>
  </div>

  <div class="offset-sm-4 col-sm-4 offset-sm-4 text-center">
    <% if @user == current_user %>
      <button type="button" class="btn btn-outline-secondary rounded-pill mt-3">
        <%= f.file_field :image, accept: 'image/jpg,image/gif,image/png' %>
      </button>
        <%= f.submit "プロフィール画像変更", class: 'btn btn-outline-dark mt-1 form-control' %>
        <%= link_to "登録情報編集", edit_user_registration_path, class: "btn btn-secondary mt-3 form-control" %>
        <%= link_to "退会", user_destroy_path, method: :delete, data: { confirm: "本当に退会しますか？" }, class: "btn btn-danger mt-3 form-control" %>
      <% end %>
      <br>
    <% end %>
  </div>
  <hr>

</div>

<div class="photos-parent text-center">
  <h5><i class="fas fa-paw"></i> <%= @user.name %>の投稿画像</h5>
  <% @post.each do |post| %>
    <%= link_to(post_path(post)) do %>
      <div class="photos-children">
        <%= image_tag post.photos.first.image.url(:medium) %>
      </div>
    <% end %>
  <% end %>
</div>

<div class="photos-parent text-center">
  <h5><i class="fas fa-paw"></i> <%= @user.name %>のお気に入り画像</h5>
  <% @like.each do |like| %>
    <%= link_to(post_path(@post)) do %>
      <div class="photos-children">
        <%= image_tag like.post.photos.first.image.url(:medium) %>
      </div>
    <% end %>
  <% end %>
</div>

<div class="paginate-parent">
  <div class="paginate-children mt-3">
    <%= paginate @post %>
  </div>
</div>

ルーティング

Rails.application.routes.draw do
  devise_for :users,
    controllers: { registrations: 'registrations', omniauth_callbacks: 'users/omniauth_callbacks' }

  root to: 'tops#index'
  get 'tops/show', to: 'tops#show'

  get '/users/:id', to: 'users#show', as: :user
  patch '/users/:id', to: 'users#update'
  delete '/users/:id', to: 'users#destroy', as: :user_destroy

  resources :posts, only: %i(new create show destroy) do
    resources :photos, only: %i(create)
    resources :likes, only: %i(create destroy)
  end
  # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html
end

説明不足等ありましたら遠慮なく言ってください。
ご教授頂ければ幸いです。よろしくお願いします。

Answer 1

<%= link_to(post_path(@post)) do %>

ここが問題です。
ここでの@postはuserが投稿したものの集合です。
(複数なのに変数名が単数形なのが勘違いの原因の一つかなとは思います…)

正しくは、

    <%= link_to(post_path(like.post)) do %>

かな、と推測します。

---

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

私見ですがこのケースでは、
find_byよりもfindを使って、異常入力の場合に404 NotFound返した方がよいと思います。

find_byを使うのならば、nilが返却された場合にどうするべきなのか考える必要があります。

Comments

[kousuke24]

解決しました！
詳細に説明も書いて下さりありがとうございます。
とてもわかりやすく、勉強になりました！

Answer 2

@postがnilになってませんか。
チェックしてみてください

Comments

[kousuke24]

すいません、@postのnilをチェックするにはどこに、何を記入すれば良いのでしょうか？
恥ずかしながら初心者でして、、、
よろしければ教えて頂きたいです！

[y_waiwai]

テスト的に、その変数の内容を表示するコードを書いてみてはどうでしょう

[kousuke24]

usersのshow.html.erbに
<%= image_tag like.post.photos.first.image.url(:medium) if @post.nil? %>
if @post.nil?と追加した結果、画像が表示されなくなりました。
正直、y_waiwaiさんから教えて頂いたことが正しくできているのかわかっていません。これであっていますでしょうか？