[ ] In the Method chapter of your report, explain how you worked and how you reasoned when writing the program.
[ ] In the Method chapter of your report, explain how you worked and how you reasoned when writing the unit tests. Explain how you chose which tests to write.
[ ] In the Result chapter of your report, briefly explain the program. Include links to your git repository, and make sure the repository is public. Also include a printout of a sample run.
[ ] In the Result chapter of your report, briefly explain the tests. Include links to your git repository, and make sure the repository is public.
[ ] In the discussion:
[ ] Is the code easily understood? Do all declarations have explaining names? Are code conventions followed?
[ ] Is there duplicated code?
[ ] Are there methods or classes that are too long?
[ ] Are primitive data or static members used instead of objects?
[ ] Are there methods with unnecessarily long parameter lists?
[ ] Is there one comment per public declaration and no comments inside methods?
[ ] Are the MVC and Layer patterns still used correctly?
[ ] Is there still low coupling, high cohesion and good encapsulation?
[ ] Is a framework, like JUnit, used for the tests?
[ ] Are all tests self-evaluating, that is, do they print an informative message if a test fails, and do they print nothing if a test passes?
[ ] Are the tests placed in the same package as the SUT, but in a different directory?
[ ] Is there one test class per tested class?
[ ] Are all branches of if-statements tested?
[ ] Do tests pass all interesting parameter values to the tested methods?
[ ] If the assessed person wrote unit tests for all classes, are the tests complete?
[ ] Are there tests for all public and package private methods (except simple setters/getters)? Are also methods writing to System.out tested?
[ ] If the assessed person wrote unit tests for only two classes, are the tests meaningful?
[ ] Is the use of programming best practices explained in the report?
[ ] Is the method and result explained in the report? Is there a discussion? Is the discussion relevant?
Basic: