CS 435 - Week 12 Lecture 2 - 2014-04-17
* "Bad smells" - things you notice in
working/passing-unit-test code that suggest
refactoring may be desired
* duplicate code
* long method
* long class
* long parameter list
* switch statement
* speculative generality
* too much communication between objects
...there are more in Chapter 7
code inspection
* another technique often used effectively
at the "unit" level
* main goal: to detect defects in work products
* is recognized as an industry "best practice"
* conducted by groups of programmers for
programmmers (e.g, by PEERS)
for IDENTIFYING problems, not solving them
* a structured process, with defined roles
* review data is recorded and used for
future monitoring