Friday, August 31, 2007

07.CodeRulerReview

Reviewer: Jeffrey Lam
Author of code: Ka Yee Leung


I'm going to reveiw Ka Yee Leung's source code. These are few code violation appare in your source code. The violation problem is around formating convention and naming convention. The following table shows the detail of the violation located.

FileLineViolationComments
MyRuler.java81,83,85,*EJS- 9Use meaningful name
MyRuler.java22EJS- 35Use documentation comments to describe the programming interface, not standardcomments.
MyRuler.java136EJS- 5Newline after end brackets

No comments: