TMA Training

Center

SW-DI-005-06 Java Checklist

Code Review Check List

For Java Language
Project ID:
Checked By:
Note:

Work product:
Date :

I - DEVIATION OBJECTIVE
#
I.1 DEVIATION
1.
Does the code correctly implement the design?
2.
Does the code implement more than the design?
3.
Is every parameter of every method passing mechanism (value or reference) appropriate?
4.
Does every method return the correct value at every method return point?
II OMISSION OBJECTIVE
#
II.1 OMISSION
5.
Does the code completely implement the design?
6.
Are there any requirements of design that were not implemented?
III - DEFECT OBJECTIVE
#
III.1 Variable and Constant Declaration
7.
Are descriptive variable and constant names used in accord with naming conventions?
8.
Is every variable correctly typed?
9.
Is every variable properly initialized?
10.
Are all for-loop control variables declared in the loop header?
11.
Are there variables that should be constants?
12.
Are there attributes that should be local variables?
13.
Do all attributes have appropriate access modifiers (private, protected, public)?
14.
Are there static attributes that should be non-static or vice-versa?
#
III.2 Method Definition
15.
Are descriptive method names used in accord with naming conventions?
16.
Do all methods have appropriate access modifiers (private, protected, public)?
17.
Is every method parameter value checked before being used?
18.
Are there static methods that should be non-static or vice-versa?
#
III.3 Class Definition
19.
Does each class have an appropriate constructor?
20.
Do any subclasses have common members that should be in the superclass?
21.
Can the class inheritance hierarchy be simplified?
#
III.4 Data Reference
22.
For every array reference: Is each subscript value within the defined bounds?
23.
For every object or array reference: Is the value certain to be non-null?
#
III.5 Computation/Numeric
24.
Are there any computations with mixed data types?
25.
Is overflow or underflow possible during a computation?
26.
For each expressions with more than one operator: Are the assumptions about order of evaluation
and precedence correct?
27.
Are parentheses used to avoid ambiguity?
28.
Does the code systematically prevent rounding errors?
29.
Does the code avoid additions and subtractions on numbers with greatly different magnitudes?

TMA Confidential

TMA Solution, 2004

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Page 1 of 4

Quality
Management
System

SW-DI-005-06 Java Checklist

30.
#
31.
32.
33.
34.
35.
36.
37.
38.
39.
#
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
#
51.
52.
53.
54.
55.
56.
57.
58.
#
59.
60.
61.
#
62.
63.
64.
65.
66.
67.
68.
69.

Are divisors tested for zero or noise?

III.6 Comparison/Relational
Has each boolean expression been simplified by driving negations inward?
For every boolean test: Is the correct condition checked?
Are there any comparisons between variables of inconsistent types?
Are the comparison operators correct?
Is each boolean expression correct?
Are there improper and unnoticed side-effects of a comparison?
Has an "&" inadvertently been interchanged with a "&&" or a "|" for a "||"?
Does the code avoid comparing floating-point numbers for equality?
Is every three-way branch (less,equal,greater) covered?
III.7 Control Flow
For each loop: Is the best choice of looping constructs used?
Will all loops terminate?
When there are multiple exits from a loop, is each exit necessary and handled properly?
Does each switch statement have a default case?
Are missing switch case break statements correct and marked with a comment?
Is the nesting of loops and branches too deep, and is it correct?
Can any nested if statements be converted into a switch statement?
Are null bodied control structures correct and marked with braces or comments?
Does every method terminate?
Are all exceptions handled appropriately?
Do named break statements send control to the right place?
III.8 Input/Output
Have all files been opened before use?
Are the attributes of the open statement consistent with the use of the file?
Have all files been closed after use?
Is buffered data flushed?
Are there spelling or grammatical errors in any text printed or displayed?
Are error conditions checked?
Are files checked for existence before attempting to access them?
Are all I/O exceptions handled in a reasonable way?
III.9 Module Interface
Are the number, order, types, and values of parameters in every method call in agreement with the
called method's declaration?
Do the values in units agree (e.g., inches versus yards)?
If an object or array is passed, does it get changed, and changed correctly by the called method?
III.10 Comment
Does every method, class, and file have an appropriate header comment?
Does every attribute,variable or constant declaration have a comment?
Is the underlying behavior of each method and class expressed in plain language?
Is the header comment for each method and class consistent with the behavior of the method or
class?
Are all comments consistent with the code?
Do the comments help in understanding the code?
Are there enough comments in the code?
Are there too many comments in the code?

TMA Confidential

TMA Solution, 2004

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Page 2 of 4

Quality
Management
System

SW-DI-005-06 Java Checklist

#

III.11 Layout and Packing

Is a standard indentation and layout format used consistently?
For each method: Is it no more than about 60 lines long?
For each compile module: Is no more than about 600 lines long?
#
III.12 Modularity
73.
Is there a low level of coupling between modules (methods and classes)?
74.
Is there a high level of cohesion within each module (methods or class)?
75.
Is there repetitive code that could be replaced by a call to a method that provides the behavior of
the repetitive code?
76.
Are the Java class libraries used where and when appropriate?
#
III.13 Storage Usage
77.
Are arrays large enough?
78.
Are object and array references set to null once the object or array is no longer needed?
#
III.14 Performance
79.
Can better data structures or more efficient algorithms be used?
80.
Are logical tests arranged such that the often successful and inexpensive tests precede the more
pensive and less frequently successful tests?
81.
Can the cost of recomputing a value be reduced by computing it once and storing the results?
82.
Is every result that is computed and stored actually used?
83.
Can a computation be moved outside a loop?
84.
Are there tests within a loop that do not need to be done?
85.
Can a short loop be unrolled?
86.
Are there two loops operating on the same data that can be combined into one?
87.
Are frequently used variables declared register?
88.
Are short and commonly called methods declared inline?
89.
Are timeouts or error traps used for external device accesses?
IV - INCONSISTENCY OBJECTIVE
#
IV.1 Performance
90.
Are there any code implement in inconsistent way?
V AMBIGUITY OBJECTIVE
#
V.1 Variable and Constant Declaration
91.
Are there variables with confusingly similar names?
92.
Are all variables properly defined with meaningful, consistent, and clear names?
#
V.2 Performance
93.
Areanymodulesexcessivelycomplexandshouldberestructuredorsplitintomultipleroutines?
VI REDUNDANCE OBJECTIVE
#
VI.1 Variables
94.
Are there any redundant or unused variables or attributes?
95.
Could any non-local variables be made local?
#
VI.2 Method Definition
96.
Are there any uncalled or unneeded methods?
#
VI.3 Performance
97.
Cananycodebereplacedbycallstoexternalreusableobjects?
98.
Are there any blocks of repeated code that could be condensed into a single method?
99.
Arethereanyleftoverstubsortestroutinesinthecode?
VII SIDE-EFFECT OBJECTIVE
#
VII.1 Method Definition

Yes No NA

70.
71.
72.

TMA Confidential

TMA Solution, 2004

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA

Yes No NA
Yes No NA

Yes No NA

Page 3 of 4

Quality
Management
System

SW-DI-005-06 Java Checklist

100.
#
101.

After changing of prototype of method, Have class which calls it considered yet?
VII.2 Data Base
Do Upgrading and Migration process follow up changing of structures or contents of a projects
data base?

TMA Confidential

TMA Solution, 2004

Yes No NA

Page 4 of 4