計算値に元づいて、レコードをソートしたいが、orderメソッドではエラーが発生する

Question

<<私のスキルレベルや開発環境は最下部に掲載>>

【やりたいこと】

要約

Railsでアソシエーション先のテーブルのデータを用いて計算し、その計算値を元にソートを行う

具体的には…

4つのテーブルがあります。
Songsテーブル
Chordsテーブル
Practicesテーブル
Usersテーブル


※今回、関係ないカラム名は省略

Songsレコードのソートを行います。
計算値に基づいてdesc順に並べます。

計算値の求め方は次の通り

Songレコードの孫に当たるPracticeレコードの数を総計して算出。
ただしuser_idが同じものは重複カウントしない。

下図の例でいうと、practiceレコードの総計は5ですが、user_id:2が重複しているで、理想的な計算結果は4です。

計算自体は下記の記述でうまくいっています。

result = song.chords.joins(:practices).distinct.count('practices.user_id')
```

計算値は仮想属性(Virtual Attribute)に代入。

```Ruby on Rails
# Songモデル　抜粋
class Song < ApplicationRecord
  attribute :song_practices, :integer, default: 0
  attr_accessor :song_practices
end

計算と結果の代入

song[:song_practices] = song.chords.joins(:practices).distinct.count('practices.user_id')

【問題】

@songs = @songs.order("song_practices desc")

orderメソッドを用いたソートを行おうとするが、エラー。

エラー文
「Mysql2::Error: Unknown column 'song_practices' in 'order clause': SELECT songs.* FROM songs ORDER BY song_practices desc」

次のように認識しました
→テーブルにそんなカラムはないよエラー

エラー検証

下記のコントローラにて、動作テストを実施。
binding.pryにて”check_point!!”で一時停止して変数の状態を確かめた

# Songsコントローラ　抜粋

# searchは検索機能のためのアクションです。
def search

    # 今回は関係がない記述(検索機能のための記述)
    params[:keyword].strip!
    keywords = params[:keyword].split(/\s+/)

    # songsレコードの呼び出し
    @songs = Song.all.includes(:chords)

   # 条件文 今回はif条件がtrueになるようにparamsの値が送られています。
    if params[:sort] == "practice"
      # 各songレコードごとに孫として紐づくpracticeの数を算出
      @songs.each do |song|
	    # 計算、同時に仮想属性に代入
        song[:song_practices] = song.chords.joins(:practices).distinct.count('practices.user_id')
      end

      # check_point!! ←ここでbinding.pry

      # orderにてソート。
      @songs = @songs.order("song_practices desc")
      
    else
      # 今回は関係がない記述(title順にソートする機能)
      @songs = @songs.order("title asc")
    end

    # 以下、今回は関係ない記述(検索機能のための記述)
    @songs = @songs.where(jam: params[:jam])  if (params[:jam] == "true")
    @songs = @songs.where(standard: params[:standard])  if (params[:standard] == "true")
    @songs = @songs.where(beginner: params[:beginner])  if (params[:beginner] == "true")
    @songs = @songs.where(vocal: params[:vocal])  if (params[:vocal] == "true")
    @songs = @songs.where(instrumental: params[:instrumental])  if (params[:instrumental] == "true")

    keywords.each do |keyword| unless (params[:keyword].nil?)
      @songs = @songs.where("title like ?", "%#{keyword}%")
    end

    respond_to do |format|
      format.html
      format.json
    end
    
  end

check_point!!時点における変数の状態は下記の通り

@songs

 [#<Song:0x00007fb29bb44c90
  id: 204533774,
  title: "Little Cabin Home on the Hill",
  jam: false,
  standard: true,
  beginner: false,
  user_id: 1,
  created_at: Mon, 25 May 2020 13:37:46 JST +09:00,
  updated_at: Mon, 22 Jun 2020 20:31:35 JST +09:00,
  vocal: false,
  instrumental: false>,
 #<Song:0x00007fb29bb449e8
  id: 307520937,
  title: "Foggy Moutain Breakdown",
  jam: true,
  standard: true,
  beginner: true,
  user_id: 1,
  created_at: Mon, 25 May 2020 13:37:46 JST +09:00,
  updated_at: Fri, 26 Jun 2020 23:22:29 JST +09:00,
  vocal: false,
  instrumental: true>,

// 以下続く

※仮想属性：song_practiceは含まれていません。

@songs[0][:song_practices]

@songs[0][:song_practices] = [“理想的な計算結果※”]

エラーは発生しませんでした
（仮想属性の記述をモデルから削除すると、can't write unknown attribute song_practices=「そんな属性存在しないよエラー」が発生 ）

※"計算結果の妥当性を示すデータ"は読みづらさの回避のため、とりあえずは掲載しません。必要あればおっしゃってください。

【ほかに試したこと】

仮説

orderメソッドが呼び出されたときの挙動 が下記のようであるため、「DBにはそんな属性存在しないよエラー」が発生する
→
①モデルのテーブルのうち、指定されたカラムを探す
②見つけたカラムのデータを用いてソート

従って…
→テーブルにカラムが存在していることが前提。

対応策

①ソートのアルゴリズムをインスタンスメソッドとして定義
②定義したメソッドを用いてソートを実施

対応策用のメソッド実装前に…
→インスタンスメソッドの定義・利用ができるかテスト

問題

先程と同様にcheck_point!!にてインスタンスメソッドの試用。

テスト用のインスタンスメソッドの定義

# Songモデル　抜粋
def test_song
    puts "test#{title}"
end

試用結果

①
@songs[0].test_song

test -> Little Cabin Home on the Hill
=> nil

→問題なく動作

②
@songs.test_song

NoMethodError: undefined method `test_song' for #<Song::ActiveRecord_Relation:0x00007fb2a5b1d230>
from /Users/imotoshouta/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/activerecord-5.2.4.2/lib/active_record/relation/delegation.rb:125:in `method_missing'

→エラー。

※メソッド自体が呼び出されていないのか、中の記述が問題なのか確かめるために③を実施

③
puts "test -> #{@songs.title}"

NoMethodError: undefined method `title' for #<Song::ActiveRecord_Relation:0x00007fb2a5b1d230>
from /Users/imotoshouta/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/activerecord-5.2.4.2/lib/active_record/relation/delegation.rb:125:in `method_missing'

→当然、エラー。
メソッド自体が呼び出された場合は、②でも同じエラーが出ると推測される。
しかしエラーの内容が異なる。
よって②においてはメソッド自体が呼び出されていない

推察

@songsと@songs[0]の違い
@songs→連想配列/インスタンス変数の配列
@songs[0]→ただのハッシュ/インスタンス変数単体

→連想配列に対してインスタンスメソッドを定義するためには別の記述方法や別の場所に記述することが必要なのではないか？

対策

連想配列に適用するメソッドといえばeach。
eachの定義文を見て、真似をすることでうまく定義できるのではないか？
→なかなか定義文にたどり着けず断念。

教えていただきたい内容

①「やりたいこと」を実現するための実装方法の案
②計算値を元にorderでソートをする方法
③連想配列に対してインスタンスメソッドを定義する方法
①〜③いずれでもアイデアがあれば教えていただけると幸いです。

ただし…

①読みづらさ回避のため、情報を少なく抑えています。他に必要な情報があればおっしゃってください
②songテーブルに計算用のカラムを用意するor計算値を格納するテーブル作成、の他の案があると、より嬉しいです
③eachメソッドなど、連想配列に使用するメソッドの定義文があればそれだけでも知りたいです。

【前提情報】

私のプログラミングレベル感

学習履歴
学習時間550h
3月後半 progate
4月 tech campカリキュラム
5月~ 自作アプリ作成

多少なりとも扱えるもの
html/css, javascript(どちらかというとjQuery), ruby, rails, MySQL, GitHub, aws

【参考】作成中のwebページ
技術レベルの把握にご利用ください

IP: 3.113.194.185

<<テストユーザ>>
メールアドレス: s@s
パスワード: 111111

※運用開始していないので自由に登録、削除していただいて構いません。
※https未対応ですのでご注意ください。

開発環境

OS macOS Catalina (10.15.4)
VScode (1.45)

言語・gem等のバージョン

ruby 2.5.1p57
rails (5.2.4.2)
haml (5.1.2)
haml-rails (2.0.1)
mysql2 (0.5.3)
sass (3.7.4)
sass-rails (5.1.0)
jquery-rails (4.4.0)

以上です。

日曜日から苦戦して、そろそろ打ち手が思いつかなくなったので質問しました。
ご助力のほど、何卒よろしくお願いいたします。

Comments

[yambejp]

まずは生のSQLベースで処理をつめて、稼働が確認できてから
railsに落とし込むとよいでしょう。
テーブル自体のサンプルと現行のSQLと想定する結果を例示されるとよいでしょう

[gouf]

NOTE: https://ja.stackoverflow.com/questions/68127/%e8%a8%88%e7%ae%97%e5%80%a4%e3%81%ab%e5%85%83%e3%81%a5%e3%81%84%e3%81%a6-%e3%83%ac%e3%82%b3%e3%83%bc%e3%83%89%e3%82%92%e3%82%bd%e3%83%bc%e3%83%88%e3%81%97%e3%81%9f%e3%81%84%e3%81%8c-order%e3%83%a1%e3%82%bd%e3%83%83%e3%83%89%e3%81%a7%e3%81%af%e3%82%a8%e3%83%a9%e3%83%bc%e3%81%8c%e7%99%ba%e7%94%9f%e3%81%99%e3%82%8b

「マルチポストとはなんですか？何か問題があるのでしょうか？
」 - https://ja.meta.stackoverflow.com/questions/2418/%e3%83%9e%e3%83%ab%e3%83%81%e3%83%9d%e3%82%b9%e3%83%88%e3%81%a8%e3%81%af%e3%81%aa%e3%82%93%e3%81%a7%e3%81%99%e3%81%8b-%e4%bd%95%e3%81%8b%e5%95%8f%e9%a1%8c%e3%81%8c%e3%81%82%e3%82%8b%e3%81%ae%e3%81%a7%e3%81%97%e3%82%87%e3%81%86%e3%81%8b/2419#2419

Answer 1

order()は、SQLのORDER BY句を生成するメソッドなので、
SQLでエラーになる＝存在しないカラムで並び替えなどはできません。

シンプルに実装するなら、songsテーブルにscoreカラムを追加し、
practicesテーブルのレコード登録/更新/削除のタイミングでスコアを計算して
scoresテーブルのスコアを更新するといった方法が良さそうです。

Comments

[shota-imoto]

ご回答ありがとうございます。
やはりデータベースありきでソートしているメソッドだったんですね。

使用頻度が低いカラムをDBに追加するのは、できる限り避けたいところでしたが、DBの内容をいじって機能実装、という方針で動くことにします。

Answer 2

とりあえず「③連想配列に対してインスタンスメソッドを定義する方法」

 a=[[1,2],[3,4],[5,6]].to_h
a.define_singleton_method(:c) do
 size
end
a.c # => 3

ただこれですと、pagingはできないですね。
pageingが必要な場合は（私の実力では）ちと厄介。
pagingが不要なら、③連想配列に対してインスタンスメソッドを定義しないでも可能です

chords　= song.chords.joins(:practices).
         sort_by{|chord| chord.practices.map(&:user_id).uniq.size}

で。

なお、song[:song_practices] = song.chords..... ではDBに反映されませんから「テーブルにそんなカラムはないよエラー」になります。仮想属性を使うなら、一度のアクセスで解決しないとだめです。

だめっぽい
で、真面目に①に取り組むと
song.chords.joins(:practices).group('practices.chord_id').order(count('practices.user_id'))
で行けるかなぁ。
user_idの重複をどう扱うのかな、
Songか、うっかりしてました。

Song.joins(chords: :practices).
  sort_by{|song| 
    song.chords.map{|chord| chord.practices.map(&:user_id)}.
    flatten.uniq.size}

では？　あれ、、、、　N+1 になってたら includesの方でも試してください。

Comments

[shota-imoto]

ご回答ありがとうございます。
教えていただいた１つ目の定義ができないか試行錯誤したものの、ハッシュの配列となると挙動を検証しきれず断念。
(アプリを作る目的と期間を鑑みると、これ以上手をかけるのは得策ではないと判断)
2点目については動作したものの、求める仕様には少しあてはまらないようで断念。
といった結果でした。

今は、songテーブルにカウント用のカラムを追加してソートする方法で実装する方針で、コーディングを進めています。

とはいえ、考えうる手法を試し切るきっかけになりましたのでベストアンサー指名させていただきたいと思います。

実装完了した段階でベストアンサー&本件クローズとさせていただきますので、少々お待ち下さいませ。

[winterboum]

2でだめですか？
どの辺りが仕様とずれてますでしょう。
カラムを追加するのは確実ではありますが、「practices が追加・修正・削除されるたびにカラムの更新が必要になる」難点があります、ので。２はpagingができない、という問題はありますが、リアルタイムで反応する点は優れています。
また、無理やりpagingする方法も有りますから。薄汚いcodeですが。

[winterboum]

あと、SQLに強い方に見えますので、yambejpさんのcommentに従ってみるのも手かと。生なら書けますか？

[shota-imoto]

2について、songレコードのソートをする、という仕様に対して、chordレコードをソートする内容となっております。

「practices が追加・修正・削除されるたびにカラムの更新が必要になる」については、色々と問題を引き起こしそうと見ています。
よってsongはsongでカウントする、という点は同じですが、別の実装を検討中です。