アプリ内でレビューの点数ランキングを作る方法

Question

前提・実現したいこと

railsを使い始めたばかりでレベルの低い質問ですがよろしくおねがいします。
現在、railsを使ってユーザーが観光地を投稿し、それに対してレビューができるアプリを作っています。
そのレビュー内で5点満点で点数をつけて投稿できるようにしているのですが、その点数を使ってランキングを作成したいと考えています。

class Review < ApplicationRecord
  belongs_to :user
  belongs_to :spot
end

class User < ApplicationRecord
    has_many :spots
    has_many :reviews
end

class Spot < ApplicationRecord
    has_many :reviews
    belongs_to :user
end

現在このように関連付けしているのですが、spotに関連付けているreviewからレビューのカラムpointの平均を
それぞれのspotごとに出すにはどうすればいいのでしょうか...?
spotテーブルに平均を入れるカラムを作ってレビューが追加されるたびそのカラムの中身を更新するという方法を考えたのですが、
レビューが投稿されるたびにデータベースを更新するやり方がいいのかわからない点、
コントローラのコードで作る方法がないかと思い質問しました。
初質問のため足りない部分があるかと思いますがよろしくおねがいします。

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

まずSpotのメソッドとして、
平均値を算出するメソッドを定義すれば良いと思います。

reviewのカラムpointの平均値は、

reviewsにおけるpointの合計値／reviewsの個数

でよいかと思います。

reviewsにおけるpointの合計値 と reviewsの個数 の表現方法はわかりますか？

前者は、pluckと配列のsum関数を、
後者は、countを上手く使えばイケルと思います。

Comments

[halu_1113]

---
@average_review = spot.reviews.average(:point)
---
平均はaverageで表示する部分を作るときにしていたので平均の出し方はなんとなくわかるのですが、それをspotレコードごとに計算させて表示するやり方がよくわかりません...すみませんがよろしくおねがいします。

[tashua314]

```
aves = []
Spot.all.each do |spot|
　aves.push spot.reviews.average(:point)
end
```

これでspotレコードごとの平均一覧になるかと思うのですが、違いました？

[halu_1113]

すみません、このコードをコントローラに記入してavesをインスタンス変数にかえてviewでeach doでひょうじするという考え方で大丈夫でしょうか...?
ranks-controller
---
def index
@aves = []
Spot.all.each do |spot|
　 @aves.push spot.reviews.average(:point)
end
end
---

index.html.erb
---
<%= @aves.each do |ave| %>
<%= ave.name %>

<%= end %>
---
とりあえず表示できるかの確認のためviewにこのように書いたのですが、

https://gyazo.com/28bca17b3a7bc2c49a0b52ef8e477c58
エラー表示になってしまいました。内容的にはメソッドに正しいデータが入っていない旨の内容だと思うのですが...何度も申し訳ありませんがよろしくおねがいします。

[tashua314]

全角の空白文字じゃないですかね？

[halu_1113]

全角が入っていました...一応平均値を持ってくことができました！
他の質問で使えそうなものがありそちらのを利用してspotのデータを表示することができました！
何度も質問に答えていただきありがとうございました！

Answer 2

spotmodel内

attr_accessor :average
def average_point
       self.reviews.average(:point) 
    end

コントローラ内

      @spots = Spot.all.each do |spot|
        spot.average = spot.average_point
      end
     
     @spots = @spots.sort_by { |spot| spot.average }
end

作り方は下記質問の回答をもとに作成しました！
この質問の回答

Answer 3

averages = Review.select("sport_id,avg(point) as avg").group(:sport_id) としますと
sport_id 毎の 平均が取れます。
average = average[i]
Sport.find(average.sport_id) で sportが
average.abg でそのpointの平均値が。

ただこのままだと １＋N問題が出てしまうので、ん〜〜〜こうかな
averages = Review.includes(sport).select("sport_id,avg(point) as avg").group(:sport_id)
これで averages[i].sport でsportが取れます。

Comments

[halu_1113]

---
NameError in RanksController#index
undefined local variable or method `spot' for #<RanksController:0x0000000003396068>
---
すみませんやってみたのですがうまくできませんでした。
ここでやっていることがよくわからないのでどういう処理をしているのか教えていただくことはできるでしょうか...よろしくおねがいします

[winterboum]

両方駄目でしたか？２つめの方が、ですか？

[winterboum]

ん？　undefined local variable or method `spot' for #<RanksController:0x0000000003396068>
controller の？
その記述の辺り書き方間違えていないかな。
どういうcodeで行ったかコピペしてください

[halu_1113]

2つ目の方ですね...すみません他のやり方が分かりそちらの方で作ることができました
その時書いたのも消してしまいまして...
作り方の他の例として知りたいのでどのような処理をしているのか教えていただくことは可能でしょうか...?
railsの書き方にまだ慣れておらず、select内の部分がよくわかりませんよろしくおねがいします

[winterboum]

ひとつ目の方で試していただけますか。
それが出来たうえでの２つめですので