0

レシピを投稿した際にいいねの数でランキングを表示しようとしています。
ランキング表示といいね機能はできたのですが
以下のように1位と2位に同じレシピが表示されてしまいます。

原因としては@cookingsの中身が毎回同じものなのでここを解消すればなんとかなると思うのですがその解決方法がわかりません。

何か追加して欲しい情報があれば追加しますのでよろしくお願いします。


現在の表示結果:

画像の説明をここに入力

ソースコード

クッキングコントローラーのrankingアクション

def ranking
    @all_ranks = Cooking.find(Like.group(:cooking_id).order('count(cooking_id) DESC').limit(3).pluck(:cooking_id))
    @cookings = Cooking.order("created_at DESC")
  end

likesコントローラー (いいね機能を実装)

class LikesController < ApplicationController
  def create
    @like = current_user.likes.create(cooking_id: params[:cooking_id])
    redirect_back(fallback_location: root_path)
  end

  def destroy
    @cooking = Cooking.find(params[:cooking_id])
    @like = Like.find_by(cooking_id: @cooking.id)
    @like.destroy
    redirect_back(fallback_location: root_path)
  end
end

ranking.html.erb

<div class="border"></div>
<div class='cooking-card-contents'>
  <h2 class='title'>レシピランキング</h2>
   <% @all_ranks.each.with_index(1) do |cooking, i| %>
      第<%= i %>位
     <%= render partial: "cooking", local:{ cooking: @cookings} %>
  <% end %>
</div>

_cooking.html.erb

<ul class='cooking-lists'>
  <% @cookings.each do |cooking|%>
    <li class='list'>
     <div class="slider">
      <% cooking.images.each do |image| %>
      <div class='cooking-img-content'>
       <%= link_to image_tag(image, class: 'sized'), cooking_path(cooking.id) %>
      </div>
      <% end %>
     </div>
     <div class="cooking-info">
       <h3 class="recipe-name"><%= cooking.cooking_name %></h3>
       <h3 class="recipe-time"><%= cooking.production_time %></h3>
     </div>
    </li>
  <% end %>
</ul>
3
  • ranking.html.erbで、<%= render partial: "cooking", local:{ cooking: @cookings} %>となっているのはどうしてでしょうか?@cookingsの中身は毎回同じなので、これだと毎回同じものがレンダーされてしまいそうです。 Commented 2021年4月29日 13:33
  • そうなんですね、確かにlocalは部分テンプレートで使いたい変数と持ってきたい値を設定するものでした。ですけどどういうふうに書けばよいですかね?
    – user45049
    Commented 2021年4月30日 4:00
  • @cookingsの中身を同じのにならないやり方がわかりません
    – user45049
    Commented 2021年4月30日 9:54

1 件の回答 1

0

まず、_cooking.html.erbについてです。このパーシャルは個別のCookingを表示するためのものですので、内部でeach(繰り返し)を使う必要はありません。

_cooking.html.erb

<li class='list'>
  <div class="slider">
  <% cooking.images.each do |image| %>
  <div class='cooking-img-content'>
    <%= link_to image_tag(image, class: 'sized'), cooking_path(cooking.id) %>
  </div>
  <% end %>
 </div>
 <div class="cooking-info">
   <h3 class="recipe-name"><%= cooking.cooking_name %></h3>
   <h3 class="recipe-time"><%= cooking.production_time %></h3>
 </div>
</li>

ここでの変更は2つ、<ul>タグは個別のアイテムに付けるタグではないので取り除き、@cookings.eachの行を削除しています。

ranking.html.erb

<div class="border"></div>
<div class='cooking-card-contents'>
  <h2 class='title'>レシピランキング</h2>
   <% @all_ranks.each.with_index(1) do |cooking, i| %>
      第<%= i %>位
      <ul>
        <%= render partial: "cooking", locals:{ cooking: cooking} %>
      </ul>
  <% end %>
</div>

こちらには、<ul>タグを追加し、locallocalsに修正し(これはミスっぽいですね)、@cookingscooking@all_ranksの繰り返しで得られるCookingオブジェクト)に修正しています。

これらの変更で想定どおりの動作になると思います。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。