On Tuesday 04 August 2009 10:07:41 Klaus Kaempf wrote:
* Josef Reidinger <jreidinger@suse.cz> [Aug 04. 2009 10:00]:
+ region = timezones.find { |reg| reg.name == params[:region] } + region = {} unless region
This can be further shortened by writing
region = (timezones.find { |reg| reg.name == params[:region] }) || {}
Where I prefer
region = (timezones.find { |reg| reg.name == params[:region] }) || Hash.new
to make it less cryptic ;-)
Heh, I think the || is pretty readable. You can even use the keyword 'or' and make it more english: region = (timezones.find { |reg| reg.name == params[:region] }) or {} I prefer [ ] and { } more than Array.new and Hash.new ;-) -- Duncan Mac-Vicar P. - Engineering Manager, YaST SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nuernberg) -- To unsubscribe, e-mail: yast-devel+unsubscribe@opensuse.org For additional commands, e-mail: yast-devel+help@opensuse.org