Beefy Boxes and Bandwidth Generously Provided by pair Networks
Welcome to the Monastery
 
PerlMonks  

Re: refactoring - is this layout a mess or should we leave well enough alone?

by ikegami (Patriarch)
on Oct 11, 2007 at 17:27 UTC ( [id://644285]=note: print w/replies, xml ) Need Help??


in reply to refactoring - is this layout a mess or should we leave well enough alone?

Method lookups are cached, so I'd say "no" if the only reason is to eliminate a search that doesn't even occur.

Replies are listed 'Best First'.
Re^2: refactoring - is this layout a mess or should we leave well enough alone?
by archimago (Pilgrim) on Oct 11, 2007 at 17:33 UTC
    ok thanks. do you find the layout convoluted or does it make sense to you?

      It don't find it very useful. It's not that it does anything bad, it's just that I don't see what value it adds. None of the "good" points hold any value for me.

      I'm not sure I agree with your second "good" point anyway. For example, when using get_client_id_for_username, the programmer knows he's using EG::DB::Clients. He *should* know he's using EG::DB::Clients. That's why the methods were placed in a seperate module in the first place. Perl doesn't care if the functions are in seperate modules or not. By hiding the partitioning of the functions, you're nullifying the advantage of creating the modules in the first place.

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://644285]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others rifling through the Monastery: (5)
As of 2024-03-29 13:56 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found