Beefy Boxes and Bandwidth Generously Provided by pair Networks
XP is just a number
 
PerlMonks  

Re^3: Howto best review code that has not been reviewed before?

by podian (Scribe)
on Aug 20, 2004 at 21:38 UTC ( #384715=note: print w/replies, xml ) Need Help??


in reply to Re^2: Howto best review code that has not been reviewed before?
in thread Howto best review code that has not been reviewed before?

This is how we did it at a telecom company as well. We just review the delta. If the delta is messy, then we look at the whole function or file.
Now I know about agile methods, I would recommend that you add a test case for each change. Add a test case and:
a) run the test case before making the change
b) change the code
c) run the test case again
I will also recommend re-factoring .
another approach is to do a code walkthrough. It is different from code review in that you take a use case (a scenario) and go through the code and see if the code will work. This is also faster than code-review.
  • Comment on Re^3: Howto best review code that has not been reviewed before?

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others chanting in the Monastery: (9)
As of 2019-10-17 16:01 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Notices?