rails ユーザーごとの合計値算出

Question

現在、ゴルフスコア管理アプリをrailsにて制作中なのですが、データベースとのやりとりについて、わからなくってきましたので、質問させて下さい。
まずやりたいこととしては、各ユーザーのトータルスコアを算出し、そのスコアとユーザ名をスコア順に並べて表示させるものです。
Scoreというテーブルに複数ユーザのスコアが入力されており、トータルを算出するも全てのユーザのスコアが合計されてしまいます。

ちなみにScoreテーブルには、
hole_score(そのホールのスコア)
par（そのホールのPar）
user_id
hole(ホール)
のカラムがあります。

controller

@total_score =Score.all.sum(:hole_score)

view

<%= @total_score  %>

で、テーブル全体の合計は出せたのですが、ユーザごとにという方法がわからず困っております。
また、もうひとつよくわからない点として、カラムを指定するのは、controller側なのか、view側なのか、
どちらが望ましいのでしょうか。
たとえば、

@scores = Score.all (controller)
<%= @scores.sum(:hole_score) %> (view)
と、
@total_score =Score.all.sum(:hole_score) (controller)
<%= @total_score %> (view)
は結果同じになると思いますが、どちらが良いのでしょうか。

ヒントやアドバイスを宜しくお願い致します。

Answer 1

ユーザごと
score だけで良いのでしたら
scores = Score.group(:user_id).select("user_id,sum(score) as sum_score")
とすると

scores[?].user_id
scores[?].sum_score
で結果が得られます

カラムを指定するのは、controller側なのか、view側なのか
場合によりますが、できるだけ controllerにするのがよいと思います。
場合ですが、
@score　から取るのが total_score　だけなら controllerが良いですが、
その他にも色々取るのだ、となると viewのほうが良いです。

「できるだけ」というのは、viewってごちゃごちゃになりやすいので、できるだけ計算は済ませておいてすっきり書きたいです

Comments

[hurousyotoku500]

ご回答ありがとうございます。
すみません、色々やってみたのですが、いまいち使い方がわからず…
scores[?].user_idの[?]はどういった値がはいるのでしょうか。
また、scores[?].user_idのは全てのユーザーを表示させるにはeach.doを使うのでしょうか？

[winterboum]

配列の0,1,2,3とかです。
scores.each で全部を。
sum(score)　ここ　sum(hole_score) ですね

[hurousyotoku500]

<% @sum_score.each do |score| %>
<p><%= score.user_id %></p>
<p><%= score.sum_score %></p>
<% end %>

このコードでuser_idとsum_scoreは表示されるようになりましたが、コード的には問題ないでしょうか。いまいちこれであっているのか自信がもてなくて…

[winterboum]

合ってます

Answer 2

ユーザごとにという方法がわからず困っております。

ユーザーごとに合計を出すのであれば下記の様にgroup byを使う感じになると思います。

例えば

Score.all.group(:user_id).sum(:hole_score)

みたいにすればユーザーごとのhole_scoreの合計が取れると思います。

カラムを指定するのは、controller側なのか、view側なのか、

どちらが望ましいのでしょうか。

ユーザーそれぞれの合計スコアを表示するのであれば　自分の場合は

controllerに

@user_total_scores = Score.all.group(:user_id).sum(:hole_score)

みたいに書いて

viewに

@user_total_scores.each do |user_total_score|
// 表示したいものを書く
end

みたいな感じにすると思います。

Comments

[hurousyotoku500]

ご返答ありがとうございます。
コントローラーを、
@user_total_scores = Score.all.group(:user_id).sum(:hole_score)
とし、
ERBを
<%= @user_total_scores.each do |user_total_score| %>
<p> <%= @user_total_scores %> </p>
<% end %>
として、
{1=>82, 2=>108}

{1=>82, 2=>108}

{1=>82, 2=>108}

のように表示されました。
userは全部で３個あるので、３回表示されているのだと思いますが、
なぜ、ハッシュ？で表示されてしますのでしょうか。

[winterboum]

Score.all.group(:user_id).sum(:hole_score)　という書き方の場合は
{ user_id => 合計, user_id => 合計 } という結果が返るのが仕様です