drawohara - Latest Comments in [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Pit Capitain — Thu, 11 Sep 2008 14:08:05 -0000

Yes, the code is great. I especially liked your idea of using a lambda for (re)defining the methods in the helper module.

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

drawohara — Thu, 11 Sep 2008 11:47:41 -0000

fixes applied here

http://s3.amazonaws.com/dra...

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

drawohara — Thu, 11 Sep 2008 11:39:43 -0000

You can say that again!

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

drawohara — Thu, 11 Sep 2008 11:39:32 -0000

hey pit - nice to see you around here ;-)

1 - great fix. it also fixes an issue this code had - that of forever littering the singleton class with module methods.

2 - yup. good call.

i'm very happy with the approach thus far though - it's vastly simply than any approach i've found thus far for rapidly building intuitive and dsls.

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Pit Capitain — Thu, 11 Sep 2008 05:13:49 -0000

Sorry for the indentation. Glad we aren't using Python :-)

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Pit Capitain — Thu, 11 Sep 2008 05:12:21 -0000

Nice code as always, Ara.

Just two remarks:

By using an instance variable in Methods.for it looks like you wanted to create the anonymous module only once, but you create a new one every time (actually two modules, but that's only a typo I suppose). If you want to create the module only once, you could change the code as follows:

    def Methods.new
      Module.new do
        def self.remove!
          instance_methods.each{|m| remove_method m}
        end
      end
    end

    def Methods.for object
      singleton_class_for(object) do
        unless defined? @dsl_methods
          @dsl_methods = Methods.new
          include @dsl_methods
        end
        @dsl_methods.module_eval(&Methods)
        @dsl_methods
      end
    end

The second remark: you don't need instance_eval and the block parameter in the #dsl method, because you already call it in the scope of self. So all you need is a yield.

Regards,
Pit

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Michael Guterl — Sat, 06 Sep 2008 17:09:54 -0000

in that case, next time i'm in boulder i'm buying you some beer! that is the least i owe you.

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

drawohara — Sat, 06 Sep 2008 13:52:15 -0000

it's quite simple really -- coffee and beer go in: code comes out. ;-)

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

drawohara — Sat, 06 Sep 2008 13:51:44 -0000

i'm psyched to play with it -- been trying to figure how to do something like that for a long time! ;-)

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Fri, 05 Sep 2008 07:13:49 -0000

you should write how your brain works!
or simpler, how to become a better rubyist.

Re: [Ruby] temporary methods for dsls in ruby, aka - how to unextend

Michael Guterl — Fri, 05 Sep 2008 00:11:34 -0000

This is really interesting. Your ruby-fu never ceases to amaze me.

Thanks for all that you contribute to the ruby community!