Code Review Checklist

Code Review Checklist

Review Date Reviewer Project Code Project Name PM Name Document Name Author Work Product Name & ID Work Product Authors Name

Sl.No

Description Yes

Tick No NA

Remarks

1. 2.

Were all materials necessary for review on hand? Did programmer(s) provide a clear explanation of design? Did they give a step-by-step simulation of module processing? Are there any walkthrough? exceptions noted during the

3.

4.

5. 6. 7.

Did the program compile? Does code appear logically sound? Did the program generate any output files?

Page 1 of 4

Code Review Checklist

Sl.No

Description Yes

Tick No NA

Remarks

8. 9. 10. 11.

Do the output file(s) appear correct? If no, what type of anomalies occurred? Does the program handle errors? Did the programmer(s) create test case(s) with errors? Did the program pass through all test case(s) correctly? If no, what errors did it throw up? Does each file include client's copyright header? Does each file have completed documentation header? Do headers explain in plain English what module does? Does code follow algorithmic description in header? Is the algorithmic flow described in detail? Does the use include files with standard data formats? Does the name adhere to project conventions? Were meaningful variable names used throughout program? Is revision history included? Does use of indentation follow project guidelines? Were variables commented on when first used? Are the uses in-line comment blocks?

12.

13. 14. 15.

16.

17. 18. 19.

20. 21.

22. 23. 24. 25.

Page 2 of 4

Code Review Checklist

Sl.No

Description Yes

Tick No NA

Remarks

26. 27. 28. 29. 30. 31. 32. 33. 34. 35. 36. 37. 38. 39. 40.

Have white space been used liberally? Is code easy to read? Is Writing coherent and understandable? Are Spelling and grammar correct? Is the source of inputs specified? Are all fields of input specified in detail? Does it address any anomalies found in raw data? Has the use of global variables indicated? Have the destination of outputs been specified? Are all fields of output specified in detail? Are preconditions clearly stated? Are post conditions clearly stated? Are there any key issues to be resolved? Were any related issues discovered? Is there any other efficient way to write the same piece of code? If yes, please specify. Are there any code breakages after checked in to source safe? Did the program pass the code review test run through code review tool? Does the Drop Down in the Web page Screen display Select option By default?

41. 42.

43.

44.

Page 3 of 4

Code Review Checklist

Sl.No

Description Yes

Tick No NA

Remarks

45

Does the Screens having ZIP functionality have the option of auto populating City, State and County, and for validation of this any icon is present next to Zip field?

Page 4 of 4