質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

PostgreSQL

PostgreSQLはオープンソースのオブジェクトリレーショナルデータベース管理システムです。 Oracle Databaseで使われるPL/SQLを参考に実装されたビルトイン言語で、Windows、 Mac、Linux、UNIX、MSなどいくつものプラットフォームに対応しています。

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Q&A

2回答

4352閲覧

ページングがおかしい。表示内容が重複する。数も正しくない。クエリも正しくない?

Aihara

総合スコア24

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

PostgreSQL

PostgreSQLはオープンソースのオブジェクトリレーショナルデータベース管理システムです。 Oracle Databaseで使われるPL/SQLを参考に実装されたビルトイン言語で、Windows、 Mac、Linux、UNIX、MSなどいくつものプラットフォームに対応しています。

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

0グッド

0クリップ

投稿2017/05/02 11:30

編集2022/01/12 10:55

使用している環境は
Rails 4.2.6
ruby 2.3.1p112
psql (PostgreSQL) 9.2.18

今回困っていること
・ページングが何故かうまくいかない
・自分の意図したクエリを回せているのか

使用しているモデルは
Restaurant
Menu
の二つで

Restaurant
has_many Menu

Menu
belongs_to Restaurant
の関係です。

RestaurantImg
も使用してありますが、Menuと同じく
Restaurantと1対多の関係です。

今回、レストランが複数持っているメニューのうち
最安価格を基準にして、レストランの表示順を安い順に設定したいです。
(ここで、自分のしたいように正しくactive recordを使えているのか、定かでない。)
しかし、何故かページングが正しく動作しません。
使用しているgemはkaminariです。
具体的に言うと、
レストランの総数は19個あり、
そのうち1ページにつき10個のレストランを表示するように設定しているのですが
1ページ目には8個しかでない、という状況になっています。

以下のようにレストラン一覧を取得しています

@restaurants = Restaurant.where(:data_status => "active").genre(params[:genre]).area(params[:area]).includes(:restaurant_imgs, :menus).where(restaurant_imgs: {size: "normal"}).sort(params[:sort]).page(@page).per(10)

こういった内容でレストランを取得していました。
各scopeは以下の通りです。

scope :genre, -> (genre) { if genre.present? && genre != "all" where("restaurants.genre = ?", genre) end }
scope :area, -> (area) { if area.present? && area != "all" where("restaurants.area = ?", area) end }
scope :sort, -> (sort) { if sort.present? && sort == "price" order("menus.price asc, restaurants.pickup desc") end }

restaurants.pickupは
数字で管理されていて、各レストランに一意に保存されていてます。

(書き忘れ)
kaminariで表示する検索結果の文言についてです。
page_entries_info
というヘルパーメソッドで検索結果の数について表示をしてくれるのですが
そこで
「19つの全てのレストランを表示しています」と出ているにもかかわらず
1ページあたり10つを表示と設定してあるページに
8つしかレストランが表示されません。
データを取り出す部分と、kaminariと
両方ともにおかしいところがあるのではないかと思っております。

また、1ページあたり100と極端に設定すると
DBに入ってある19つのレストランが全て表示されます。
不可解です。。

(ログを見てみる)

SQL (3.3ms) SELECT DISTINCT "restaurants"."id", menus.price AS alias_0 FROM "restaurants" LEFT OUTER JOIN "menus" ON "menus"."restaurant_id" = "restaurants"."id" AND "menus"."data_status" = $1 WHERE "restaurants"."data_status" = $2 ORDER BY menus.price ASC LIMIT 10 OFFSET 0 [["data_status", "active"], ["data_status", "active"]]  (3.2ms) SELECT COUNT(DISTINCT "restaurants"."id") FROM "restaurants" LEFT OUTER JOIN "menus" ON "menus"."restaurant_id" = "restaurants"."id" AND "menus"."data_status" = $1 WHERE "restaurants"."data_status" = $2 AND "restaurants"."id" IN (13, 4, 5, 10, 17, 2, 6, 18, 6, 5) [["data_status", "active"], ["data_status", "active"]]  (3.1ms) SELECT COUNT(DISTINCT "restaurants"."id") FROM "restaurants" LEFT OUTER JOIN "menus" ON "menus"."restaurant_id" = "restaurants"."id" AND "menus"."data_status" = $1 WHERE "restaurants"."data_status" = $2 [["data_status", "active"], ["data_status", "active"]] CACHE (0.0ms) SELECT DISTINCT "restaurants"."id", menus.price AS alias_0 FROM "restaurants" LEFT OUTER JOIN "menus" ON "menus"."restaurant_id" = "restaurants"."id" AND "menus"."data_status" = $1 WHERE "restaurants"."data_status" = $2 ORDER BY menus.price ASC LIMIT 10 OFFSET 0 [["data_status", "active"], ["data_status", "active"]]

以上のようなログが出ているのですが

ORDER BY menus.price

これが入っていると

"restaurants"."id" IN (13, 4, 5, 10, 17, 2, 6, 18, 6, 5)

これが入るのですが、なぜここで5,6が重複してしまうのでしょうか。
これが原因で、10件表示としているにもかかわらず8件しか出ない、というようになっていると思います。
並び順を指定するだけで重複するようなことになってしまうのが、よく分かりません。。。

宜しくお願い致します。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

十分な情報がないので推測の域をでませんが
おそらく

ruby

1.where(restaurant_imgs: {size: "normal"})

がいけません。
これですと、せっかくincludesでrestaurant_img を left outer join
しても、size: "normal"で絞り込んでしまうので、inner joinと同じ
restaurant_imgs: {size: "normal"}を持つレストラントしか抽出されません。

こういった場合、例えば、restaurant_imgs: {size: "normal"}がトップ画像とか、で一枚しかないならば

ruby

1has_many :restaurant_imgs

の他に

ruby

1has_one :restaurant_img,->{where(restaurant_imgs: {size: "normal"})}

みたいなものを追加して
複数ある場合は(この時点で実装が間違えている気もしますが...)

ruby

1has_many :normal_size_restaurant_imgs,->{where(restaurant_imgs: {size: "normal"})},class_name: 'RestrantImage'

として

ruby

1.includes(:normal_size_restaurant_imgs, :menus)

としてみてください

投稿2017/05/03 05:26

編集2017/05/03 06:58
moke

総合スコア2241

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Aihara

2017/05/05 13:16

moke様 ご回答して頂きありがとうございます。 restauranはrestaurant_imgsを複数枚所有していて さらにrestaurant_imgsは横長のものと丸型のものと2種類あります。 DBに入っている全てのrestaurantは写真を各種とも複数枚所有しているため where(restaurant_imgs: {size: "normal"}) を外してみても、表示結果に変わりはありませんでした。 has_one, has_manyは重複して設定できることは知りませんでした。。 なるほどなと思いました!!
guest

0

再現していないのでよくわかりませんが、development.log に出てくるSQL文を見てみると何か気がつくことがあるかもしれません。

投稿2017/05/02 22:23

SugiTK

総合スコア495

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問