130 likes | 243 Views
Code/Design Inspections. Reference. Technical Report TR00.2763 June 10, 1976 Written by Michael E. Fagan Kingston Laboratory. Design Review. General survey of something in words Educational - a presentation of what has been done and what is planned. Design Inspection.
E N D
Reference • Technical Report TR00.2763 June 10, 1976 • Written by Michael E. Fagan • Kingston Laboratory
Design Review • General survey of something in words • Educational - a presentation of what has been done and what is planned
Design Inspection • Thorough comparison of objectives vs design and test criterion to prove design • Scrutinize carefully, investigate, interrogate • All instructions addressed at least once during inspection
Inspection Process • Planning • Participants designated • Schedules for inspection established • Participants briefed • Overview • For education, designer gives tutorial • Preparation • For education, inspectors prepare from distributed material
Inspection Process Continued • Inspection • find errors not solutions • Inspection reader presents code • examine product against specifications • record errors • Rework • Allow time in schedule for rework • Must be complete and verified by moderator prior to start of next development phase
Inspection Process Continued • Follow up • Moderator documents results in inspection report within 24 hours • Design change reports written • include results in database
Moderator • Key person in successful inspection • Independent, capable, leader • Preserve objectivity • Increase integrity of inspection • Manage inspection team
Moderator must counteract • Inability of participants to give/take criticism • Lack of enthusiasm for finding errors • Doing too much at one meeting • Arguments over style or technique • Author’s fear of management reprisal for errors
Inspection Outline • Overview • Designer describes overall design • Designer describes detail design • Distribute design documentation to participants • Preparation • Participants “do their homework” to understand design, intent, and logic • Some errors found at this phase
Inspection outline continued • Inspection • Every piece of logic covered • Every branch taken • Inspection team must find errors • Moderator controls meeting, inspection is to find not fix errors • Errors and severity documented by moderator
Inspection outline continued • Rework • Errors found in inspection must be resolved by coder/designer • Follow up • Moderator ensures every error is resolved • TR states if >5% of material reworked, 100 % reinspection • <5%, moderator verifies rework or decides on reconvening of inspection team
Successful approach to inspection process • Inspections limited to 2 hour sessions • Presenter of code not author • Objectivity • Programmer backup