Home > Great Links > On code reviews and maintainability

On code reviews and maintainability

Prakash has a very good post on Maintainability here. Please go read it first.

I will expand on pair programming point a bit and add my perspective.

You add more members to the team. They start writing code. Even though you can educate your developers about how the code is written, but everyone brings some baggage with them (culture, how they worked in previous projects, companies etc…)

 

There is a gap in terms of skill level, experience level and understanding (being on the same page with respect to the architecture, design, development practices) when the team ramps up. We need to pair at least till the gap is bridged.

I see two aspects to maintainability, preventive and reactive. Both are equally important. Pairing is probably one of the tools in the the former and reviews the later. Reactive probably has a negative connotation to it so i might reword it as ongoing effort or continuous effort.

This just like how we should handle Performance. Design for performance and still plan for performance tuning when things fall out of place.

To generalize aim for the best and ideal, you won’t reach there because there is always a gap between ideal and real. So when things fall out of place have checks and counter measures to pull them back on track.

As some one who hates reviews I have come to a compromise that they are necessary evil.

Advertisements
  1. No comments yet.
  1. No trackbacks yet.

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 )

Google+ photo

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

Connecting to %s

%d bloggers like this: