Re: A better idiomatic way of doing this?!

On Tue, Sep 07, 2010 at 03:12:30AM +0900, Tim Romberg wrote:
> @Brian: you really are a gem. Thanks for all your help. I tried to 
> follow what you said put found it hard to incorporate all of it. I did 
> like this but does not seem to work so well:
...
> I get an undefined method lists_menu' for #<Menus::Menu:0x10019aa90> 
> (NoMethodError)
> I guess I have declared something wrong somewhere. Do not mean to make 
> you debug all of my code but could you maybe give a hint?! Thanks

If you reformat your code properly - see attached - it should become
clearer. You have your "end"s misplaced, so you're defining lists_menu
outside of the class, but inside the module.

You might be able to find a good editor helps you with that, or there are
utilities around for reformatting ruby source I think.  If you have a copy
of ruby 1.9 lying around, then running it with the -w flag will also give
you hints where things aren't aligned properly (I keep 1.9 around for only
that purpose)

It would also be conventional to put your 'main program', the bit which
creates the menus and runs them, outside of the enclosing module. That would
give you:

module Menus
  class Menu
    ..
  end
end
m = Menus::Menu.new

It's a bit more unwieldy because of the use of the enclosing Module, which
you have to use explicitly if you're outside.

Regards,

Brian.