ruby metaprogramming alternative to eval

Question

I have the following statement which does work although it uses eval:

  def resource_name
    self.class.to_s.match(/(.+)Controller/)[1].singularize
  end

  def collection
    @collection ||= eval "#{resource_name}.all(sort: [[:name, :asc]])"
  end

Is there a better way of doing this that does not use eval?

Anton · Accepted Answer · 2012-07-28 13:37:13Z

4

Use Object#send instead, after you constantize the resource name: http://ruby-doc.org/core-1.9.3/Object.html#method-i-send

answered Jul 28, 2012 at 13:37

Anton

3,0565 gold badges27 silver badges38 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Meier Over a year ago

better use public_send instead of send, it prevents you from calling private methods like system

Frederick Cheung · Accepted Answer · 2012-07-28 13:36:22Z

2

If I understand correctly you're trying to go from the string 'Product' to the constant Product, so you could just use

Object.const_get(resource_name)

If you're using Rails, you can also use constantize which has the advantage of understanding things like Module::SomeClass

answered Jul 28, 2012 at 13:36

Frederick Cheung

84.3k9 gold badges157 silver badges175 bronze badges

Collectives™ on Stack Overflow

ruby metaprogramming alternative to eval

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related