Beefy Boxes and Bandwidth Generously Provided by pair Networks
We don't bite newbies here... much

Re: Re: Touching it when it ain't broke

by pdcawley (Hermit)
on Aug 11, 2002 at 22:39 UTC ( #189348=note: print w/replies, xml ) Need Help??

in reply to Re: Touching it when it ain't broke
in thread Touching it when it ain't broke

Hear hear! I've worked on site where the existing code is sufficiently complex, and the existing test suite so non existent, that we've ended up applying the 'Extract Algorithm' refactoring to give ourselves a restricted set of well tested functionality, and then added tests and refactored towards the same functionality as the old code.

It seems scary at first, but old code with no tests is scarier still. Especially when that code is a tightly coupled mess of objects with no consistent naming style, let alone a consistent code layout style. Our new code doesn't do everything the old code does. Yet. But it's reasonably clear, naming is consistent and IT HAS TESTS! Tests are great. I really cannot recommend them highly enough.

  • Comment on Re: Re: Touching it when it ain't broke

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://189348]
[marto]: can I nominate users ? :P
[Discipulus]: AM will rulez!

How do I use this? | Other CB clients
Other Users?
Others having an uproarious good time at the Monastery: (11)
As of 2018-05-22 08:30 GMT
Find Nodes?
    Voting Booth?