1 / 21

Architecture Review The Process

Architecture Review The Process. CS 121 “ Ordering Chaos ”. Today: What will happen…. “ Mike ” Michael A. Erlinger. ArchReview.f12.ppt. Administrivia. Issues???. Architecture Review. What: Have skilled people assess your work Requires that you have work product that is reviewable

dahlia-garner
Download Presentation

Architecture Review The Process

An Image/Link below is provided (as is) to download presentation Download Policy: Content on the Website is provided to you AS IS for your information and personal use and may not be sold / licensed / shared on other websites without getting consent from its author. Content is provided to you AS IS for your information and personal use only. Download presentation by click this link. While downloading, if for some reason you are not able to download a presentation, the publisher may have deleted the file from their server. During download, if you can't get a presentation, the file might be deleted by the publisher.

E N D

Presentation Transcript

  1. Architecture ReviewThe Process CS 121 “Ordering Chaos” • Today: • What will happen…. “Mike” Michael A. Erlinger ArchReview.f12.ppt

  2. Administrivia • Issues???

  3. Architecture Review What: • Have skilled people assess your work • Requires that you have work product that is reviewable Why: • Find problems early • Find omissions

  4. Types of reviews • Requirements review • Architectural review – we will do • UI design review – we will do • Component design review • Test plan review – we will do • Code review – we will do

  5. Architectural Review The Questions: 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? What else?

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

  7. Architecture Review • 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

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

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

  10. Review package (hyperlinked) Introduction Agenda Criteria Work products Reviewer assignments

  11. Review Package - Introduction 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 From Proposal From Goal Stack

  12. Review Package (hyperlinked) Introduction Agenda Criteria Work products Reviewer assignments Order materials will be reviewed.

  13. Review - Criteria We will provide rubric: Phase 2 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.

  14. Review package (hyperlinked) Intro Agenda Criteria Work products Reviewer assignments --- Reviewing Team Use Cases, Class Diagrams, Sequence Diagrams, etc.

  15. 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 material, e.g., share Use Cases, UML

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

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

  18. Preparation Author Moderator Reviewers prepare work products examine work products read package ready for review? study work products Y N prepare review package prepare comments schedule review send out packages DESIGN REVIEW

  19. 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

  20. Review process cont. Reviewers • Raise questions, concerns Author and design team • Answers questions but is otherwise silent Stick to specified level Avoid re-specifying/designing system Avoid getting sidetracked

  21. The End

More Related