Unit 5: Tutor Feedback
Reason | Deductions | Mark | Comment |
---|---|---|---|
Documentation | 0 | 5.0 | Good |
Block Comments | 0 | 5.0 | Good commenting |
Poor Names | 0 | 5.0 | Good Naming |
Poor Layout | 0 | 5.0 | Good, but one mistake |
Poor Structure | 0 | 5.0 | Good |
Code Not Working | 0 | 5.0 | Code works fully |
#Comments You have used SimpleIO for some console input, but you should have used it for all console input. You have one layout mistake
Reason | Deductions | Mark | Comment |
---|---|---|---|
Documentation | 0 | 5.0 | Good |
Block Comments | 0 | 5.0 | Good commenting |
Poor Names | 0 | 5.0 | Good Naming |
Poor Layout | 0 | 5.0 | Good, but one mistake |
Poor Structure | 0 | 5.0 | Good |
Code Not Working | 0 | 4.5 | One mistake in array index |
You have made a simple mistake and your loop counter only goes from 1 to 9 and not 1 to 10
Reason | Deductions | Mark | Comment |
---|---|---|---|
Documentation | 0 | 5.0 | Good |
Block Comments | 0 | 5.0 | Good commenting |
Poor Names | 0 | 5.0 | Good Naming |
Poor Layout | 0 | 5.0 | Good, but one mistake |
Poor Structure | 0 | 5.0 | Good |
Code Not Working | 0 | 4.5 | players name missing |
Your input method does not ask for the players score by name.
Reason | Deductions | Mark | Comment |
---|---|---|---|
Documentation | -1.0 | 4.0 | Screen shots only |
Block Comments | -0.5 | 3.5 | Missing class comment |
Poor Names | 0 | 3.5 | Good Naming |
Poor Layout | 0 | 3.5 | Good |
Poor Structure | 0 | 3.5 | Good |
Code Not Working | 0 | 3.5 | Good but not tested well |
You have done all the hard work, but have not documented well or tested it well. You entered the numbers in numeric order so there is no way to know whether the sort worked or not. You should have entered the numbers in reverse order to test that they have been sorted.
Reason | Deductions | Mark | Comment |
---|---|---|---|
Documentation | -1.0 | 4.0 | Poor Documentation |
Block Comments | -0.5 | 3.5 | poor class comment |
Poor Names | 0 | 3.5 | Good Naming |
Poor Layout | 0 | 3.5 | Good |
Poor Structure | -0.5 | 3.5 | Good |
Code Not Working | -0.5 | 2.5 | mistake in testing not identified |
Your use case diagram is very poor, much too complicated. Your class diagram shows the use of SimpleIO class but you made no use of it. You could have simplified your code by using SimpleIO. You did not spot that there is a mistake in the booking of a room. The Book Room method is too large and should be split into two.
Georgia Gkegka 21800490