Tap that object

tap_or_no_tap.rb

# To tap
def activate_person(person_id)
  Person.find(person_id).tap do |person|
    person.activate!
  end
end

# or not to tap
def activate_person(person_id)
  person = Person.find(person_id)
  person.activate!
  person
end

# that is the question

@franciscoj yeah, that I could actually get behind.

@pjb3 do you have any particular reason to prefer the no-tap version? In what way is the tap version worse, in your view?

(I actually like tap, but I don't have any real rationale for why … I'd love to see a reasoned argument for one way or the other.)

@glv @pjb3 A few months back @brixen saw me using #tap and lamented on how it's not ideal because it creates a new scope when a perfectly suitable one already exists. From what I remember he said it's not worth extending the stack for no good reason. Why make the VM do more work when the context is already set up already? @brixen, did I get that right?