いいね機能実装中にエラー（NoMethodError）が発生しました

Question

前提・実現したいこと

ruby on rails 初学者です。
写真投稿アプリを制作中、
いいね機能の実装の際にエラーメッセージが発生しました。

現状、deviseを導入した上で、
likeテーブルを作り、
自分がいいねをしていない場合は「いいね」、
いいねをしている場合は「いいねを取り消す」という処理にしたいです。

こちらの記事を参考にしました。
https://eiji-hb.hatenablog.com/entry/2019/10/23/151224

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

NoMethodError in Posts#index

undefined method `id' for nil:NilClass

該当のソースコード

app/view/posts/index.html.haml

.post_lists
  - @posts.each do |post|
    %div.post_content
      %div.post_content__upper_box
        %div.post_content__upper_box__name
          = link_to post.user.name, user_path(post.user), class: "post_content__upper_box__username"
        - if user_signed_in? && current_user.id == post.user_id
          = link_to "投稿を削除", post_path(post.id), method: :delete, class: "post_content__upper_box__delete"
      = image_tag post.image.url, class: "post_content__image"
      %div.post_content__rower_box
        %div.post_content__rower_box__caption
          = post.text
        %div.post_content__rower_box__like
          -if current_user.likes.find_by(post_id: @post.id)
            = link_to 'いいね！を取り消す', post_like_path(@post), method: :delete
          -else
            = link_to 'いいね！', post_likes_path(@post), method: :post

app/config/routes.rb

Rails.application.routes.draw do
  devise_for :users
  root to: 'posts#index'
  
  resources :users, only: [:index, :show]
  resources :posts, only: [:index, :new, :create, :destroy] do
    resources :likes, only: [:create, :destroy]
  end
end

app/controllers/posts_controller.rb

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

  def index
    @posts = Post.includes(:user).order(created_at: :DESC)
    @like = Like.new
  end

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

  def new
    @posts = Post.new
  end

  def create
    @post = Post.new(post_params)
    if @post.save
      redirect_to root_path
    else
      render :new
    end
  end

  def destroy
    post = Post.find(params[:id])
    post.destroy
    redirect_to root_path
  end

  private
  def post_params
    params.require(:post).permit(:image, :text).merge(user_id: current_user.id)
  end

end

app/controllers/likes_controller.rb

class LikesController < ApplicationController
  before_action :set_post, only: [:create, :destroy]
  def create
    @like = current_user.likes.create(like_params)
    redirect_to root_path(@post)
  end

  def destroy
    @like = Like.find_by(like_params, user_id: current_user.id)
    @like.destroy
    redirect_to root_path(@post)
  end

  private
  def set_post
    @post = Post.find(params[:post_id])
  end

  def like_params
    params.permit(:post_id)
  end
end

app/controllers/users_controller.rb

class UsersController < ApplicationController

  def index
    @users = User.all
  end

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

end

試したこと

app/view/posts/index.html.haml内の条件分岐、「-if current_user.likes.find_by(post_id: @post.id)」以下において、post_idが取れていないことが原因かと考え、routes.rbのネストができているか確認をしてみました。

補足情報（FW/ツールのバージョンなど）

Rails 5.2.4.1
ruby 2.5.1

Answer 1

app/view/posts/index.html.hamlを下記のようにすることで、暫定的にですが、エラーの解消ができました。

.post_lists
  - @posts.each do |post|
    %div.post_content
      %div.post_content__upper_box
        %div.post_content__upper_box__name
          = link_to post.user.name, user_path(post.user), class: "post_content__upper_box__username"
        - if user_signed_in? && current_user.id == post.user_id
          = link_to "投稿を削除", post_path(post.id), method: :delete, class: "post_content__upper_box__delete"
      = image_tag post.image.url, class: "post_content__image"
      %div.post_content__rower_box
        %div.post_content__rower_box__caption
          = post.text
        %div.post_content__rower_box__like
          - like = current_user.likes.find_by(post_id: post.id)
          - if like != nil
            = link_to 'いいね！を取り消す', post_like_path(post,like), method: :delete
          - else
            = link_to 'いいね！', post_likes_path(post), method: :post

Answer 2

controllerのエラーの様な表示ですが多分viewの方ですね。
viewに @post というのがあります。複数形の@postsは定義されていますが単数形は定義されていません。ので　@post.id で掲示のエラーになります。
＠のない、post.id　の間違いだと思います

Comments

[ikdnaht]

ご回答いただきありがとうございます！
おっしゃっていただいた通り、post.idに変更したところ、indexのビューが表示されるようにはなったのですが、いいねを押すと今度は

ActionController::UrlGenerationError in Posts#index　と表示されてしまいます。
```haml
-if current_user.likes.find_by(post_id: post.id)
= link_to 'いいね！を取り消す', post_like_path(post), method: :delete
- else
= link_to 'いいね！', post_likes_path(post), method: :post
```
post_likes_path(post)の記述では、post_idが取ってこれないのでしょうか？

[winterboum]

routes.rb の記述漏れですね。
通常は index　は GET のものなので、そこにPOSTとかDELETEとか入れるならそれなりの宣言が必要です。
あ、
違うかも。routes.rb の問題ではありそうですが

[winterboum]

post_likes_path(post)　→　 post_like_path(post)　かな

[ikdnaht]

rails routesでは、下記のようになっていたので、createアクションにはpost_likes_path(post)をあてていたのですが違いますでしょうか？

post_likes POST /posts/:post_id/likes(.:format) likes#create
post_like DELETE /posts/:post_id/likes/:id(.:format) likes#destroy

routes.rbは、下記のような状態です

Rails.application.routes.draw do
devise_for :users
root to: 'posts#index'

resources :users, only: [:index, :show]
resources :posts, only: [:index, :new, :create, :destroy] do
resources :likes, only: [:create, :destroy]
end
end

[winterboum]

rails routes で
post_likes POST /posts/:post_id/likes(.:format) likes#create
となってます？？

[ikdnaht]

なっています！

[winterboum]

それも、うぅむ〜〜　ですが。。。
それで likes#createに飛んでいないということは /posts/:post_id/likes　になっていないのかな
その部分のhtmlに変換されたところどうなってます？

[ikdnaht]

= link_to 'いいね！', post_likes_path(post), method: :post

としています！

[winterboum]

いえ、それがどうhtmlに変換されたか、　です。

[ikdnaht]

申し訳ありません。検証ツールでは、該当箇所が下記のように表示されています。
どうhtmlに変換されているかとは、こういった意味でしょうか？

<a rel="nofollow" data-method="post" href="/posts/36/likes">いいね！</a>

[winterboum]

それです。href="/posts/36/likes"　になってますね。
この件私の理解を超えているのが２点
1) post_likes_path　と post なのに複数形になっている
2) そのとおりのURLなのに Post#indexに飛ぶ
あと一点確認させていただいて、それで理解不能でしたらごめんなさいです。
rails routes の先頭からpost_likesまでを見せてください

[ikdnaht]

2)に関しては、likes_controllerのcreateアクションにredirect_to root_pathを入れているからでしょうか？いいねをした後に、root_pathのPost#indexページに戻したいという意図で、記述してしまっています。

rails routesこちらです

Prefix Verb URI Pattern Controller#Action
new_user_session GET /users/sign_in(.:format) devise/sessions#new
user_session POST /users/sign_in(.:format) devise/sessions#create
destroy_user_session DELETE /users/sign_out(.:format) devise/sessions#destroy
new_user_password GET /users/password/new(.:format) devise/passwords#new
edit_user_password GET /users/password/edit(.:format) devise/passwords#edit
user_password PATCH /users/password(.:format) devise/passwords#update
PUT /users/password(.:format) devise/passwords#update
POST /users/password(.:format) devise/passwords#create
cancel_user_registration GET /users/cancel(.:format) devise/registrations#cancel
new_user_registration GET /users/sign_up(.:format) devise/registrations#new
edit_user_registration GET /users/edit(.:format) devise/registrations#edit
user_registration PATCH /users(.:format) devise/registrations#update
PUT /users(.:format) devise/registrations#update
DELETE /users(.:format) devise/registrations#destroy
POST /users(.:format) devise/registrations#create
root GET / posts#index
users GET /users(.:format) users#index
user GET /users/:id(.:format) users#show
post_likes POST /posts/:post_id/likes(.:format) likes#create
post_like DELETE /posts/:post_id/likes/:id(.:format) likes#destroy

[winterboum]

すみません、お手上げです。
前の方に似たような宣言があって、そちらに引っ張られてるってのを想定したのですがありませんでした。

[ikdnaht]

そうでうか。
ありがとうございます！エラーに対するアプローチの仕方など、勉強になりました！もう少しコードを見て、調べてみます！