rails3初心者 複数の絞込検索

質問

絞り込み条件で、メーカーやカテゴリを選択してから
金額指定があれば、金額指定も含めて検索結果を実装する事はできますか？
またボタンは1個にする形にしたいです。

よろしくお願い致しますm(_ _)m

絞込み検索内容
＝＝＝＝＝＝＝＝＝＝
・メーカー　（チェックボックス式）
・カテゴリ　（チェックボックス式）
・金額指定
＝＝＝＝＝＝＝＝＝＝

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
views:
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
<p>条件検索(複数選択可)</p>

<%= form_tag ({:controller => :products, :action => :check }), {:method => :get} do %>

<p>メーカーから探す</p>
<% Maker.all.each do |maker|%>
<%= check_box_tag "maker_id[]", maker.id, (params[:maker_id].include?(maker.id.to_s) if params[:maker_id]) %>
<%= maker.maker_name %><br />
<% end %>

<p>カテゴリから探す</p>
<% Category.all.each do |c| %>
<%= check_box_tag "category_id[]", c.id, (params[:category_id].include?(c.id.to_s) if params[:category_id]) %>
<%= c.name %><br />
<% end %><br />

<%= submit_tag "検索"%>　＃このボタンを外してひとつのボタンにしたいです。
<% end %>

<p>金額を指定して絞り込む</p>
<%= form_tag ({:controller => :products, :action => :search_price }), {:method => :post} do %>
　<%= text_field_tag "price1",(params[:price1] if params[:price1]) %> ～　
　<%= text_field_tag "price2",(params[:price2] if params[:price2]) %>
<%= submit_tag "検索"%>
<% end %>

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
products_controller.rb
＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
def check
@product = Product.all

if params[:maker_id] && params[:category_id]　　 # 両方選択された場合
@products = Product.where(:category_id => params[:category_id], :maker_id => params[:maker_id] )
elsif params[:maker_id] 　　#makerが選択された場合
@products = Product.where(:maker_id => params[:maker_id]) if params[:maker_id]
elsif params[:category_id] 　 #カテゴリが選択された場合
@products = Product.where(:category_id => params[:category_id]) if params[:category_id]
elsif params[:maker_id] == nil && params[:category_id] == nil 　 #両方ともチェックがなかった場合
@products = Product.all
end
render :action => 'check'
end

def search_price
@products = Product.where('price >=? AND price <=?', params[:price1],params[:price2])
@products = Product.where('price >=?', params[:price1]) if params[:price2].blank?
@products = Product.where('price <=?', params[:price2]) if params[:price1].blank?

render :action => 'index'
end

ややこしくてすみません^^;
どうぞよろしくお願い致します。

回答No.2

tatsu99 · ベストアンサー

view側のボタンを１つにまとめます。
=================================================
<%= form_tag({:controller => :products, :action =>:search_price }) do %>
<p>メーカーから探す</p>
<% Maker.all.each do |maker|%>
<%= check_box_tag "maker_id[]", maker.id %>
<%= maker.maker_name %><br />
<% end %>

<p>カテゴリから探す</p>
<% Category.all.each do |c| %>
<%= check_box_tag "category_id[]", c.id %>
<%= c.name %><br />
<% end %><br />

価格指定
<%= text_field_tag "price1" %> ～
<%= text_field_tag "price2" %></br>
<%= submit_tag "検索"%>
<% end %>
===================================================
尚、params[:maker_id].include?　等のようにメーカの指定がある場合は、この例は考慮していません。
全てのメーカー、全てのカテゴリから選択させています。ですので、ここは、適宜修正してください。

contoller側は、search_priceにまとめるものとします。
この場合、以下のパターンになります。
１．メーカーでいくつか選択されたか又は一切選択されていない。
２．カテゴリでいくつか選択されたか又は一切選択されていない。
３．価格下限が指定されたか、又は指定されていない。
４．価格上限が指定されたか、又は指定されていない。
これらの全ての組み合わせは２×２×２×２＝１６通りになります。
従って、ifの組み合わせで１６個の検索文を記述することになりますが、これを１つの検索文にすることが
今回の目標です。

予備知識として、「whereメソッドは複数連結して記述することが可能である」ということを理解しておきます。
これは、
@products = Product.where('price >=? AND price <=?', params[:price1],params[:price2])
は
@products = Product.where('price >=?',params[:price1]).where('price <=?',params[:price2])
のようにも記述できるということです。
今回は
@products = Product.where(.....).where(....).where(....).where(....)
のように書くことを目指します。
この場合のwhereの条件はＡＮＤ条件で連結されます。
また、where(....)の....の部分が、「空っぽいもの」の場合は、このwhereは、効力をもたず
無いものとして扱われます。
「空っぽいもの」は
.blank?が成立するものです。
where("")　・・・空文字列
とか
where({})　・・・空のハッシュ
などが該当します。
つまり
@products = Product.where('price >=?',params[:price1]).where("")と
@products = Product.where('price >=?',params[:price1])　は同じ結果になります。

以上のことを前提にすると、
メーカー、カテゴリ、価格下限、価格上限が全て指定された場合は、
@products = Product.where(:maker_id => params[:maker_id]).where(:category_id => params[:category_id]).where('price >= ?',params[:price1]).where('price <= ?',params[:price2])
と記述すればよいことになります。
従って、ここで、メーカー、カテゴリがともに指定されていないなら、
@products = Product.where({}).where({}).where('price >= ?',params[:price1]).where('price <= ?',params[:price2])
でも良いわけです。
更に、価格下限、価格上限が指定されない場合（要は全て指定されない場合）は、
@products = Product.where({}).where({}).where("").where("")のようになります。
上記は、正しいのですが、価格下限、価格上限はプレイスフォルダを使うことが望ましい（セキュリティ対策）ので、
@products = Product.where({}).where({}).where('price >= ?',0).where('price <= ?',2147483647)
のようにします。
（価格はＤＢ上は0円以上で、2147483647円以下とします。2147483647は32ビット整数の最大値です。
今回、2147483647としましたが、正確には、ＤＢ上に実装される価格の最大値を指定してください）
上記を踏まえたsearch_priceメソッドは、以下のようになります。
====================================
def search_price
p1 = p2 = Hash.new
p3 = 0
p4 = 2147483647
if params[:maker_id]
p1[:maker_id] = params[:maker_id]
end
if params[:category_id]
p2[:category_id] = params[:category_id]
end
unless params[:price1].blank?
p3 = params[:price1]
end
unless params[:price2].blank?
p4 = params[:price2]
end
@products = Product.where(p1).where(p2).where('price >= ?',p3).where('price <= ?',p4)
render :action => 'index'
end
====================================
更に、価格順に取得したい場合は、
@products = Product.where(p1).where(p2).where('price >= ?',p3).where('price <= ?',p4).order('price ASC')
のようにします。
尚、今回は、価格下限、価格上限の入力チェックはしていません。
（数字でないものが入力された場合とか、価格下限＞価格上限のとなる値が指定された場合、本来はエラーメッセージを出すべきと考えています）

お礼

tatsu99様

出来ました！
すごく丁寧に説明していただきありがとうございます。
ご足労をお掛けした事と思います。
本当にありがとうございましたm(_ _)m

またお手すきの時に、コードの説明をしていただいてもよろしいでしょうか？
======================
unless params[:price1].blank?
p3 = params[:price1]
======================

これは、ユーザーが入力した場合も、nilの場合も動いていますが、なぜでしょうか？^^;
unless params[:price1].blank?　＃空白じゃなかったら、
p3 = params[:price1]　#入力された数字をp3に代入
という意味ではないのですか？^^;

補足

価格順に表示を取得する時のorderメソッドを使用して実装してみたのですが、
なにも変化がございませんでした。。

@products = Product.where(p1).where(p2).where('price >= ?',p3).where('price <= ?',p4).order('price ASC')

""ダブルクオーテーションで囲っても一緒でした。。
@products = Product.where(p1).where(p2).where('price >= ?',p3).where('price <= ?',p4).order("price ASC")

ログではprice DESCとなっています。どんな原因が考えられるでしょうか？
ご教授いただければ幸いです。

＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊
SELECT "products".* FROM "products" WHERE (price >='1000') AND (price <='200000') ORDER BY created_at DESC, price DESC
＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊＊

よろしくお願い致します。

回答No.3

tatsu99

>======================
>unless params[:price1].blank?
>p3 = params[:price1]
>======================

>これは、ユーザーが入力した場合も、nilの場合も動いていますが、なぜでしょうか？^^;
>unless params[:price1].blank?　＃空白じゃなかったら、
>p3 = params[:price1]　#入力された数字をp3に代入
>という意味ではないのですか？^^;

unless params[:price1].blank?
p3 = params[:price1]
end
この意味は、ユーザーが下限値を入力した場合のみ、P3にその入力値を設定します。
ユーザーは、入力をしなかった場合はp3 = params[:price1]は実行されません。
if params[:price1] != ""
p3 = params[:price1]
end
と同じ意味です。
p3 = params[:price1]
が実行されない場合は、
事前にp3=0があるので、結果としてp3=0になります。
つまりp3はユーザーが入力すればその値、入力しなければ0になります。
その状態で、where('price >= ?',p3)を行なうと、
入力しなかった場合は、where('price >= ?',0)になり、結果として、全ての価格を対象としていることに
なります。
p4についても同様に、
where('price <= ?',p4)を行なうと、入力しなかった場合は
where('price <= ?',2147483647)になるので、結果として、全ての価格を対象としていることに
なります。

お礼

tatsu99様
いつもご教授ありがとうございます。
なるほどなるほど！理解できました。
if文ではなくて、unless文を使う意味が分かりました。
勉強になりました。ありがとうございます。

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝
ifとunlessの違い

p3 = 0

if params[:price1].blank? # 空白だったら(trueの場合）
p3 = params[:price1] # 値が入っていない
end #ここでストップする

if params[:price1].blank? #入力値があったら(falseの場合)
p3 = params[:price1] #実行されない
end #次の文へ

＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝

unless params[:price1].blank? # 入力値があったら(falseの場合)
　p3 = params[:price1]　＃入力値が代入される
end #次の文へ

unless params[:price1].blank? #空白だったら(trueの場合)
　p3 = params[:price1]　#実行されないので p3 = 0が実行される
end ＃次の文へ

自分への備忘録も兼ねてif とunlessを使った場合の違いをここに書かせていただきました^^;
ありがとうございましたm(_ _)m

補足

すみません。

if文の書き方は#1の方でご説明くださっておりました。

＃1、＃２と順を追ってご説明くだっておりましたのに、
先に＃2の方を試してしまいましたので、if文を使った方法を理解できずに
的外れなコメントをしました事お詫び致しますm(_ _)m

また順を追わずに、大変失礼な事をした事を、深くお詫び申し上げます。

回答No.1

tatsu99

全ての回答ではありません。現行のスクリプトの問題点の指摘です。
@products = Product.where('price >=? AND price <=?', params[:price1],params[:price2])
@products = Product.where('price >=?', params[:price1]) if params[:price2].blank?
@products = Product.where('price <=?', params[:price2]) if params[:price1].blank?
これは一見正しく動作しているように見えますが、２点問題点があります。
1点目は、 params[:price1]とparams[:price2]の両方がブランクの場合、上記の３行は全て実行されることです。
従って、その場合、最後の３行目の実行結果が @productsに格納されることです。
そして、そのことをあなたは期待していないということです。（結果的にＯＫかも知れませんが）
（両方がブランクのケースは考慮不要であれば、この限りにあらず）
2点目は、params[:price2]のみがブランクの場合、1行目と２行目が実行され、
結果として、2行目の実行結果が@productsに格納され、正しい結果を得ることができます。
しかしながら、ＤＢへのアクセスを２回行なうことは、パフォーマンス上、良いことではありません。
つまり、ＤＢへのアクセスは（やむを得ない場合を除いて）1回にすべきです。
そのことを考慮して、処理コードを記述すると以下のようになります。
if params[:price1].blank?
if params[:price2].blank?
@products = Product.all
else
@products = Product.where('price <= ?',params[:price2])
end
else
if params[:price2].blank?
@products = Product.where('price >= ?',params[:price1])
else
@products = Product.where('price >= ? AND price <= ?',params[:price1],params[:price2])
end
end

これは、検索条件により、検索方法が４通りあり、それを素直にコーディングしたものです。
これは、間違いではないですが、できれば、１つの検索文を記述するだけですむようにしたいはずです。
そのほうが、コードもシンプルになり、試験も楽になるはずです。
その実装方法は、次の回答で記述します。

お礼

補足で価格順表示を実装できないと記述致しましたが、教えていただいたコードで実装できました。
ややこしい事を申し上げてすみません。
ありがとうございました。

@products = Product.where(p1).where(p2).where('price >= ?',p3).where('price <= ?',p4).order('price ASC')