Which do you prefer?

which.rb

# Obviously the second solution is faster or a large collection, but some said the first solution is more readable.
# Which would you use?

result = []

collection.each do |c|
  result << some_method(c)
end

result.reject!(&:nil?)


# Versus


result = []

collection.each do |c|
  value = some_method(c)
  result << value if value
end

@fbernier I would say:

collection.each_with_object([]) do |c, result|
  value = some_method(c)
  result << value if value
end

@chatgris Yes, that's the best solution IMO.