1

CompanyDisclosureというモデルが有り Company

class Company < ActiveRecord::Base
  has_many :disclosures, dependent: :destroy
end

の関係になっています。

ここでDisclosurepublished_atというdatetimeのカラムを持つのですが、複数のCompanyDisclosurepublished_atカラムの最新の物を用いてソートするにはどのような方法があるでしょうか?

現在はCompanylatest_disclosure_published_atというソート用のカラムを追加し、Disclosureの更新時にCompany側の情報を更新しています。

Companyにカラムを追加せずにこのようなソートをするもっと良い方法はありますか?

追記

@yasu さんに教えて頂いた方法でローカルではできたのですが、サーバー上で試してみた所エラーが出てしまいました。

PG::GroupingError: ERROR:  column "companies.code" must appear in the GROUP BY clause or be used in an aggregate function`

サーバー上とローカルでPostgresのバージョンが異なってしまっているのですが、それが原因なのでしょうか…?

Company.joins(:disclosures).
group('companies.id, companies.code').
select('companies.id, companies.code, MAX(disclosures.published_at) disclosure_published_at')

のようにselectするカラムを全てgroupで指定すれば呼び出せるようですが、できれば*で一括で指定したいです。

サーバー上(

psql --version
psql (PostgreSQL) 8.4.20

psql --version
psql (PostgreSQL) 9.5.2

2
  • 2
    PostgreSQLでyasuさんの書かれたような記法が通るようになったのは、9.1からだそう(PostgreSQL 9.1 の新機能「GROUP BY が MySQL 風に緩めに」の部分)なので、それより前のバージョンを使うのであればGROUP BY(Railsのgroupメソッド)にはSELECTで取得するカラムを全部指定する必要があります。(*は使えません。)と言うわけで、8.4.2(いくらなんでも古すぎるのでは?)を使うのであれば、「できれば*で一括で指定したい」は「できない」と言うことになるかと思います。
    – OOPer
    Commented 2017年6月27日 16:02
  • なるほど、9.1からだったのですね。これまで特に問題がなかったので古いバージョンのものを使っていたのですが、バージョンをあげることにします。
    – ironsand
    Commented 2017年6月28日 3:32

1 件の回答 1

1

JOIN してこんな感じかなあ...

Company
  .joins(:disclosures)
  .group("companies.id")
  .select("companies.*, MAX(disclosures.published_at) disclosure_published_at")
  .order("MAX(disclosures.published_at)")

MAX(disclosures.published_at) の値を取るには特異メソッドの disclosure_published_at を使ってください。

追記

GROUP BY で指定した以外のカラムを SELECT で使うことは標準SQLとして正しくない方法で、私の回答があまりよくありませんでした。

カラムをいちいち指定するのも面倒なので、こんな感じの妥協案を。

company_cols = Company.column_names.map { |name| "#{Company.table_name}.#{name}" }
Company
  .joins(:disclosures)
  .group(company_cols)
  .select(*company_cols, "MAX(disclosures.published_at) disclosure_published_at")
  .order("MAX(disclosures.published_at)")
3
  • MAXをこういう風に使えばいいんですね、ありがとうございます。
    – ironsand
    Commented 2017年6月22日 16:50
  • すいません、ローカルでは教えて頂いた方法で解決したの解決済みにしたのですが、サーバー上ではエラーが出てしまったのでベストアンサーのチェックを外してエラーの詳細を質問に追記しました。
    – ironsand
    Commented 2017年6月26日 9:48
  • 1
    SQL-99以降の標準SQLでは、オプション仕様として、GROUP BY指定カラムに関数従属製のあるカラムはSELECTに記載できるような拡張を認めています(SQL optional feature T301)ので、「標準SQLとして正しくない」と言うのは不正確かと思います。「標準SQLでもオプション仕様なので互換性を考えるなら使用すべきでない」くらいでしょうか。
    – OOPer
    Commented 2017年6月30日 0:12

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。