Friday, August 31, 2007

Code Ruler Review





































FileLinesViolationsComments
MyRuler.java22, 23, 25, *EJS-5Code indents 4 space. It should be 2
MyRuler.java22, 23, 90, *EJS-7
MyRuler.java36, 37, 38, *EJS-8
MyRuler.java66EJS-35Need to clearly describe what this method does in terms of strategy.
MyRuler.java232EJS-54Need to describe the x and y parameter.
Posted by at

No comments:

Post a Comment

Subscribe to: Post Comments (Atom)