-
Notifications
You must be signed in to change notification settings - Fork 6
Feedback Part3
Feedback Part3
You finished less than half half of the project. While the code looks clean.It would be better to have different packages for storing model and view classes. Also, I couldn’t run your code. Please fix this.
The application is demonstrable and clear to use.
You shouldn’t add “R” classes to your javadoc file. You can simply avoid this, see this link: http://stackoverflow.com/questions/10354716/how-to-remove-r-java-from-javadoc
Documentation is consistent with code, but there is not enough description for most methods. You need to give a clear description about the packages, classes, methods, params and return values.
You have some tests but not enough to satify. Also I couldn’t run the test part. Please fix this.
The UML diagram has been improved, but there are still some flaws in the diagram. You should also add some labels and notes for your classes such as Robosherlock list fragment, since no methods and attributes are included in the diagram; so that makes it hard to find the roles and relations.
The release planning is clear.
All feedback from part 2 has been addressed properly.
7/11