いいね機能の実装について

Question

errorを解決していいねボタンを表示させたいです

Hamlでいいね機能を組み込んだシステムを作っています。
いいね機能を実装中に以下のエラーメッセージが発生しました。

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

NoMethodError in Halls#index
Showing /Users/yuuka/projects/wedding-app/app/views/likes/_like.html.haml where line #2 raised:

undefined method `like_user' for #<Hall::ActiveRecord_Relation:0x00007fd0d2843c80>
Extracted source (around line #2):
              
１- if user_signed_in?
２  - if halls.likes_user(current_user.id)

該当のソースコード

# app/models/hall.rb
class Hall < ApplicationRecord
  belongs_to :user
  has_many :likes, dependent: :destroy

  def like_user(user_id)
    likes.find_by(user_id: user_id)
  end
end

# views/likes/_like.html.haml

- if user_signed_in?
  - if halls.like_user(current_user.id)
    .hall__name__dislike
      =link_to user_hall_like_path(current_user, hall, like), method: "DELETE", remote: true do
        %i.fas.fa-heart
        %span いいね！
        %span 
          = h.likes_count
  - else
    .hall__name__like
      =link_to user_hall_likes_path(current_user, hall), method: "POST", remote: true do
        %i.fas.fa-heart
        %span いいね！
        %span
          = h.likes_count
-else
  .hall__name__like
    %i.fas.fa-heart
      %span いいね！
      %span 
        = h.like_count

# halls_controller

class HallsController < ApplicationController
  def index
    @halls = Hall.all
  end

end

# views/halls/index.html.haml

hall
  - if @halls.present?
    - @halls.each do |h|
      .hall__name
        = h.name
        = render partial: 'likes/like', locals: { halls: @halls, likes: @likes, like: @like}
      .hall__image
        = image_tag h.image
      .hall__place
        = h.place
      .hall__link
        = link_to top_page_index_path(current_user.id) do
          詳細をみる

# controller/likes_controller
class LikesController < ApplicationController

  def create
    @like = Like.create(user_id: current_user.id, hall_id: params[:hall_id])
    @likes = Like.where(hall_id: params[:hall_id])
    get_hall
  end

  def destroy
    @like = Like.find_by(user_id: current_user.id, hall_id: params[:hall_id])
    @like.destroy
    @likes = Like.where(hall_id: params[:hall_id])
    get_hall
  end

  def get_hall
    @hall = Hall.find(params[:hall_id])
  end

end

試したこと

• if halls.like_user(current_user.id)

hallsの中には@halls.each do |h|の情報が入っており、
current_user.idも情報が入っていますがエラーが起きます
like_userのなかに情報が入ってくれないのではないか？

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

このサイトを真似してみました
https://qiita.com/haruya_hamasaki/items/75d41a1aafb87408737d

Comments

[aiandrox]

views/likes/_like.html.hamlを読み込んでいる元のビューファイルと、halls_controllerのファイルを見せていただきたいです。

[退会済みユーザー]

修正しました。

Answer 1

undefined method like_user' for #<Hall::ActiveRecord_Relation..`
となっています。
おそらく controller で　where で絞り込むか all でとり込むかした値がそのまま view の　hall　に渡っているのだと思います。（ActiveRecord_Relationとなってる)
find_by で取るか、もしくは　first(一つしか無いか、最初ので良いかなら）してHallのインスタンスにしてください

追記
indexであるなら、複数ですから

# CONTROLLER
def index
  @halls = Hall.all # params[:id]があるんですか？このidは何のid?　それに応じたwhereにする 
end

# views/halls/index.html.haml
.hall
  - if @halls.present?
    - @halls.each do |h|

で問題は　likes/like です。
locals: { hall: @hall, halls: @halls, likes: @likes, like: @like}
と沢山渡してますがこの中で定議されているのはひとつだけ。
このpartialはindexの中で何回呼ばれるのでしょうか？
index.htmlを見た限りでは一回だけです。すると沢山ある hall のそれぞれに沢山あるlikeのどれをdeleteするのですか？

Comments

[退会済みユーザー]

回答いただきありがとうございます。
find_byで取り込んで見た所
views/halls/index.html.hamlのeachでエラーが出たので- if @hall.present?としてみたのですが
for nil:NilClassと表示されてしまいます。- if @hall.present?と言うコードがおかしいでしょうか？
度々、すいません。

[winterboum]

indexだったんだ、すると複数個ある可能性があるのですから、@halls と複数形にしてください。文法的には間違いではないですが、人間が読むと一つのHallを表していると勘違いして、私の回答の様になってしまいます。
回答に追記します

[退会済みユーザー]

＠hallsに変更しました。
nil エラーに変わりました。
一番下にlikes_controllerファイルを追加しました。
こちらの変数を渡そうとしています。

[winterboum]

そのnilエラーのメッセージ載せてください。
どこでなにがnilかわからないと何もわからんです

それと回答に追記してありますが、最後の所読みました？

[退会済みユーザー]

修正しました。
読みました。一回だけです。
初心者で質問の理解がきちんとできず申し訳ないです。

[winterboum]

1つだけとすると、
どのhallのどのlikeを対象とするのでしょう？？？

[退会済みユーザー]

index.html.hamlの.hallの情報を対象としています。
この答えであってますでしょうか

[winterboum]

ぜんぜん。
indexでは複数のhallがありますよね？
そのどれを対象にするのでしょう

[退会済みユーザー]

views/halls/index.html.hamlの.hallに作られた情報に対していいねをつけたいので
このhallの一つずつにつけたいです。
複数あるホール情報全てに一つずつつけたいです。

[winterboum]

それを一つの「%span いいね！」でやるということは、一回押すと全部にいいねが着くのですか？

[退会済みユーザー]

つきません。イメージはtweeterのようなツイートがhallでその一つに対していいねするようにしたいです。
views/halls/index.html.hamlの.hallがそのツイートの部分にあたります

[winterboum]

すると、いいねが着くのはどのhallですか？

[退会済みユーザー]

views/halls/index.html.hamlファイルのこの部分です

.hall
- if @hall.present?
- @hall.each do |h|
.hall__name
= h.name
.hall__image
= image_tag h.image
.hall__place
= h.place
.hall__link
= link_to top_page_index_path(current_user.id) do
詳細をみる

[winterboum]

@hall.each　ってあるように複数です。
この中のどのhollを対象にするのか、を聞いてます

[退会済みユーザー]

この全部を対象にしたいですが、
絞るのであれば.hall__nameのみにつけることができれば嬉しいです。

[winterboum]

全部を対象にするなら、一つ一つに
render partial: 'likes/like',
をつける必要があります

[退会済みユーザー]

nameのみを対象にするのであればnameにつければよいということですよね？
やってみます！！

[winterboum]

？？？

[winterboum]

なんか話が通じない。
このテーマをやるのはまだむずかしいのでは。
ruby、rails、object指向をもう少し勉強してみてください。
チュートリアルとかはやりましたか？

[退会済みユーザー]

nameを対象にするのは難しいですか？
チュートリアルはしてません。勉強しようと思います。
エラー文を修正するのでみていただけますか？

[winterboum]

エラー文は修正しないでそのまま載せるようにしてください