Homework 5

Code review (100 points)

Each review team reviews one project. Each review team can choose to turn in one review report, or to turn in individual reports. If a team-based report is chosen, the points will be equally divided between all members of the review team.

The review team must read through all the source code of the project. Its task is to note all unusual or erroneous parts of it.

• A list of all the classes/modules. For each class/module the following information has to be given: name, comments on its design, comments on how it is documented, comments on individual parts of its code (individual methods or individual lines).
• A summary that discusses overall issues with the code, such as coding style and documentation.
• Recommendations for the project team on how to improve their code.

The reviewers should take note of duplicated code, lacking documentation, obvious bugs, badly named variables or methods, methods or classes or modules that are too long or too short, bad use of libraries, and other issues.