Unit 5: Tutor Feedback - Georgia04/C0543_ConsoleApp GitHub Wiki
Task 5.1 (5.0/5.0)
| 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
Task 5.2 MP3 Chart (4.5/5.0)
| 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 |
Comments
You have made a simple mistake and your loop counter only goes from 1 to 9 and not 1 to 10
Task 5.3 (4.5/5.0)
| 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 |
Comments
Your input method does not ask for the players score by name.
Task 5.4 Bubble Sort (3.5/5.0)
| 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 |
Comments
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.
Task 5.6 Bates Motel (3.0/5.0)
| 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 |
Comment
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.