Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                

What Is Software Review

Download as docx, pdf, or txt
Download as docx, pdf, or txt
You are on page 1of 8

What is Software Review?

Software review is an important part of "Software Development Life Cycle


(SDLC)" that assists software engineers in validating the quality, functionality,
and other vital features and components of the software. As mentioned above, it
is a complete process that involves testing the software product and ensuring
that it meets the requirements stated by the client.

It is systematic examination of a document by one or more individuals, who


work together to find & resolve errors and defects in the software during the
early stages of Software Development Life Cycle (SDLC). Usually performed
manually, software review is used to verify various documents like requirements,
system designs, codes, "test plans", & "test cases".

Formal Review Vs Informal Review:


Formal and informal review are two very important types of reviews that are
used most commonly by software engineers to identify defects as well as to
discuss ways to tackle these issues or discrepancies. Therefore, to understand
these important types of software review, following is a comparison of the two:

Formal Review:
A type of peer review, "formal review" follows a formal process and has a
specific formal agenda. It has a well structured and regulated process, which is
usually implemented at the end of each life cycle. During this process, a formal
review panel or board considers the necessary steps for the next life cycle.
Features of Formal Review:

 This evaluates conformance to specification and various standards.

 Conducted by a group of 3 or more individuals.


 The review team petitions the management of technical leadership to act
on the suggested recommendations.

 Here, the leader verifies that the action documents are verified and
incorporated into external processes.

 Formal review consists of six important steps, which are:

o Planning.

o Kick-off.

o Preparation.

o Review meeting.

o Rework.

o Follow up.

Informal Review:
Unlike Formal Reviews, Informal reviews are applied multiple times during the
early stages of software development process. The major difference between
the formal and informal reviews is that the former follows a formal agenda,
whereas the latter is conducted as per the need of the team and follows an
informal agenda. Though time saving, this process is not documented and does
not require any entry criteria or large group of members.
Features of Informal Review:

 Conducted by a group of 2-7 members, which includes the designer an any


other interested party.

 Here the team identifies errors & issues as well as examine alternatives.

 It is a forum for learning.


 All the changes are made by the software designer.

 These changes are verified by other project controls.

 The role of informal review is to keep the author informed and to improve
the quality of the product.

Phases of a formal review


In contrast to informal reviews, formal reviews follow a formal process. A typical
formal review process consists of six main steps:

1 Planning - The review process for a particular review begins with a 'request
for review' by the author to the moderator (or inspection leader). A moderator
is often assigned to take care of the scheduling (dates, time, place and
invitation) of the review.

2 Kick-off - An optional step in a review procedure is a kick-off meeting. The goal


of this meeting is to get everybody on the same wavelength regarding the
document under review and to commit to the time that will be spent on
checking.

3 Preparation - The participants work individually on the document under


review using the related documents, procedures, rules and checklists provided.
The individual participants identify defects, questions and comments, according
to their understanding of the document and role. All issues are recorded,
preferably using a logging form.

4 Review meeting - The meeting typically consists of the following elements


(partly depending on the review type): logging phase, discussion phase and
decision phase. During the logging phase the issues, e.g. defects, that have been
identified during the preparation are mentioned page by page, reviewer by
reviewer and are logged either by the author or by a scribe. For a more formal
review, the issues classified as discussion items will be handled during this
meeting phase. Informal reviews will often not have a sep-arate logging phase
and will start immediately with discussion. At the end of the meeting, a decision
on the document under review has to be made by the participants. The most
important exit criterion is the average number of critical and/or major defects
found per page (e.g. no more than three critical/major defects per page).

5 Rework - Based on the defects detected, the author will improve the
document under review step by step.

6.Follow-up - The moderator is responsible for ensuring that satisfactory actions


have been taken on all (logged) defects, process improvement suggestions and
change requests.
Types of reviews:
1. Walkthrough
A walkthrough is characterized by the author of the document under review
guiding the participants through the document and his or her thought processes,
to achieve a common understanding and to gather feedback. This is especially
useful if people from outside the software discipline are present, who are not
used to, or cannot easily understand software development documents. The
content of the document is explained step by step by the author, to reach
consensus on changes or to gather information.

Within a walkthrough the author does most of the preparation. The partici-
pants, who are selected from different departments and backgrounds, are not
required to do a detailed study of the documents in advance. Because of the
way the meeting is structured, a large number of people can participate and this
larger audience can bring a great number of diverse viewpoints regarding the
contents of the document being reviewed as well as serving an educational
purpose. If the audience represents a broad cross-section of skills and disci-
plines, it can give assurance that no major defects are 'missed' in the walk-
through. A walkthrough is especially useful for higher-level documents, such as
requirement specifications and architectural documents.

The specific goals of a walkthrough depend on its role in the creation of the
document. In general the following goals can be applicable:
• to present the document to stakeholders both within and outside the soft
ware discipline, in order to gather information regarding the topic under
documentation;
• to explain (knowledge transfer) and evaluate the contents of the docu ment;

• to establish a common understanding of the document;


• to examine and discuss the validity of proposed solutions and the viability of
alternatives, establishing consensus.
Key characteristics of walkthroughs are:
• The meeting is led by the authors; often a separate scribe is present.
• Scenarios and dry runs may be used to validate the content.
• Separate pre-meeting preparation for reviewers is optional.
2. Technical review
A technical review is a discussion meeting that focuses on achieving consensus
about the technical content of a document. Compared to inspections, technical
reviews are less formal. During technical reviews defects are found by experts,
who focus on the content of the document. The experts that are needed for a
technical review are, for example, architects, chief designers and key users. In
practice, technical reviews vary from quite informal to very formal.
The goals of a technical review are to:
• assess the value of technical concepts and alternatives in the product and
project environment;

• establish consistency in the use and representation of technical concepts;


• ensure, at an early stage, that technical concepts are used correctly;
• inform participants of the technical content of the document.
Key characteristics of a technical review are:

• It is a documented defect-detection process that involves peers and technical


experts.
• It is often performed as a peer review without management participation.
• Ideally it is led by a trained moderator, but possibly also by a technical expert.
• A separate preparation is carried out during which the product is examined
and the defects are found.
• More formal characteristics such as the use of checklists and a logging list or
issue log are optional.

3.Inspection
Inspection is the most formal review type. The document under inspection is
prepared and checked thoroughly by the reviewers before the meeting, compar-
ing the work product with its sources and other referenced documents, and
using rules and checklists. In the inspection meeting the defects found are
logged and any discussion is postponed until the discussion phase. This makes
the inspection meeting a very efficient meeting.
Depending on the organization and the objectives of a project, inspections can
be balanced to serve a number of goals. For example, if the time to market is
extremely important, the emphasis in inspections will be on efficiency. In a
safety-critical market, the focus will be on effectiveness.
The generally accepted goals of inspection are to:
• help the author to improve the quality of the document under inspection;
• remove defects efficiently, as early as possible;
• improve product quality, by producing documents with a higher
level of quality;
• create a common understanding by exchanging information
among the inspection participants;
• train new employees in the organization's development process;
• learn from defects found and improve processes in order to
prevent recur rence of similar defects;
• sample a few pages or sections from a larger document in order to
measure the typical quality of the document, leading to improved
work by individuals in the future, and to process improvements.
Key characteristics of an inspection are:
• It is usually led by a trained moderator (certainly not by the author).
• It uses defined roles during the process.
• It involves peers to examine the product.
• Rules and checklists are used during the preparation phase.
• A separate preparation is carried out during which the product is
examined and the defects are found.
• The defects found are documented in a logging list or issue log.
• A formal follow-up is carried out by the moderator applying exit criteria.
• Optionally, a causal analysis step is introduced to address process
improve ment issues and learn from the defects found.
• Metrics are gathered and analyzed to optimize the process.

You might also like