giving_up_nil.rb

# I see code like this all the time:

def get_results
  if have_results?
    return results  # an Enumerable object
  else
    nil
  end
end

def process_results
  if results = get_results
    results.each do |result|
      # whatever...
    end
  end
end

# But that can almost always be simplified to:

# get_results() not needed.  Sometimes will just be [ ].  That's totally OK.

def process_results
  results.each do |result|  # works fine on [ ] too
    # whatever...
  end
end