Although it can often be a very good thing, a lot of programmers hate to do code reviews. So this week we get a hard learned lesson is code reviews.
I have been tasked to review two other groups WattDepot project mentioned in our previous blogs. Watch as I tear it up.
To read my full review of the code refer to the links below:
Summary:
Eolu
Not all the commands were completed. Overall the design could use work, the methods can be split into there respective classes and the javadocs require updating. Comments need to also be added to many methods, it is inconsistent.
Ehiku
All but the chart command is completed and functioning. The design for the system is very good, all methods are separated into their own class. Commenting and naming conventions need to be more consistent across the different programmers. Overall well programmed.
No comments:
Post a Comment