Peer Code Review: Difference between revisions

From Federal Burro of Information
Jump to navigationJump to search
Line 30: Line 30:
; phase-injection
; phase-injection
: developer error, design oversight, requirements mistake, QA error
: developer error, design oversight, requirements mistake, QA error
== Dunmoor 2000: Object Oriented Reviews ==
Three approaches:
# checklist oriented
# systematic review
# use-case
checklist was the most effective and efficient at finding defects.


== Bibliography ==
== Bibliography ==

Revision as of 21:06, 17 November 2010

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

Dunmoor 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.

Bibliography