-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathfeedback.html
166 lines (119 loc) · 4.74 KB
/
feedback.html
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
<html>
<head>
<title>COMP90041 Programming and Software Development</title>
</head>
<body>
<h3>COMP90041 ProjC Feedback: aprasetiya</h3>
<h4>Program Presentation</h4>
Including: layout and style, readability, adherence to coding
expectations, general care and appearance.
<br>
<p>
This section is worth <b>1.5 marks</b>.
<b>Marks in each section can NOT be negative</b>
<p>
<i>Deductions</i>
<p>
Some subset of the following lines will be selected by the marker.
Deduct 0.5 marks for <b>any error listed below</b>.
<p><ul>
<!-- <li>constants not in upper case;<b>(point out where the mistake happens if there is deduction)</b>
<li>bad choices for method names;
<li>bad choices for variable names;
<li>excessive commenting;
<li>inconsistent bracket placement;
<li>inconsistent indentation;
<li>insufficient commenting; -->
<!-- <li>lack of whitespace (visual appeal); -->
<li>lines >100 chars: line 200, 252, 255 in Nimsys.java (-0.5)
<!-- <li>no authorship statement;
<li>use of magic numbers;
<li>other stylistic issue, deduct 0.5 mark for a single occurence; -->
</ul><p>
<i>Additions (only makes up for marks lost in this section)</i>
<p><ul>
<li>overall care and presentation, +0.5;
</ul><p>
<i>Other Comments from Marker</i><p>
Good!
<h4>Structure and Approach</h4>
Including: decomposition in to methods, declaration of instance variables
at the appropriate locations, choice of parameters to methods.
<p>
<p>
This section is worth <b>3.5 marks</b>.
<p>
<i>Deductions</i>
<br>
Some subset of the following lines will be selected by the marker. Deduct 0.5 marks <b>any error listed below</b> or deduct marks specified for particular issues.
<p><ul>
<li>duplicate code segments: The game-play process code for two players are duplicated, consider replacing them with methods. (-0.5)
<!-- <li>main method too long or too complex;
<li>other methods too long or too complex;
<li>insufficient use of methods;
<li>overly complex algorithmic approach;
<li>unnecessary duplication/copying of data;
<li>overly complex method argument lists;
<li>avoidance of non-static instance variables/methods, -1.0 mark; <b>(Please give detailed suggestions for improvement if deducting under these -1.0/-2.0 items)</b>
<li>avoidance of private instance variables/methods, -1.0 mark;
<li>not using try/catch to handle Exceptions, -1.0 mark;
<li>AI player has been implemented without making use of inheritance/polymorphism
(use if-else to test player type in game-play), -2.0 marks;
<li>other structural issue, if major then deduct 1 mark; -->
<li>other structural issue, if major then deduct 1 mark: Should use more specific/detailed exceptions. Either Java built-in exception (e.g. "ArrayOutOfBoundException") or customized exception (e.g. "InvalidCommandException") are fine.
</ul><p>
<i>Other comments from marker</i>:<p>
Good!
<h4>Program execution</h4>
Including: compilation, execution on test data, output presentation
and readability.<br>
Programs that do not compile in the test environment will lose all
marks in this section.
Be sure to <tt>verify</tt> your submission and <b>check the output</b>
before you say "finished" to yourself.
<p>
The full marks for this section of marking are as follows, depending on
up to which stage you have completed.
<p>
This section is worth <b>5 marks</b>.
<p>
<p><ul>
<li>Exception Handling, +2.
<li>Player Statistics Recording, +1.
<li>AI Player, +2.
</ul><p>
<i>Deductions</i>
<p>Some subset of the following lines will be selected by the marker.
<p><ul>
<!--
<li>somewhat incorrect Exception Handling output on test1, -1;
<li>grossly incorrect Exception Handling output on test1, -2;
<li>somewhat incorrect Player Statistics Recording output on test2, -0.5;
<li>grossly incorrect Player Statistics Recording output on test2, -1;
<li>somewhat incorrect AI Player output on test3, -1;
<li>grossly incorrect AI Player output on test3, -2; -->
</ul><p>
<!-- <p>
<b><i>Bonus</i></b>
<p>
Bonus is worth <b>1.5 marks</b>.
<p>
<p>In each test, there are three sub-tests. For each sub-test, if winning ratio under BONUS TASK <b>or</b> BONUS TASK VISIBLE TEST is 100%, we will award the 0.5 mark. </p>
<p><ul>
<li>Wining ratio is 100% if moves first, against dummy rival player, +0.5
<li>Wining ratio is 100% if moves first, against oracle rival player, +0.5
<li>Wining ratio is 100% if moves second, against dummy rival player, +0.5
</ul><p>
-->
<p><b>The total mark for Project C is up to 11.5</b></p>
Total marks:
<h3 style="color:blue">8.5</h3>
<b style="color:black"><i>Overall comments from marker:</i></b><p><div style="background-color:Beige">
Some minor mismatch in game-play output, but no mark deducted.
</div>
<p>
<i>Assignment Marker: Dongge Liu</i>
<br>
<i>If you have any questions regarding your mark, please contact the lecturers</i>
</body>
</html>