Beefy Boxes and Bandwidth Generously Provided by pair Networks
Syntactic Confectionery Delight
 
PerlMonks  

Re^6: VarStructor 1.0

by pfaut (Priest)
on Sep 02, 2004 at 13:59 UTC ( #387914=note: print w/replies, xml ) Need Help??


in reply to Re^5: VarStructor 1.0
in thread VarStructor 1.0

Complexity increases as scope widens. If you have references to a variable throughout a 3000 line script, then you need to understand all 3000 lines to know how that variable is used. Multiply this by the number of global variables.

On the other hand, if you package functionality into smaller units, all you need to understand is the interfaces provided by each unit. A well designed unit would present an intuitive interface and hide a lot of implementation detail. If the interface is well thought out, the implimentation could easily be changed without affecting any other code.

If I had to maintain that program, my first inclination would be to refactor it into several smaller units that could each be understood on its own. If I didn't have the time, I'd locate all of the globals, declare them in one place, and write a subroutine to initialize them. I'd then call this initialization routine at the beginning of the loop. After I got this done and put the program to use, I'd go back and refactor it so it could be properly maintained and extended if necessary. Under no circumstances would I attempt to extend code like this without refactoring it first.

90% of every Perl application is already written.
dragonchild

Replies are listed 'Best First'.
A reply falls below the community's threshold of quality. You may see it by logging in.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others avoiding work at the Monastery: (8)
As of 2020-09-18 14:47 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    If at first I donít succeed, I Ö










    Results (112 votes). Check out past polls.

    Notices?