Use join:
def self.all_email_addresses User.all.collect {|u| u.email}.join ', ' end
str.chop.chop # ...or... str[0..-3]
Although this does answer the exact question, I agree that it isn't the best way to solve the problem.