Weblog About All Tech Stuff

March 23, 2008

XP and code reviews

Filed under: Software — Tags: , , , — jerry35 @ 9:48 pm

CrazyBob and Chiara are debating XP… I haven’t done true XP, however I can comment on some of the principles…


Many developers have massive egos. The simple practice of requiring every checkin to have a code review seems like a basic rule. Peer review should be a requirement. I have actually heard a developer say “No one can code review my changes, they are too complex.”


I agree with CrazyBob about having someone review your changes keeps you honest. When I review changes I try to set a really high bar – and I want people to do the same for me. When you get caught up in writing tons of code, it is so easy to cut corners. I believe that we all need someone looking over our sholders to keep us honest. Maybe I’m the freak :)


I’m not sure that “pair programming” is the solution to this – but injecting some process into development is a good thing. It is all about balancing process and productivity.


http://www.simplegeek.com/permalink.aspx/102

Leave a Comment »

No comments yet.

RSS feed for comments on this post.

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out / Change )

Twitter picture

You are commenting using your Twitter account. Log Out / Change )

Facebook photo

You are commenting using your Facebook account. Log Out / Change )

Connecting to %s

Theme: WordPress Classic. Blog at WordPress.com.

Follow

Get every new post delivered to your Inbox.