tap.rb

##### Pointless: #####

# Bad:
Foo.new.tap do |foo|
  foo.bar = "<3"
end

# Good:
foo = Foo.new
foo.bar = "<3"
foo

# (I prefer the latter because it flows linearly, which it
# should, as the sequence of events is linear.)

##### Just use Enumerable: #####

# Bad:
{}.tap do |hash|
  foos.each do |foo|
    hash[foo.name] = foo
  end
end

# Good:
Hash[foos.map { |foo| [foo.name, foo] }]