Peer Code Review

From Federal Burro of Information
Jump to navigationJump to search

Types of code reveiw

formal inspection
author
reveiwer (who ndoes the explained based on the review, no author input till meeting)
observer
over-the-shoulder
one walks through the code while the other watches.
pair programming
like in XP/agile
tool assisted
file gathering
combined display: diff comments defects
automated metric collection
review enforcement,
email pass-around

Code Errors

severity
major, minor
type
algorithm, documentation, data-usage, error-handling, input, output
phase-injection
developer error, design oversight, requirements mistake, QA error

Dunsmore 2000: Object Oriented Reviews

Three approaches:

  1. checklist oriented
  2. systematic review
  3. use-case

checklist was the most effective and efficient at finding defects.

reviews should not take longer than 1 hour

Things to look up

  • Path analysis
  • beta coefficient from a logarithmic least-sqares analysis is used as the measure of pair-wise correlation strength ( consider also confidence rate ~ 0.01 , P value ? )

Bibliography