Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

"Joining" strings which may be nil (or) Handling Option hashes

1 view
Skip to first unread message

Gavri Fernandez

unread,
Feb 13, 2005, 11:25:14 AM2/13/05
to
Hi everyone,
I want to create a query based on options passed in through
an options hash to my method.

The query would look like
"title:ruby and author:dave and publisher:oreilly"
or
"title:ruby and publisher:addison-wesley"
or
"author:dave"
or some other combination based on what options have been set in the hash.

options[:author] |= "" followed by Array#join doesn't help me
because it would lead to successive 'and's in the query.

What is the Ruby Idiom to achieve what I want?

TIA
--
Gavri
---------------------------------------------------
I blog here: http://gavri.blogspot.com


Robert Klemme

unread,
Feb 13, 2005, 11:42:46 AM2/13/05
to

"Gavri Fernandez" <gavri.f...@gmail.com> schrieb im Newsbeitrag
news:e3ecfac705021...@mail.gmail.com...

> Hi everyone,
> I want to create a query based on options passed in through
> an options hash to my method.
>
> The query would look like
> "title:ruby and author:dave and publisher:oreilly"
> or
> "title:ruby and publisher:addison-wesley"
> or
> "author:dave"
> or some other combination based on what options have been set in the hash.
>
> options[:author] |= "" followed by Array#join doesn't help me
> because it would lead to successive 'and's in the query.
>
> What is the Ruby Idiom to achieve what I want?

Lots of, here are some:

>> opts = {"title"=>"ruby", "author"=>"dave", "publisher"=>"oreilly",
>> "foo"=>nil}
=> {"title"=>"ruby", "author"=>"dave", "foo"=>nil, "publisher"=>"oreilly"}
>> opts.select{|k,v|v}
=> [["title", "ruby"], ["author", "dave"], ["publisher", "oreilly"]]
>> opts.select{|k,v|v}.map{|k,v| "#{k}=#{v}"}.join(" and ")
=> "title=ruby and author=dave and publisher=oreilly"

Here's a more efficient variant - using #inject of course :-)

>> opts.inject(nil){|s,(k,v)| v ? (s ? s << " and " : "") << k << "=" << v :
>> s}
=> "title=ruby and author=dave and publisher=oreilly"

Kind regards

robert

Francis Hwang

unread,
Feb 13, 2005, 12:02:16 PM2/13/05
to
On Feb 13, 2005, at 11:44 AM, Robert Klemme wrote:
> Lots of, here are some:
>
>>> opts = {"title"=>"ruby", "author"=>"dave", "publisher"=>"oreilly",
>>> "foo"=>nil}
> => {"title"=>"ruby", "author"=>"dave", "foo"=>nil,
> "publisher"=>"oreilly"}
>>> opts.select{|k,v|v}
> => [["title", "ruby"], ["author", "dave"], ["publisher", "oreilly"]]
>>> opts.select{|k,v|v}.map{|k,v| "#{k}=#{v}"}.join(" and ")
> => "title=ruby and author=dave and publisher=oreilly"
>
> Here's a more efficient variant - using #inject of course :-)
>
>>> opts.inject(nil){|s,(k,v)| v ? (s ? s << " and " : "") << k << "="
>>> << v : s}
> => "title=ruby and author=dave and publisher=oreilly"

Maybe the OP isn't worried about this, but: Is there a way to represent
ORs as well as ANDs if you're doing this?

Francis Hwang
http://fhwang.net/

Gavri Fernandez

unread,
Feb 13, 2005, 12:02:40 PM2/13/05
to
On Mon, 14 Feb 2005 01:44:57 +0900, Robert Klemme <bob....@gmx.net> wrote:
>
> "Gavri Fernandez" <gavri.f...@gmail.com> schrieb im Newsbeitrag
> news:e3ecfac705021...@mail.gmail.com...

> Lots of, here are some:


>
> >> opts = {"title"=>"ruby", "author"=>"dave", "publisher"=>"oreilly",
> >> "foo"=>nil}
> => {"title"=>"ruby", "author"=>"dave", "foo"=>nil, "publisher"=>"oreilly"}
> >> opts.select{|k,v|v}
> => [["title", "ruby"], ["author", "dave"], ["publisher", "oreilly"]]
> >> opts.select{|k,v|v}.map{|k,v| "#{k}=#{v}"}.join(" and ")
> => "title=ruby and author=dave and publisher=oreilly"
>
> Here's a more efficient variant - using #inject of course :-)
>
> >> opts.inject(nil){|s,(k,v)| v ? (s ? s << " and " : "") << k << "=" << v :
> >> s}
> => "title=ruby and author=dave and publisher=oreilly"


This is the solution I hit upon right after I sent my mail. Please
critique while I try to understand your solutions :)

def get_query(options)
query_fragments = []
options.each do |key, value|
query_fragments.push("#{key.id2name}:#{value}")
end
query = query_fragments.join(" and ")
end

Thanks Robert

E S

unread,
Feb 13, 2005, 12:25:21 PM2/13/05
to
> Lähettäjä: "Robert Klemme" <bob....@gmx.net>
> Aihe: Re: "Joining" strings which may be nil (or) Handling Option hashes

>
>
> "Gavri Fernandez" <gavri.f...@gmail.com> schrieb im Newsbeitrag
> news:e3ecfac705021...@mail.gmail.com...
> > Hi everyone,
> > I want to create a query based on options passed in through
> > an options hash to my method.
> >
> > The query would look like
> > "title:ruby and author:dave and publisher:oreilly"
> > or
> > "title:ruby and publisher:addison-wesley"
> > or
> > "author:dave"
> > or some other combination based on what options have been set in the hash.
> >
> > options[:author] |= "" followed by Array#join doesn't help me
> > because it would lead to successive 'and's in the query.
> >
> > What is the Ruby Idiom to achieve what I want?
>
> Lots of, here are some:
>
> >> opts = {"title"=>"ruby", "author"=>"dave", "publisher"=>"oreilly",
> >> "foo"=>nil}
> => {"title"=>"ruby", "author"=>"dave", "foo"=>nil, "publisher"=>"oreilly"}
> >> opts.select{|k,v|v}
> => [["title", "ruby"], ["author", "dave"], ["publisher", "oreilly"]]
> >> opts.select{|k,v|v}.map{|k,v| "#{k}=#{v}"}.join(" and ")
> => "title=ruby and author=dave and publisher=oreilly"
>
> Here's a more efficient variant - using #inject of course :-)
>
> >> opts.inject(nil){|s,(k,v)| v ? (s ? s << " and " : "") << k << "=" << v :
> >> s}
> => "title=ruby and author=dave and publisher=oreilly"

Or, possibly more legibly:

hsh.to_a.map!{|i| i.join('=')}.join(' and ')

?

> Kind regards
>
> robert

E

Robert Klemme

unread,
Feb 13, 2005, 12:34:25 PM2/13/05
to

I'm missing the condition. As far as I understood you you want to be able
to skip nil values. Did I get this wrong? Apart from that it does
certainly what you want. If you want to spare the intermediate array you
can do a bit optimization:

def get_query(options)
q = nil
options.each do |k,v|
q = (q ? q << " and " : "") << k << ":" << v
end
q
end

This is basically a verbose variant of my inject version.

> Thanks Robert

You're welcome!

Kind regards

robert

Robert Klemme

unread,
Feb 13, 2005, 12:52:22 PM2/13/05
to

"Francis Hwang" <se...@fhwang.net> schrieb im Newsbeitrag
news:0682b7cb1757683c...@fhwang.net...

You could provide multiple values for a key and create an OR from that:

opts = {"title"=>["ruby","foobar"], "author"=>"dave"}

opts.inject(nil) do |s,(k,v)|
if v


(s ? s << " and " : "") <<

(Enumerable === v && ! (String === v) ?
v.inject(nil) {|s2,v2| (s2 ? s2 << " or " : "(") << k << ":" << v2} <<
")" :
"#{k}:#{v}")
else
s
end
end

Slightly unreadable... :-)) You don't need the outer if-else-end though if
v is never nil.

Kind regards

robert

William James

unread,
Feb 13, 2005, 4:42:47 PM2/13/05
to
Robert Klemme wrote:
>
> Here's a more efficient variant - using #inject of course :-)
>
> >> opts.inject(nil){|s,(k,v)| v ? (s ? s << " and " : "") << k << "="
<< v :
> >> s}
> => "title=ruby and author=dave and publisher=oreilly"


I prefer this:

opts.to_a.map{|x| x.join('=') if x[1]}.compact.join(' and ')

----> "title=ruby and author=dave and publisher=oreilly"

Gavri Fernandez

unread,
Feb 14, 2005, 8:06:41 AM2/14/05
to
On Mon, 14 Feb 2005 02:35:02 +0900, Robert Klemme <bob....@gmx.net> wrote:

> I'm missing the condition. As far as I understood you you want to be able
> to skip nil values. Did I get this wrong? Apart from that it does

My bad. My mail was cluttered with
1) The problem
2) Possible solution by handling unused options by not considering them.
3) Possible solution by setting unused options as nil and handling
them while "joining"

Anyway, thanks for the solutions.

William James

unread,
Feb 14, 2005, 8:15:01 PM2/14/05
to

Too prolix. Better:

opts.map{|x| x.join('=') if x[1]}.compact.join(' and ')

Here are the stages through which the data travels:

{"title"=>"ruby", "author"=>"dave", "foo"=>nil, "publisher"=>"oreilly"}

["title=ruby", "author=dave", nil, "publisher=oreilly"]
["title=ruby", "author=dave", "publisher=oreilly"]

0 new messages