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

No Comments Yet »

No comments yet.

RSS feed for comments on this post.

Leave a comment

Blog at WordPress.com.