Code Review 7) Tools of the trade

Welcome to the final part of BSP Software’s ‘Code Review’ series

Part 1) Author and Reviewer Preparation
Part 2) Readibility
Part 3) Structure
Part 4) Algorithmic soundness
Part 5) Bugs found, now what?
Part 6) Implementation strategy

code-review-with-peter-gabrisPeter Gabris, MD of BSP Software Inc

In this section we focus on
Part 7) Tools of the trade

 

Code review can be done without any tools. However, a lot of time and effort can be saved by the right tool.  The most popular,  affordable and recommended general tool is CodeCollaborator by SmartBear Software.

Efficient version control system is necessary as well. The current market leader is Subversion in combination with TortoiseSVN. Both are free.

Code profilers and review tools depend on the programming language and development environment. Many can be customized according to the company coding policy.

Implementation steps

Project Managers

  1. Familiarize yourself with the Code Review Process document
  2. Read this document
  3. Ask team leader of each project to name the code reviewer for each program (or program type) in the project
  4. Accept the programming step 100% done only after you received the code review confirmation.

Team Leaders

  1. Familiarize yourself with the Code Review Process document
  2. Read this document
  3. Ask team leader of each project to name the code reviewer for each program (or program type) in the project

Programmers

  1. Familiarize yourself with the Code Review Process document
  2. Make sure each programmer in your team had selected the code reviewer and the code reviewer accepted. If a single piece of code is developed by more than one programmer, only one code reviewer is needed.
  3. Get ready to resolve disagreements between programmers and reviewers with a great diplomacy.

Management

  1. Remember that your programmers are doing very demanding work. The requirements, tools, environment and methods evolve permanently and permanent learning is an integral part of their daily job.

Feel free to leave any questions or comments below

Bookmark the permalink.

Leave a Reply

Your email address will not be published. Required fields are marked *