Beefy Boxes and Bandwidth Generously Provided by pair Networks
Clear questions and runnable code
get the best and fastest answer
 
PerlMonks  

Re: Code Review - What Medium?

by kirbyk (Friar)
on May 31, 2005 at 23:01 UTC ( [id://462251]=note: print w/replies, xml ) Need Help??


in reply to Code Review - What Medium?

Through unit testing.

While some other languages get you in bad habits of writing massive monolithic chunks of code before you try to (compile and) run it, Perl is ideally suited to running frequent tests and writing tests scripts to hit the code. My eyes will gloss over things that actually running it won't. It's not a great use of time for me to read my own code again - but writing a re-usable test to verify it works, and it still works after things change, that's where it's at.

Alternate answer: on my coworker's screen. While I'm not likely to see bugs or a better approach in my own code, someone else sure is. It's enormously instructive to have an equally experienced or better programmer review your code, and one of the best ways to learn new techniques.

-- Kirby, WhitePages.com

Replies are listed 'Best First'.
Re^2: Code Review - What Medium?
by halley (Prior) on Jun 01, 2005 at 14:17 UTC
    Unit testing is good, however...

    Correct behavior is not the only thing that a code review should cover. Performance, documentation, readability, maintainability, and whether or not you're using personal idioms that are not typical for your peers. Remember, the source code is ultimately for the human's benefit at least as much as it is for the computer's benefit.

    --
    [ e d @ h a l l e y . c c ]

Log In?
Username:
Password:

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

How do I use this?Last hourOther CB clients
Other Users?
Others imbibing at the Monastery: (5)
As of 2024-04-25 13:37 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found