Rails １対多関係にあるテーブルカラムの値を取り出し、合算したものをViewで表示させたい

Question

##【実現したい事】
①１対多の関係にあるテーブルがあり、子関係にあるテーブルのカラムの値を取り出したい
②取り出したものをsum関数などで計算し、合算した値を、View上で表示させたい（＝赤枠部分の合計練習時間を表示させたい）

##【環境】
※未経験からエンジニア転職を目指してオリジナルアプリを開発中の者です（学習期間約2ヶ月程度）。
・Ruby：2.6.5
・Rails:5.2.4.3
・DB設計 ※詳細は下記コード
親：Recordテーブル
子：Practiceテーブル

##【試した事】
①Controllerのindexアクションに、total_practice_timeという新しい変数を宣言し、子レコード（Practiceテーブル）モデルの配列「:practices」を代入させる

②次の処理で、ブロック引数|total|を定めて、格納されているpractice_timeの値を全て足し算したものを結果として取り出す処理をする

しかし、「Syntax error」と出ており、何をどう直すべきか分からないため、ご意見・アドバイスを頂戴できますと幸いです。

class RecordsController < ApplicationController
before_action :authenticate_user!

  def index
    @records = current_user.records.includes(:practices).page(params[:page]).per(8)
    total_practice_time = Record.includes(:practices)
      total_practice_time.each do |total|
        total.sum(:practice_time)
      end
    @q = Record.ransack(params[:q])
    @search_records = @q.result(distinct: true)
  end

  def show
    @record = Record.find(params[:id])
  end

  def new
    @record = Record.new
    output = @record.outputs.build
    practice = @record.practices.build
    task = @record.tasks.build
  end

  def create
    @record = Record.new(record_params)
    logger.info "###### #{@record.inspect}"
    if @record.save
      flash[:success] = "練習内容の登録が完了しました。"
      redirect_to records_url
    else
      flash[:alert] = "登録に失敗しました。"
      render :new
    end
  end

  def edit
    @record = Record.find_by(id: params[:id])
  end

  def update
    @record = Record.find_by(id: params[:id])
    if @record.update(record_params)
        flash[:success] = "練習内容の更新が完了しました。"
        redirect_to records_url
      else
        flash[:alert] = "更新に失敗しました。"
        render :edit
      end
  end

  def destroy
    record = Record.find_by(id:params[:id])
    record.destroy

    redirect_to root_path, notice: "練習記録を削除しました。"
  end

  def aggregate_result
    @record = Record.find(params[:id])
    gon.data = Record.where(params[:practice_time])
    6.times do
      # gon.data << rand(100.0)
    end
  end

  private

  def set_user
    @user = current_user || User.new
  end

  def record_params
    params.require(:record).permit(:record_id, :training_date, :learning_point, outputs_attributes:[:output_name, :id], practices_attributes:[:practice_item, :practice_time, :id], tasks_attributes:[:task_name, :id]).merge(user_id: current_user.id)
  end

end

h1 練習記録一覧
.container
  .row
    .col-sm-6
      .form-group
        = form_with model: @record do |f|
          = f.date_field :training_date

table.table-hover.table-respnsive
  thead
    tr
      th 練習日
      th 登録日時
      th 総練習時間
      th
      th
  tbody
    - @records.each do |record|
      - record.practices.each do |practice|
        tr
          td= link_to record.training_date, record_path(record)
          td= record.created_at
          td= total
          / td= practice.practice_time
          td
            = link_to '編集する', edit_record_path(record), class: 'btn btn-primary mr-3'
            = link_to '削除する', record_path(record), method: :delete, data: { confirm: "練習記録を削除します。よろしいですか？" }, class: 'btn btn-danger'

##【コード】

class Record < ApplicationRecord
  validates :learning_point, presence: true
  validates :training_date, presence: true

  belongs_to :user
  has_many :practices, dependent: :destroy
  accepts_nested_attributes_for :practices
end
Practiceモデル

class Practice < ApplicationRecord
  validates :practice_item, presence: true
  validates :practice_time, presence: true
  belongs_to :record
end

ActiveRecord::Schema.define(version: 2020_05_25_064157) do

  create_table "records", force: :cascade do |t|
    t.string "user_id"
    t.text "learning_point"
    t.date "training_date"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

  create_table "practices", force: :cascade do |t|
    t.string "practice_item"
    t.integer "practice_time"
    t.bigint "record_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.index ["record_id"], name: "index_practices_on_record_id"
  end

#エラーメッセージと画像
~~再開したところ、Syntaxerrorでした。
~~
→indexアクションに、1つ余分にendが入っていたことに気づいたので修正しました。すると、下記のログと画像を確認しました。

sumメソッドが提示されていませんと怒られています。。。

Started POST "/__better_errors/c4bbb4c567585e78/variables" for ::1 at 2020-06-16 22:19:03 +0900
Started GET "/records?utf8=%E2%9C%93&q%5Btraining_date_eq%5D=&commit=%E7%B7%B4%E7%BF%92%E6%97%A5%E3%81%A7%E6%A4%9C%E7%B4%A2" for ::1 at 2020-06-16 22:19:04 +0900
Processing by RecordsController#index as HTML
  Parameters: {"utf8"=>"✓", "q"=>{"training_date_eq"=>""}, "commit"=>"練習日で検索"}
  User Load (0.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 ORDER BY "users"."id" ASC LIMIT $2  [["id", 3], ["LIMIT", 1]]
  ↳ /Users/user/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-5.2.4.3/lib/active_record/log_subscriber.rb:98
  Record Load (0.3ms)  SELECT "records".* FROM "records"
  ↳ app/controllers/records_controller.rb:7
  Practice Load (0.5ms)  SELECT "practices".* FROM "practices" WHERE "practices"."record_id" IN ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10, $11, $12, $13, $14, $15, $16, $17, $18, $19, $20, $21, $22, $23, $24, $25, $26, $27, $28, $29, $30, $31, $32, $33, $34, $35, $36, $37, $38, $39, $40, $41)  [["record_id", 1], ["record_id", 2], ["record_id", 3], ["record_id", 4], ["record_id", 5], ["record_id", 6], ["record_id", 7], ["record_id", 8], ["record_id", 9], ["record_id", 10], ["record_id", 11], ["record_id", 12], ["record_id", 13], ["record_id", 14], ["record_id", 15], ["record_id", 16], ["record_id", 17], ["record_id", 18], ["record_id", 19], ["record_id", 23], ["record_id", 24], ["record_id", 25], ["record_id", 26], ["record_id", 27], ["record_id", 28], ["record_id", 29], ["record_id", 30], ["record_id", 31], ["record_id", 32], ["record_id", 33], ["record_id", 34], ["record_id", 35], ["record_id", 36], ["record_id", 37], ["record_id", 38], ["record_id", 39], ["record_id", 40], ["record_id", 41], ["record_id", 62], ["record_id", 63], ["record_id", 64]]
  ↳ app/controllers/records_controller.rb:7
Completed 500 Internal Server Error in 38ms (ActiveRecord: 9.7ms)



NoMethodError - undefined method `sum' for #<Record:0x00007ffaf6c1acc0>:
  app/controllers/records_controller.rb:8:in `block in index'
  app/controllers/records_controller.rb:7:in `index'

Started POST "/__better_errors/28783482a38955a1/variables" for ::1 at 2020-06-16 22:19:04 +0900
  Record Load (0.3ms)  SELECT  "records".* FROM "records" WHERE "records"."user_id" = $1 LIMIT $2 OFFSET $3  [["user_id", "3"], ["LIMIT", 8], ["OFFSET", 0]]
  ↳ /Users/user/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-5.2.4.3/lib/active_record/log_subscriber.rb:98
  Practice Load (1.2ms)  SELECT "practices".* FROM "practices" WHERE "practices"."record_id" IN ($1, $2, $3)  [["record_id", 62], ["record_id", 63], ["record_id", 64]]
  ↳ /Users/user/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-5.2.4.3/lib/active_record/log_subscriber.rb:98
  CACHE Record Load (0.0ms)  SELECT  "records".* FROM "records" WHERE "records"."user_id" = $1 LIMIT $2 OFFSET $3  [["user_id", "3"], ["LIMIT", 8], ["OFFSET", 0]]
  ↳ /Users/user/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-5.2.4.3/lib/active_record/log_subscriber.rb:98
  CACHE Practice Load (0.0ms)  SELECT "practices".* FROM "practices" WHERE "practices"."record_id" IN ($1, $2, $3)  [["record_id", 62], ["record_id", 63], ["record_id", 64]]
  ↳ /Users/user/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/activerecord-5.2.4.3/lib/active_record/log_subscriber.rb:98

##最初に質問していた時のエラーメッセージと画像

Started GET "/" for ::1 at 2020-06-16 21:56:55 +0900

SyntaxError - syntax error, unexpected end, expecting end-of-input:
  app/controllers/records_controller.rb:80:in `'

Started POST "/__better_errors/cbfc53cbe6b50718/variables" for ::1 at 2020-06-16 21:56:56 +0900
Started GET "/" for ::1 at 2020-06-16 21:56:57 +0900

SyntaxError - syntax error, unexpected end, expecting end-of-input:
  app/controllers/records_controller.rb:80:in `'

Started POST "/__better_errors/66effe8f5f8cef8c/variables" for ::1 at 2020-06-16 21:56:57 +0900

Comments

[winterboum]

そのエラーメッセージをを載せてください。
回答に重要な情報が詰まってますので

[ikutyan46]

早速ありがとうございます！！先ほどエラーメッセージと画像添付しました。念のためご確認頂ければ幸いです。

[winterboum]

そのエラーは endのとじ方がおかしい、というものですので、controller全体がないとわからないですね

[ikutyan46]

ご丁寧にありがとうございます。Syntax errorは、indexアクションの「end」を1つ多くつけていたために発生していたことを確認しました。

それを消して新しいエラー（No method）が発生しましたので、Controllerも全部載せて変更しました。恐れ入りますが、今一度ご確認頂ければ幸甚です。

[winterboum]

そこで、回答の所に進みます

Answer 1

方向は合っていそうです。
ただ、変数の命名は注意しましょう。
名前から何であるのかがわかるようにする。
勘違いしないようにする
total_practice_time = Record.includes(:practices)
得られるものは （複数の)Practceですから、
practices = Record.includes(:practices)
に。
total.sum(:practice_time)のsumはArrayにならありますが、totalすなわちPracticeに定義してありますか？多分無いですよね
sum(:practice_time)の書き方はSQL関数っぽい。。SQLで求めさせることもできますが、ruby、RAILSのわざと言うよりSQLの技になるのでそれはあとにまわしてruby、RAILSでやるとすると

total_practice_time = 0
practices.each{|practice| total_practice_time += practice.practice_time }

というのをスマートに書くと

total_practice_time = 
  practices.inject(0){|sum,practice| sum + practice.practice_time }

ただこれですと全員の合計になってしまいますね。
（というところまで進めてから、より欲しい機能に修正しましょう）

Comments

[ikutyan46]

winterboumさん、ありがとうございます。一度、やってみます＾＾