ランキング機能実装

Question

現状コード：
当方rails 初心者です。

やりたいこと：
現在、自分が描いた漫画を投稿して、みんなで読めるようなwebアプリを作成しています。
そして、その漫画一つ一つにいいねをつけることができ、そのいいねの数によってランキングされるという機能を実装中です。

エラーコード： index.html.erbで発生
undefined method `front_cover' for [1, 1]:Array

現状コード：

#Comics_controller
class ComicsController < ApplicationController
  before_action :authenticate_user! ,only: [:show, :search]
  def index
    like_ranking = Like.group(:comic_id).count
    @comic_ranking = like_ranking.sort_by{|_, v| -v }
  end
end

#Likes_controller
def create
    @comic_liked = Comic.find(params[:id])
    @comic_liked.likes.find_or_create_by(user_id: params[:user_id]) 
    @comic_liked.update(like: @comic_liked.like+1)
    redirect_to :controller => 'comics', :action => 'index'
 end

#index.html.erb
<% @comic_ranking.each do |like| %>
   <tr>
      <td><%= image_tag like.front_cover.to_s %></td>
      <td><%= like.title %></td>
      <td><%= like.author %></td>
   </tr>
  <% end %>
<table>
  <thead>
   <tr>
    <th>作品表紙</th>
    <th>作品タイトル</th>
    <th>作者</th>
   </tr>
  </thead>

#Comic.rb
class Comic < ActiveRecord::Base
  has_many :users, through: :likes
  has_many :likes, dependent: :destroy
  mount_uploader :front_cover, FrontCoverUploader
end

#Like.rb
class Like < ActiveRecord::Base
  belongs_to :comic, counter_cache: :like
  belongs_to :user
end

質問：エラーコード解消法を教えてください。
ハッシュデータがレシーバだからメソッドが使えないのかと思ったのですが、お時間がある時にお聞かせ願えればと思います。足りないコードがあればすぐに対応いたします。

宜しくお願い申し上げます。

Answer 1

ハッシュデータがレシーバだからメソッドが使えないのかと思ったのですが

その通りです
comicだったものをlikeに置き換えたところでそのまま動くわけがありません
そもそも@comic_rankingは[comic_id,like_count]という配列の配列であって
comicで表されていたActiveRecordとは全然違うものです。
def index
@comic_ranking=Comic.joins(:likes).references(:likes).group(:comic_id).order("count(likes.id dsec"))
end

<% @comic_ranking.each do |comic|%>
   <tr>
      <td><%= image_tag comic.front_cover.to_s %></td>
      <td><%= comic.title %></td>
      <td><%= comic.author %></td>
   </tr>
  <% end %>
<table>
  <thead>
   <tr>
    <th>作品表紙</th>
    <th>作品タイトル</th>
    <th>作者</th>
   </tr>
  </thead>

とりあえずこれで、エラーは出ませんが、n+1問題が発生してコミックが増えた時、非常に重くなるでしょう。

N+1問題に対応しました
携帯からの更新なので見苦し買ったらすみません。

Comments

[tomoharu]

ご回答ありがとうございます。言われた通りにやってみたのですが、ランキングされず、特定のコミックしかランキング画面に出てこないようになってしまいました。
変更したものは以下です。
<%@comic_ranking.each do |comic_id, like_count| %>
<% comic = Comic.find_by(comic_id) %>
お時間がある時にご教授いただければと思います。

補足情報として、likeテーブルの中で、comic_idの数が一番多いコミックがランキング上位にしたいと思っています。

何卒よろしくお願いいたします。

[moke]

@comic_ranking = like_ranking.sort_by{|_, v| -v }
があっているかは検証していないのですが...
とりあえず。comic_id,like_countが入れ替わっているかもしれないので。
@comic_ranking.each do |comic_id,like_count|
を
@comic_ranking.each do |like_count,comic_id|
して見てください

[moke]

回答を修正しました、内容が変わっています。
一応両方試して下さい

[tomoharu]

ありがとうございます。やってみます。