Rails/postgreSQL テーブルをまたいだデータの計算

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
Ruby on Rails (version 5.2.4.2)で評価システムを作っています。
PostgreSQLでデータベースを作っているのですが、テーブル間の計算機能を実装中に以下の計算が実行されないエラーが発生しました。

実行したい計算は、Elements_edu テーブルにある "faculty_student_n"と "faculty_teacher_n"を以下のように計算し、Evaluate_edu　テーブルのraitio_stとしてデータベースに保存することです。なお、db/seeds.rbにてfaculty_student_nとfaculty_teacher_nはすでにデータベースに保存されており、raitio_stはカラムが空の状態です。
(計算式：raitio_st=ElementsEdu.faculty_student_n/ElementsEdu.faculty_teacher_n)

発生している問題・エラーメッセージ

エラーメッセージ：なし

該当のソースコード

class EvaluateEdu < ApplicationRecord
    belongs_to :assessment, optional: true
    has_one :elements_edu

    def raitio_st
        @evaluate_edus = EvaluateEdu.all

        @evaluate_edus.each do |ratio_st|
            ratio_st = ElementsEdu.faculty_student_n/ElementsEdu.faculty_teacher_n
            EvaluateEdu.raitio_st.update()
        end
    end
end

class ElementsEdu < ApplicationRecord
    belongs_to :evaluate_edu, optional: true
end

試したこと

・def ... end 文の追加
・pメソッドを使ってどのように値が表示されるかの確認。
・テーブルの初期化

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

使用PC: Macbook Pro
環境: Visual Studio Code, Ruby on Rails 5, PostgreSQL

Answer 1

EvaluateEduモデルで一括で各レコードのratio_stをupdateしたいということだと思いますので、、、

EvaluateEduモデル

def self.update_all_ratio_st
  all.each do |evaluate_edu|
    evaluate_edu.update(ratio_st: evaluate_edu.calc_ratio_st)
  end
end

def calc_ratio_st
  elements_edu.faculty_student_n / elements_edu.faculty_teacher_n
end

実行は、EvaluateEdu.update_all_raitio_st です。

でどうでしょうか。

Comments

[shayashi683]

ご回答ありがとうございます！そして、返信が遅くなり申し訳ありません。
EvaluateEduモデルにいただいたコードを入れたのですが、「実行」というのはterminalに'EvaluateEdu.update_all_raitio_s'を入れるということでしょうか？今のところ、rails db:migrateを実行してもevaluate_eduテーブルのratio_stには計算結果は表示されておりません。
ここの計算で開発がずっととまっておりますので、ご回答していただき大変助かっております。ありがとうございます。

[Cojiro]

rails db:migrate を実行しても反映はされないでしょう。

> faculty_student_nとfaculty_teacher_nはすでにデータベースに保存されており
という前提です。
ターミナルでもいいですし、ソースに書いてもいいです。

[shayashi683]

faculty_student_nとfaculty_teacher_nはすでにデータベース(elements_edu テーブル)に保存されております。
理解が悪くてすいません。ソースに書くというのは、EvaluateEduモデルに書くということでしょうか？このように書いてみたのですが、
```
def self.update_all_ratio_st
all.each do |evaluate_edu|
evaluate_edu.update(ratio_st: evaluate_edu.calc_ratio_st)
end
end

def calc_ratio_st
self.faculty_student_n / self.faculty_teacher_n
end

EvaluateEdu.update_all_ratio_st
```

[Cojiro]

こちらはメソッドを提案しているので、
モデルでもコントローラーでもターミナルでも好きなところでそのメソッドを利用してください。

何がしたいかが明確に記載されていないので、
回答の冒頭で
> EvaluateEduモデルで一括で各レコードのratio_stをupdateしたいということだと思いますので、、、
とさせていただいておりましたが、この部分が違うということですか？
違うのであれば、何がしたいかを今一度提示していただけますか？

[shayashi683]

現在、私が困っているところは以下の2つのテーブルに関するものです。
テーブル: evaluate_edu, カラム: ratio_st(現在null)
テーブル: elements_edu, カラム: faculty_student_n, faculty_teacher_n (seedsファイルでデータは入っている)
このelements_eduテーブルの値を計算、
evaluate_edu.ratio_st=elements_edu.faculty_student_n/faculty_student_n.faculty_teacher_n
そして、evaluate_eduテーブルに保存させたいです。

[Cojiro]

それは質問に書いてることですよね？

それを下記のように解釈したのですが、違いますか？
> EvaluateEduモデルで一括で各レコードのratio_stをupdateしたいということだと思いますので、、、

[shayashi683]

その通りです。
となりますと、やはりいただいたコードでいいということでしょうか？
コードにエラーは出ませんが、EvaluateEduテーブルに保存するための実行方法が分からないです。

[Cojiro]

使う方法は先述の通りです。
Rails consoleでもコントローラーでも、他モデルでもという感じです。
これ以上に説明のしようがありません。。。
※ あなたの環境を持っているわけではなく、動作確認はしてないので、エラーが出るものに関しては申し訳ないです。

もしかしたらクラスメソッドが初めてでイメージつかないのかもしれないですね。
インスタンスメソッドとクラスメソッドの違いは下記などを読んでください。(ぐぐるといっぱい出てきます。)
https://qiita.com/right1121/items/c74d350bab32113d4f3d

[shayashi683]

seedファイルにEvaluateEdu.update_all_raitio_st を記載して実行することができました。　本当にありがとうございます!!