Click here to Skip to main content
11,479,170 members (56,170 online)
Click here to Skip to main content

Tagged as

Code Review: What's all the fuss about?

, 28 Aug 2010 CPOL 5.9K 1
Rate this:
Please Sign up or sign in to vote.
My brief thoughts on how and what a code review process is and should be about

"Code review", for those of you who are not familiar with the terminology, is the practice of reviewing code written by another developer/programmer in a development team. Here are my brief thoughts on how and what a code review process is and should be about.

Is there an Emotional Side to it?

Much too often, code reviews are sought with distress. This can easily be attributed to a lack of understanding as to the purpose of a code review. A code review policy in a team should be aimed at creating and maintaining a consistent, structured and standardized code base. Nothing more, nothing less.

So We Got the Essence of it... Now What?

To achieve all of the above, we need to come up with a list of processes and tools that can help achieve our goals. To get you started, here are a few:

  1. Determine when you will conduct your code review, is it after each development iteration, is it after testing?
  2. To achieve standardization, create a guide and update it regularly with necessary checks you want to cover with your code review. Here are a few to start with.
    1. Check for hard coded data.
    2. Check for performance bottlenecks IO operations, DB connections, etc.
    3. Check for adherence to naming convention.
  3. Train your team in your code review process.
  4. Regularly discuss your code review policy with all involved. Remember code review is not a solitary exercise and your code review policy will have a wider acceptance when everyone's opinions/ideas are heard.
  5. The code review exercise is not one just for your Team Leads. Anyone and everyone in the team should have the ability to conduct a code review. This will help inculcate your coding standards across the team.

Tools

Here are some tools that can help with your code review objectives:

  1. A good code repository that can help you identify the code required to be reviewed
  2. A good diff tool that can highlight the differences between the latest and previous versions of your code
  3. A good refactoring tool that will automate most of your code review process

Conclusion

I'd like to end my thoughts on the code review process with the following.

For the reviewer: Focus on the code, not the developer! This is an opportunity to learn for you too, you can learn how to do it or how not to do it, so keep an open mind.

For the developer: This is not a critic of your coding skills, but an exercise to improve the code in the code base.

License

This article, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Share

About the Author

Ritesh Ramesh
Architect Infosolvex Solutions Inc
Australia Australia
Ritesh is an IT consultant with over ten years of experience in the IT industry varying from consultation, architecture, design, development to technical management. He has a strong background in solutions and applications architecture with a focus on Microsoft’s .Net platform. His area of expertise spans design and implementation of client/server, database and web-based systems. He has worked with C#, ASP.NET 1.1 and 2.0, ADO.NET, Web Services and SQL technology on several enterprise class projects.




Freedom is not worth having if it does not include the freedom to make mistakes.
Mahatma Gandhi

Comments and Discussions

 
-- There are no messages in this forum --
| Advertise | Privacy | Terms of Use | Mobile
Web03 | 2.8.150520.1 | Last Updated 29 Aug 2010
Article Copyright 2010 by Ritesh Ramesh
Everything else Copyright © CodeProject, 1999-2015
Layout: fixed | fluid