Question

I would like to refactor this piece of code which is not DRY at all 🙂

def my_method
 if session[:my_params].try(:include?, :answer)
   session[:my_params][:answer]
 elsif session[:my_params].try(:include?, :question)
   session[:my_params][:question]
 end
end

Is there a way to implicitly return either :answer or :question depending on which one is present (if present) in session[:my params]?

Answer 1

A hash will return nil if a key has no value associated in it. Because nil and false are both handled the same in conditionals, we can shorten this method very aggressively. The if session[:my_params] ensures that the session[:my_params] is not nil.

def my_method
  session[:my_params][:answer] || session[:my_params][:question] if session[:my_params]
end