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

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

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

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

2回答

1633閲覧

ジャンル検索においての重複のない抽出の仕方

choco_duck

総合スコア4

Ruby

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

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2020/04/04 07:14

前提・実現したいこと

Windows 10 Home
ruby 2.5.1p57
Rails 5.2.4.2
SQLite 3.30.1

Progateで学習を終え、
Ruby on Rails5でショップをまとめ、検索出来るサイトの作成中なのですが、
ジャンル指定でのショップ絞り込み機能の実装中に躓いてしまいました。

Genreモデルは

idgenre_idfloorname
111本・コミック・雑誌
1212

というカラムを持っています。
Genreモデルの中身

発生している問題

一つのショップに、同じgenre_idのジャンルを複数設定してある場合
ShopsGenre.where(genre_id: @search_genre.genre_id)では、
同じshop_idを持つデータも抜き取ってしまい、出力すると画像のように
ショップが重複して表示されてしまいます。
挿入画像3
現在のソースコードは以下の通りです

ShopsController

1 before_action :set_search_genre 2 3 def set_search_genre 4 @search_genre = Genre.find_by(id: session[:search_genre_id]) 5 end 6 7 def index 8 # session[:search_genre_id]セット 9 if @search_genre 10 @genres = Genre.where(genre_id: @search_genre.genre_id, floor: 2) 11 else 12 @genres = Genre.where(floor: 1) 13 end 14 if params[:genre] 15 @genre = Genre.find_by(id: params[:genre]) 16 session[:search_genre_id] = @genre.id 17 redirect_to("/shops/index") 18 end 19 ----------------------------------------------------------------------------------- 20 if @search_genre 21 if params[:keyword] 22 @shops = Shop.where("name Like ?", "%#{params[:keyword]}%").order(id: :desc) 23 else 24 if @search_genre.floor == 1 25 @hit_shops = ShopsGenre.where(genre_id: @search_genre.genre_id) 26 else 27 @hit_shops = ShopsGenre.where(genre_column_id: @search_genre.id) 28 end 29 if @hit_shops != nil 30 @shops = [] 31 @hit_shops.each do |hit_shop| 32 @shops.insert(0, Shop.find_by(id: hit_shop.shop_id ) ) 33 end 34 end 35 end 36 ----------------------------------------------------------------------------------- 37 else 38 if params[:keyword] 39 @shops = Shop.where("name Like ?", "%#{params[:keyword]}%").order(id: :desc) 40 else 41 @shops = Shop.all.order(id: :desc) 42 end 43 end 44 if @shops 45 @result_amount = @shops.count 46 end 47 end 48

ShopsGenreモデルの内容
挿入画像2

試したこと

ShopsGenre.select(:shop_id).distinctだと、
重複はなくなりますが、where(genre_id: @search_genre.genre_id)の
指定ができないので、genre_idが異なるジャンルを設定してあるショップも
表示されてしまいます。
ShopsGenre.select(:shop_id).distinctと
ShopsGenre.where(genre_id: @search_genre.genre_id)を
両方適用するような処理はどのようにすればいいのでしょうか。

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

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

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

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

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

guest

回答2

0

genre_idとfloorでジャンル内の階層を表現していると仮定しますが
genre_idじゃなくidで絞り込むのが良いと思います。

投稿2020/04/04 10:49

toppunobice0

総合スコア17

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

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

choco_duck

2020/04/04 12:04 編集

回答ありがとうございます! 仰る通り、genre_idとfloorでジャンル内の階層を表現しています。 絞り込み条件に、floorが2のジャンルを選択したときは、 idで絞り込みをしているのですが、 floorが1のジャンルを選択したときは、 そのジャンル(同じgenre_id)の子(floorが2)のジャンルが 設定されているショップを一覧表示したいのです。 ショップに設定するジャンルの階層はすべてfloorが2です。
guest

0

自己解決

pluckとuniqを用いることで解決することが出来ました。
if @search_genre.floor == 1の中の処理を以下のように変更することで、
目的の動作を実現することが出来ました。

if @search_genre.floor == 1 @hit_duplication_shops = ShopsGenre.where(genre_id: @search_genre.genre_id) @hit_duplication_shops_ids = @hit_duplication_shops.pluck(:shop_id) @hit_shops_ids = @hit_duplication_shops_ids.uniq @hit_shops = [] @hit_shops_ids.each do |hit_shops_id| @hit_shops.insert(0, ShopsGenre.find_by(shop_id: hit_shops_id)) end else

投稿2020/04/05 05:59

choco_duck

総合スコア4

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問