質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • Ruby

    8167questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    7657questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

ランキング機能実装 

受付中

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 725

tomoharu

score 26

現状コード:
当方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


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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

0

>>ハッシュデータがレシーバだからメソッドが使えないのかと思ったのですが
その通りです
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問題に対応しました
携帯からの更新なので見苦し買ったらすみません。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/04/15 17:31

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

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

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

    キャンセル

  • 2017/04/15 17:33 編集

    @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|
    して見てください

    キャンセル

  • 2017/04/15 17:52 編集

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

    キャンセル

  • 2017/04/16 00:09

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

    キャンセル

同じタグがついた質問を見る

  • Ruby

    8167questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    7657questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。