Loading up a hash with variables if they are defined

[Patrick Doyle]

[Note: parts of this message were removed to make it a legal post.]

Is there a cleaner, more Rubyesque way to do this?

# Return a hash of the navigation parameters
def nav_params
params = {}

params[refix] = @prefix if @prefix
params[:category_id] = @category_id if @category_id
params[age] = @page if @page

params
end

I want to load a parameters hash with the contents of some instance
variables, but only if they are defined.

--wpd

 

[James Coglan]

[Note: parts of this message were removed to make it a legal post.]

# Return a hash of the navigation parameters
def nav_params
params = {}

params[refix] = @prefix if @prefix
params[:category_id] = @category_id if @category_id
params[age] = @page if @page

params
end

I want to load a parameters hash with the contents of some instance
variables, but only if they are defined.

params = [refix, :category_id, age].inject({}) do |hash, name|
var = instance_variable_get("@#{name}")
hash[name] = var if var
hash
end

 

[Jesús Gabriel y Galán]

Is there a cleaner, more Rubyesque way to do this?

# Return a hash of the navigation parameters
def nav_params
params = {}

params[refix] = @prefix if @prefix
params[:category_id] = @category_id if @category_id
params[age] = @page if @page

params
end

I want to load a parameters hash with the contents of some instance
variables, but only if they are defined.

irb(main):005:0> {refix => @prefix, :category_id => @category, age
=> @page}.reject {|k,v| v.nil?}
=> {refix=>"asdasdf", :category_id=>"cat"}

Jesus.

 

[Patrick Doyle]

[Note: parts of this message were removed to make it a legal post.]

# Return a hash of the navigation parameters
def nav_params
params = {}

params[refix] = @prefix if @prefix
params[:category_id] = @category_id if @category_id
params[age] = @page if @page

params
end

I want to load a parameters hash with the contents of some instance
variables, but only if they are defined.

params = [refix, :category_id, age].inject({}) do |hash, name|
var = instance_variable_get("@#{name}")
hash[name] = var if var
hash
end

Thanks... that's certainly along the lines of what I was searching for.
I'll go read about #instance_variable_get now.

--wpd

 

[Joel VanderWerf]

Is there a cleaner, more Rubyesque way to do this?

# Return a hash of the navigation parameters
def nav_params
params = {}

params[refix] = @prefix if @prefix
params[:category_id] = @category_id if @category_id
params[age] = @page if @page

params
end

I want to load a parameters hash with the contents of some instance
variables, but only if they are defined.

Watch out: "only if they are defined" is not the same as "only if they
are not nil or false". If you want the former, then this might be what
you are looking for:

class C
NAV_VARS = %w{ @prefix @category_id @page }
NAV_VAR_KEYS = NAV_VARS.inject({}) {|h,v| h[v] = v.delete("@").to_sym; h}

def nav_params
(instance_variables & NAV_VARS).inject({}) do |params,var|
params[NAV_VAR_KEYS[var]] = instance_variable_get(var)
params
end
end
end

c = C.new
c.instance_eval do
@prefix = "foo"
@page = false
end

p c.nav_params # ==> {refix=>"foo", age=>false}