Monday, June 13, 2011

Code Complete by Steve McConnell : Chapter 24

The author focuses on importance of reviews in this chapter. The author describes the basic reason behind technical reviews: All developers have confirmation bias so they are blind to some trouble spots in their work and other people do not have the same blind spots.

The author points out that code inspection has higher effectiveness compared to software testing or unit testing and the defect discovered by reviews are different from testing.

Technical reviews help in assessing the progress and quality of the project.

The author then discuss about inspections and how they differ from other kind of reviews. The author then discuss different roles played during an inspection including:
  1. Moderator
  2. Author
  3. Scribe
  4. Reviewer
  5. Management
The author emphasizes to have no less than three participants in an inspection since author, moderator and reviewer roles should be played by different people.

The author then outlines the general procedure for inspection:
  1. Planning
  2. Overview
  3. Preparation
  4. Inspection meeting
  5. Inspection report
  6. Rework
  7. Follow-up
  8. Third-hour meeting
  9. Fine-tuning the inspection
The author also discusses the problem of ego arising during code-inspection. The purpose of the inspection should be to find defects and not to criticize author of the code or to debate about who is right or wrong.

The author then discusses about other kind of reviews including:
  1. Walkthroughs: The author presents several things common for all walkthroughs
    • The purpose of a walkthrough is to improve the quality of a program rather than assess it.
    • It is a chance for senior programmers to pass on experience and corporate culture to junior programmers.
        The author then compares inspections with walkthroughs on several properties.
  2. Code Reading: The author cites a study saying that more defects are discovered in code reading than in testing per hour of effort. It is different from walkthrough or review such that there is no meeting done.
  3. Dog and Pony shows: The product is demonstrated to a customer.

No comments:

Post a Comment