4 - AAS Software Reviews - All Sides
4 - AAS Software Reviews - All Sides
119
Software reviews
120
• Walkthrough
Presentation done by the author of an artifact to a peer audience to provide the
author comments and feedbacks. Usually, involves limited documentation of the
process and the issues uncovered, which makes defect tracking difficult.
• Formal inspection
Formal and managed peer review process.
Henrique M adeiraAdapted from Dr Issa Traore slides, University of Victoria, Canada Analysis of Software Artifacts, DEI-FCTUC, 2022/2023 121
121
• Walkthrough
Presentation done by the author of an artifact to a peer audience to provide the
author comments and feedbacks. Usually, involves limited documentation of the
process and the issues uncovered, which makes defect tracking difficult.
• Formal inspection
Formal and managed peer review process.
Henrique M adeiraAdapted from Dr Issa Traore slides, University of Victoria, Canada Analysis of Software Artifacts, DEI-FCTUC, 2022/2023 122
122
• Goal
To identify defects (i.e., faults or bugs) as closely as possible to the point of
occurrence in order to facilitate corrective actions.
M. E. Fagan, “Design and code inspections to reduce errors in program development”, IBM
System Journal. vol. 15, No. 3, pages 183–211, 1976.
123
124
125
126
127
What is a defect?
128
129
• Common steps
– Use a static analysis technique
– Perform a visual examination of the software products/artifacts
– Detect (and correct later on):
• Defects
• Violation of design standards
• Other problems
130
131
Henrique M adeira Picture from Chris Durand, CTO, Bridge360 Analysis of Software Artifacts, DEI-FCTUC, 2022/2023 132
132
133
134
135
136
Inspection materials
• Source document
• Checklists
• Supporting documents
• Invitations
• Inspection plan
• Issue/defect log
• Data summary
137
Inspections roles
• The roles established for the inspection
– Designated moderator
An inspector responsible for organizing and reporting on inspection
– Author
Developer of software product
– Reader
An inspector who guides the examination of the product (often reads aloud)
– Recorder
An inspector who enters all the defects found on the defect list
– Inspector
Member of inspection team. Often chosen to represent specific role such as
designer, tester, technical writer, SQA, etc.
138
139
Steps of inspection
140
141
• Planning
• Overview • Individual activity (done by all inspectors)
• Author provides all the material required for
• Preparation inspection.
• Meeting • Inspectors study the material and complete
inspection log.
• Rework • Defects are noted at this step, but not collected
• Follow-up
142
• Rework
• Follow-up
143
144
145
• Planning
• Performed by the author in response to defect
disposition determined at meeting
• Overview
• Preparation
• Meeting
• Rework
• Follow-up
146
• Follow-up
147
Entry Exit
Planning/ Preparation Inspection Rework
Overview
148
149
150
151
152
153
154
155
156
Requirements inspections
157
158
• Unambiguous
• Complete
This is the key for checklist
• Verifiable of a review of requirements
• Consistent
• Modifiable
• Traceable
• Usable
159
160
161
162
163
164
Code inspections
• Inspectors use checklists for each programming language to
help in the task of finding defects. The fault classes are
similar across the different languages.
• In general, the defect classification used is general, and
works for different types of products
• Elaborate defect classifications can be used such as
Orthogonal Defect Classification (ODC).
(addressed later on in the course)
165
166
Defects Found: ____ Major, ____ minor Defects Corrected: ____ Major ____ minor
1.
Used by individual inspectors during
2. preparation and used as inspection log in
the meeting by the moderator.
3.
4.
167
6.
7.
9.
Analysis of Software Artifacts
Departamento de Engenharia Informática, FCTUC
Inspection Identification:
Project:
Inspection ID:
Meeting Date:
Verifier: __________________
Inspection Lessons Learned Questionnaire
Projected Rework Completion Date:
1. Did the inspection meet the author’s and team’s objectives? If not, why not?
2.Copyright
Does the© team
2001 feel
by Karl
theyE.were
Wiegers. Permission
able to is granted
significantly to use,
improve modify,of
the quality and
thedistribute this document.
work product
through the inspection?
3. Did everyone have sufficient time to do preparation? If not, how much time do they need prior
to the inspection meeting?
4. Did anyone use the checklist or rules for this type of work product during preparation? Was it
helpful in finding defects? Can the checklist or rules be improved?
169 5. Were the right participants present? If not, who was missing or didn’t need to be there?
Typo List
Record any typographical errors that you find during your inspection preparation on this list,
including spelling, grammatical, formatting, and style errors. These should be corrected but need not
be discussed at the inspection meeting. They will not be counted as defects.
Inspector:
Scheduled Inspection Meeting Date:
Work Product Description:
170
Copyright © 2001 by Karl E. Wiegers. Permission is granted to use, modify, and distribute this document.