Page 85 -
P. 85

To address this, the moderator must help the author understand the benefit of the criti-
                          cism. It’s the moderator’s job to make sure that the meeting does not become personal
                          criticism, and that the comments are always constructive. An effective way to do this is to
                          focus the discussion on each defect and come up with a specific resolution. It’s the job of
                          the inspection team to do more than just identify the problems; they must also come up
                          with the solutions. The moderator compiles all of the defect resolutions into an inspection
                          log (see Figure 5-1 for an example).



                             # of issues:  16
                             Review date:  March 16, 2003



                                                     Read
                             Attendees               document Time spent preparing
                             Mike (project manager)  Y       Author
                             Barbara (VP)            Y       1.0 hours
                             Quentin (requirements analyst)  Y  2.0 hours
                             Sophie (senior QA engineer  Y   3.0 hours
                             Jill (senior programmer)  Y     0.5 hours


                                                     Identified
                             Issue no. Section/page  by      Issue
                             1     Global            Quentin  The term “standard contract” should be
                                                             replaced with “pro-forma contract.”
                             2     Section           Sophie  The contents of the cells in the table are out of order.
                                   3.1.1 Line 165            It looks like some cells were marked down.
                             3     Section 3.1.2     Jill    Specify the look up is by contract number
                                   Line 190                  and artist name.
                             4     Section 3.3b      Sophie  Tile of the section needs to be changed to “Deletion
                                   Line 623                  file (maintenance).” To be consistent with section 3.2.1 #1.


                          FIGURE 5-1. Sample inspection log

                          At the top of each inspection log is information about the inspection meeting: what work

                          product was being reviewed, when it was held, who was in attendance, whether or not
                          the work product was read by each inspector, how long each inspector spent reviewing
                          the work product, and how many issues (including both defects and open issues) were
                          found. Each work product should have a unique version number, to ensure that the
                          inspection log can be matched up to the proper version of the work product.

                          The inspection moderator should ask each team member how long he or she spent
                          reviewing the work product and record that number in the log. This stands as a record of
                          how much effort went into the work product, which will help in future estimation, project
                          planning, and impact analysis activities. If any inspector failed to review the work product,
                          the moderator must halt the meeting and reschedule it in order to allow all of the inspec-
                          tors enough time to review the work product.


                                                                                                REVIEWS  77
   80   81   82   83   84   85   86   87   88   89   90