A nicer way to join my hash?

rubyonrails-talk
1

Hi,

@message = Message.new(params[:message]) @message.recipients = Volunteer.all(:conditions => "email IS NOT NULL
AND email <> ''", :group => "email").map{|v| v.email}.join(", ")

2

Or even this:

@message = Message.new(params[:message])

volunteers = Volunteer.all(:conditions => "email IS NOT NULL AND email <> ''", :group => "email")

# essentially the same as .map { |v| v.email } only letting ruby figure out the "v" part. @message.recipients = volunteers.map(&:email)

-- Josh http://iammrjoshua.com

Julian Leviston wrote:

3

No, &: is much slower and uglier

Blog: http://random8.zenunit.com/ Learn rails: http://sensei.zenunit.com/

4

Julian Leviston wrote:

@message = Message.new(params[:message]) @message.recipients = Volunteer.all(:conditions => "email IS NOT NULL AND email <> ''", :group => "email").map{|v| v.email}.join(", ")

Whoa.. cool.. I'm gonna read up on Ruby basics, thanks heaps.

5

You might want rails basics, coz that contains SQL, ruby AND rails code

Blog: http://random8.zenunit.com/ Learn rails: http://sensei.zenunit.com/

6

Project Euler!

Blog: http://random8.zenunit.com/ Learn rails: http://sensei.zenunit.com/