Presentation is loading. Please wait.

Presentation is loading. Please wait.

Reviews mae, fall 11. Review What: – Have skilled people assess your work –Requires that you have work product that is reviewable Why: – Find problems.

Similar presentations


Presentation on theme: "Reviews mae, fall 11. Review What: – Have skilled people assess your work –Requires that you have work product that is reviewable Why: – Find problems."— Presentation transcript:

1 Reviews mae, fall 11

2 Review What: – Have skilled people assess your work –Requires that you have work product that is reviewable Why: – Find problems early – Find omissions

3 Types of reviews Requirements review Architectural review UI design review Component design review Test plan review Code review

4 Architectural review Is it capable of meeting the requirements? Is it practical to build, test, and support? Does it satisfy good design principles? Is it documented such that someone could review it.

5 Preparation prepare work products + plan examine work products prepare review package ready for review? read package study work products prepare comments, Rubric schedule review send out packages calendar dates Author (from Team being reviewed) Moderator (from team doing review) Reviewers (team doing review) Y N

6 Work products (design) –use cases –class diagrams –primary responsibility of each class –sequence diagrams –models –etc. Plan –what will be reviewed and the order –what types of issues will be covered don’t hide risks

7 Preparation prepare work products + plan examine work products + plan prepare review package ready for review? read package study work products prepare comments schedule review send out packages AuthorModerator Reviewers Y N

8 Preparation prepare work products examine work products prepare review package ready for review? read package study work products prepare comments schedule review send out packages AuthorModerator Reviewers Y N

9 Review package (hyperlinked) Intro Agenda Criteria Work products Reviewer assignments

10 Review package - Intro Background –What project are we discussing –What do reviewers need to know about it history, key problems, important decisions, etc. –Where can reviewers find more info requirements, designs, analysis Goals for review –specific work products to be reviewed –scope (what is in/out of bounds) –what approval means

11 Review package (hyperlinked) Intro Agenda Criteria Work products Reviewer assignments Order materials will be reviewed.

12 Review - criteria We will provide rubric: Are the uses cases appropriate? Is the architecture able to realize the use cases? Is the design feasible? Are underlying models clear? Does the design follow good design principles? Does it use patterns effectively The author and moderator should amend as appropriate.

13 Review package (hyperlinked) Intro Agenda Criteria Work products Reviewer assignments Use cases, class diagrams, sequence diagrams, etc.

14 Review package (hyperlinked) Intro Agenda Criteria Work products Reviewer assignments What each reviewer is responsible for; e.g. give each reviewer responsibility for a subset of the classes

15 Preparation prepare work products examine work products prepare review package ready for review? read package study work products prepare comments schedule review send out packages AuthorModerator Reviewers Y N We are going to schedule it !!!

16 Preparation prepare work products examine work products prepare review package ready for review? read package study work products prepare comments schedule review send out packages AuthorModerator Reviewers Y N

17 Preparation prepare work products examine work products prepare review package ready for review? read package study work products prepare comments schedule review send out packages AuthorModerator Reviewers Y N DESIGN REVIEW

18 Review process Moderator – not a member of reviewed team –keeps review moving –ensures all voices are heard and key points covered –ensures decisions are made: accepted, major/minor revisions, further review Recorder – member of the reviewed team –takes notes, records all issues raised and decisions reached, all questions, suggestions, and action items –publishes a report of the review

19 Review process cont. Reviewers –Raise questions, concerns Author and design team –Answers questions but is otherwise silent

20 Review cont. Stick to specified level Avoid re-specifying/designing system Avoid getting sidetracked


Download ppt "Reviews mae, fall 11. Review What: – Have skilled people assess your work –Requires that you have work product that is reviewable Why: – Find problems."

Similar presentations


Ads by Google