|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| The constructor invokes a non-final method |
Rewrite the constructor so it does not invoke any non-final methods on the object being constructed. |
Medium |
ColorPickerAndButton.java |
11 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Dangling else clause |
Enclose the then clause in a block. |
Medium |
PenUpDownAction.java |
27 |
| Dangling else clause |
Enclose the then clause in a block. |
Medium |
Model.java |
115 |
| Dangling else clause |
Enclose the then clause in a block. |
Medium |
Model.java |
117 |
| Dangling else clause |
Enclose the then clause in a block. |
Medium |
PropertiesFactory.java |
31 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Empty finally clause |
Delete the empty finally clause. |
Medium |
SLogoNodeFactory.java |
30 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DisplayTurtleAction.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenUpDownAction.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenUpDownAction.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SetTowardsAction.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
5 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
7 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
8 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
9 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
10 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
11 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
12 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
13 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
14 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
15 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
16 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
17 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
20 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
21 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ModelConstants.java |
22 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
79 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
80 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
81 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
82 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
83 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
84 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Controller.java |
85 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AddTurtleToWorkspaceButton.java |
9 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AddTurtleToWorkspaceButton.java |
25 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AddTurtleToWorkspaceButton.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ChooseTurtleImageButton.java |
12 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ChooseTurtleImageButton.java |
25 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ChooseTurtleImageButton.java |
34 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DashPenButton.java |
11 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DashPenButton.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DotPenButton.java |
11 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DotPenButton.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HelpPageButton.java |
14 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HelpPageButton.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
24 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
25 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
HistoryFeature.java |
35 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
65 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
66 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
67 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
68 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
69 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
70 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
71 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
KeyControlFeature.java |
72 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
NewWorkspaceButton.java |
34 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenColorPickerAndButton.java |
19 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenColorPickerAndButton.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenSizeSlider.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenSizeSlider.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PenSizeSlider.java |
40 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ShowAndHideTurtleButton.java |
10 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ShowAndHideTurtleButton.java |
18 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ShowAndHideTurtleButton.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ShowAndHideTurtleButton.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ShowAndHideTurtleButton.java |
31 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ShowAndHideTurtleButton.java |
34 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SolidPenButton.java |
10 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SolidPenButton.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
77 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
83 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
96 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
103 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
114 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
116 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
118 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
141 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Model.java |
155 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
31 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
33 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
41 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
48 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
56 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
57 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
59 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
61 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
62 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
64 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
66 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
68 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
69 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
72 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
75 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PropertiesFactory.java |
77 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SLogoCanvas.java |
82 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SLogoCanvas.java |
137 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SLogoCanvas.java |
138 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
BooleanOperations.java |
31 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
BooleanOperations.java |
32 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
BooleanOperations.java |
33 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
BooleanOperations.java |
34 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ControlStructuresNode.java |
18 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ControlStructuresNode.java |
19 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ControlStructuresNode.java |
20 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ControlStructuresNode.java |
21 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DifferenceNode.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
DifferenceNode.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ProductNode.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ProductNode.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
QuotientNode.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
QuotientNode.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SumNode.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
SumNode.java |
29 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TwoChildMathOperationsNode.java |
21 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TwoChildMathOperationsNode.java |
22 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
GroupStartNode.java |
18 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
ListStartNode.java |
18 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PunctuationNode.java |
23 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
PunctuationNode.java |
30 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleCommandsNode.java |
23 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
MakeNode.java |
25 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
86 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
88 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
89 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
92 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
94 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
95 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
99 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
100 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Turtle.java |
206 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleCollection.java |
86 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
View.java |
77 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
View.java |
80 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
View.java |
91 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
View.java |
92 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
View.java |
118 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Workspace.java |
30 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
Workspace.java |
35 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
8 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
9 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
10 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
11 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
12 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
13 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
14 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
15 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
16 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
17 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
18 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
19 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
20 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
21 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
22 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
23 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
24 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
25 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
AbstractDrawerTest.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
22 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
23 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
24 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
25 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
26 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
27 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
28 |
| Code within a block is not indented |
Reformat the code so that the code within the block is properly indented one level more than the code containing the block. |
Low |
TurtleTester.java |
29 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid field name: "AVAILABLE_OPERATIONS" starts with uppercase letters |
Rename the field from "AVAILABLE_OPERATIONS" to "aVAILABLEOPERATIONS". |
Medium |
BooleanOperations.java |
18 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid visibility for field: public |
Correct the visibility of the field. |
Medium |
AbstractAction.java |
15 |
| Invalid visibility for field: public |
Correct the visibility of the field. |
Medium |
AbstractAction.java |
16 |
| Invalid visibility for field: public |
Correct the visibility of the field. |
Medium |
AbstractAction.java |
17 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
SetPaletteAction.java |
15 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SetPositionAction.java |
13 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SetTowardsAction.java |
13 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
Controller.java |
63 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SLogoException.java |
4 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
ColorPickerAndButton.java |
8 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
ColorPickerAndButton.java |
9 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
ColorPickerAndButton.java |
10 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
GenericGUIFeature.java |
17 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
PenColorPickerAndButton.java |
10 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
PenSizeSlider.java |
14 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
PenSizeSlider.java |
15 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
PenSizeSlider.java |
16 |
| Invalid visibility for field: package |
Correct the visibility of the field. |
Medium |
ToggleReferenceGridButton.java |
8 |
| Invalid visibility for field: public |
Correct the visibility of the field. |
Medium |
Model.java |
25 |
| Invalid visibility for field: public |
Correct the visibility of the field. |
Medium |
Model.java |
26 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
BooleanOperations.java |
18 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
BooleanOperations.java |
19 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
ControlStructuresNode.java |
7 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SLogoNode.java |
20 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SLogoNode.java |
21 |
| Invalid visibility for field: public |
Correct the visibility of the field. |
Medium |
SLogoNode.java |
22 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SLogoNode.java |
23 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
SLogoNode.java |
24 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
TurtleQueriesNode.java |
7 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
Turtle.java |
18 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
Turtle.java |
19 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
Turtle.java |
20 |
| Invalid visibility for field: protected |
Correct the visibility of the field. |
Medium |
Turtle.java |
21 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
AddTurtleToWorkspaceButton.java |
13 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
BackgroundColorPickerAndButton.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ChooseTurtleImageButton.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ColorPickerAndButton.java |
12 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
DashPenButton.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
DotPenButton.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
HelpPageButton.java |
19 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
NewWorkspaceButton.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PenColorPickerAndButton.java |
13 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ShowAndHideTurtleButton.java |
14 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SolidPenButton.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SubmitTextButtonAndField.java |
19 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ToggleReferenceGridButton.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
DoTimesNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ForNode.java |
12 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
IfElseNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
IfNode.java |
12 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
RepeatNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ClearStampsNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
DisplayCommandsNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PenColorNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetBackgroundNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetIndexNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetPaletteNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetPenColorNode.java |
17 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetShapeNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ShapeNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
StampNode.java |
14 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
InteractionCommandsNode.java |
14 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
OnClickNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
OnKeyNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
OnMoveNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
AtanNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
CosNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
DifferenceNode.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
LogNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
MathOperationsNode.java |
13 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
MinusNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
OneChildMathOperationsNode.java |
13 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PowNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ProductNode.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
QuotientNode.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
RandomNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
RemainderNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SinNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SumNode.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
TanNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
TwoChildMathOperationsNode.java |
14 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ConstantNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
NumberNode.java |
13 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
GroupEndNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
GroupNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
GroupStartNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ListEndNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ListNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ListStartNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PunctuationNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
BackwardNode.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ClearScreenNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
DisplayTurtleNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ForwardNode.java |
17 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
HideTurtleNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
HomeNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
LeftNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
MoveNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PenDownNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PenNode.java |
17 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
PenUpNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
RightNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
RotateNode.java |
11 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetHeadingNode.java |
18 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetPositionNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
SetTowardsNode.java |
16 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
ShowTurtleNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
TurtleCommandsNode.java |
15 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
XCorNode.java |
9 |
| Default constructor invocation found |
Call to super() is made automatically by compiler, it should thus be removed. |
High |
MakeNode.java |
15 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Too many fields: 58 |
Reduce the number of fields by splitting the class into multiple smaller classes. |
Medium |
GUIconstants.java |
3 |
| Too many fields: 19 |
Reduce the number of fields by splitting the class into multiple smaller classes. |
Medium |
Controller.java |
23 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Line 20 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
AbstractAction.java |
20 |
| Line 10 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ActionFactory.java |
10 |
| Line 18 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
18 |
| Line 19 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
19 |
| Line 20 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
20 |
| Line 21 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
21 |
| Line 26 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
26 |
| Line 34 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
34 |
| Line 37 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
37 |
| Line 61 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
61 |
| Line 62 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GUIconstants.java |
62 |
| Line 5 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ModelConstants.java |
5 |
| Line 36 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
36 |
| Line 38 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
38 |
| Line 40 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
40 |
| Line 46 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
46 |
| Line 48 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
48 |
| Line 58 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
58 |
| Line 62 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Controller.java |
62 |
| Line 7 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
InvalidSLogoSyntax.java |
7 |
| Line 19 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
BackgroundColorPickerAndButton.java |
19 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
BackgroundColorPickerAndButton.java |
23 |
| Line 19 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
HistoryFeature.java |
19 |
| Line 22 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
22 |
| Line 27 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
27 |
| Line 29 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
29 |
| Line 40 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
40 |
| Line 41 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
41 |
| Line 46 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
46 |
| Line 47 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
47 |
| Line 52 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
52 |
| Line 53 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
53 |
| Line 58 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
58 |
| Line 59 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
59 |
| Line 76 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
KeyControlFeature.java |
76 |
| Line 13 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
NewWorkspaceButton.java |
13 |
| Line 20 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
NewWorkspaceButton.java |
20 |
| Line 33 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
NewWorkspaceButton.java |
33 |
| Line 13 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SubmitTextButtonAndField.java |
13 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SubmitTextButtonAndField.java |
23 |
| Line 32 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SubmitTextButtonAndField.java |
32 |
| Line 7 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ToggleReferenceGridButton.java |
7 |
| Line 13 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ToggleReferenceGridButton.java |
13 |
| Line 42 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
42 |
| Line 79 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
79 |
| Line 81 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
81 |
| Line 88 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
88 |
| Line 91 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
91 |
| Line 96 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
96 |
| Line 97 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
97 |
| Line 101 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
101 |
| Line 104 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
104 |
| Line 106 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
106 |
| Line 108 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
Model.java |
108 |
| Line 30 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
PropertiesFactory.java |
30 |
| Line 18 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
AndNode.java |
18 |
| Line 18 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
BooleanOperations.java |
18 |
| Line 15 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
EqualToNode.java |
15 |
| Line 16 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GreaterThanNode.java |
16 |
| Line 16 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
LessThanNode.java |
16 |
| Line 16 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
NotEqualToNode.java |
16 |
| Line 21 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ForNode.java |
21 |
| Line 22 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ForNode.java |
22 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ForNode.java |
23 |
| Line 22 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
PowNode.java |
22 |
| Line 22 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
RandomNode.java |
22 |
| Line 22 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
RemainderNode.java |
22 |
| Line 22 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
TwoChildMathOperationsNode.java |
22 |
| Line 18 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
GroupStartNode.java |
18 |
| Line 18 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
ListStartNode.java |
18 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
PunctuationNode.java |
23 |
| Line 16 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
16 |
| Line 17 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
17 |
| Line 45 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
45 |
| Line 46 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
46 |
| Line 51 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
51 |
| Line 53 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
53 |
| Line 54 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNode.java |
54 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SetPositionNode.java |
23 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SetTowardsNode.java |
23 |
| Line 23 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
MakeNode.java |
23 |
| Line 35 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNodeFactory.java |
35 |
| Line 43 is longer than 80 characters |
Break the line into multiple shorter lines. |
Low |
SLogoNodeFactory.java |
43 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid local variable name: "LanguageSyntax" starts with uppercase letters |
Rename the variable. |
Medium |
PropertiesFactory.java |
71 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Missing block in if clause: canvas.display().getChildren().remove(turtle.display()); |
Add a block around the body of the if clause. |
Medium |
DisplayTurtleAction.java |
27 |
| Missing block in if clause: turtle.getPen().setPenUp(); |
Add a block around the body of the if clause. |
Medium |
PenUpDownAction.java |
26 |
| Missing block in else clause: turtle.getPen().setPenDown(); |
Add a block around the body of the else clause. |
Medium |
PenUpDownAction.java |
28 |
| Missing block in if clause: return listOfActions; |
Add a block around the body of the if clause. |
Medium |
Model.java |
77 |
| Missing block in if clause: makeListOfSLogoNodeTreesHelper(root,currentNode,nodeFactory,remainingInput); |
Add a block around the body of the if clause. |
Medium |
Model.java |
96 |
| Missing block in if clause: return; |
Add a block around the body of the if clause. |
Medium |
Model.java |
103 |
| Missing block in if clause: nextCommand = myCommands.get(nextCommand); |
Add a block around the body of the if clause. |
Medium |
Model.java |
114 |
| Missing block in if clause: nextCommand = myVariables.get(nextCommand); |
Add a block around the body of the if clause. |
Medium |
Model.java |
116 |
| Missing block in else clause: return nextCommand; |
Add a block around the body of the else clause. |
Medium |
Model.java |
118 |
| Missing block in if clause: myScanner.close(); |
Add a block around the body of the if clause. |
Medium |
Model.java |
141 |
| Missing block in if clause: keywords.addAll(new ArrayList<String>(Arrays.asList(keywordInSpecifiedLanguage.split(",")))); |
Add a block around the body of the if clause. |
Medium |
PropertiesFactory.java |
30 |
| Missing block in else clause: keywords.add(keywordInSpecifiedLanguage); |
Add a block around the body of the else clause. |
Medium |
PropertiesFactory.java |
32 |
| Missing block in if clause: childrenOfInterest = myChildren.get(0).getChildren(); |
Add a block around the body of the if clause. |
Medium |
DifferenceNode.java |
26 |
| Missing block in if clause: break; |
Add a block around the body of the if clause. |
Medium |
DifferenceNode.java |
29 |
| Missing block in if clause: childrenOfInterest = myChildren.get(0).getChildren(); |
Add a block around the body of the if clause. |
Medium |
ProductNode.java |
26 |
| Missing block in if clause: break; |
Add a block around the body of the if clause. |
Medium |
ProductNode.java |
29 |
| Missing block in if clause: childrenOfInterest = myChildren.get(0).getChildren(); |
Add a block around the body of the if clause. |
Medium |
QuotientNode.java |
26 |
| Missing block in if clause: break; |
Add a block around the body of the if clause. |
Medium |
QuotientNode.java |
29 |
| Missing block in if clause: childrenOfInterest = myChildren.get(0).getChildren(); |
Add a block around the body of the if clause. |
Medium |
SumNode.java |
26 |
| Missing block in if clause: break; |
Add a block around the body of the if clause. |
Medium |
SumNode.java |
29 |
| Missing block in if clause: if (myChildren.get(0) instanceof GroupStartNode)
return myChildren.get(myChildren.size()-1) instanceof GroupEndNode; |
Add a block around the body of the if clause. |
Medium |
TwoChildMathOperationsNode.java |
21 |
| Missing block in if clause: return myChildren.get(myChildren.size()-1) instanceof GroupEndNode; |
Add a block around the body of the if clause. |
Medium |
TwoChildMathOperationsNode.java |
22 |
| Missing block in if clause: return !(myChildren.get(myChildren.size()-1) instanceof GroupEndNode); |
Add a block around the body of the if clause. |
Medium |
GroupStartNode.java |
18 |
| Missing block in if clause: return !(myChildren.get(myChildren.size()-1) instanceof ListEndNode); |
Add a block around the body of the if clause. |
Medium |
ListStartNode.java |
18 |
| Missing block in if clause: return !(myChildren.get(myChildren.size()-1) instanceof PunctuationNode); |
Add a block around the body of the if clause. |
Medium |
PunctuationNode.java |
23 |
| Missing block in for statement: myActions.addAll(myChildren.get(0).evaluate()); |
Add a block around the body of the for statement. |
Medium |
PunctuationNode.java |
30 |
| Missing block in for statement: myActions.addAll(myChildren.get(0).evaluate()); |
Add a block around the body of the for statement. |
Medium |
TurtleCommandsNode.java |
23 |
| Missing block in for statement: childrenActionList.addAll(myChildren.get(i).evaluate()); |
Add a block around the body of the for statement. |
Medium |
MakeNode.java |
25 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid numeric literal: 600 |
Consider defining a constant name to represent the value. |
High |
SetTowardsAction.java |
24 |
| Invalid numeric literal: 300 |
Consider defining a constant name to represent the value. |
High |
SetTowardsAction.java |
25 |
| Invalid numeric literal: 300 |
Consider defining a constant name to represent the value. |
High |
SetTowardsAction.java |
26 |
| Invalid numeric literal: 20 |
Consider defining a constant name to represent the value. |
High |
ChooseTurtleImageButton.java |
21 |
| Invalid numeric literal: 500 |
Consider defining a constant name to represent the value. |
High |
ChooseTurtleImageButton.java |
22 |
| Invalid numeric literal: 750 |
Consider defining a constant name to represent the value. |
High |
HelpPageButton.java |
32 |
| Invalid numeric literal: 500 |
Consider defining a constant name to represent the value. |
High |
HelpPageButton.java |
32 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
Model.java |
158 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
Model.java |
158 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
ForNode.java |
14 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
ForNode.java |
23 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
IfElseNode.java |
20 |
| Invalid numeric literal: 3 |
Consider defining a constant name to represent the value. |
High |
IfElseNode.java |
32 |
| Invalid numeric literal: 4 |
Consider defining a constant name to represent the value. |
High |
SetPaletteNode.java |
17 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
SetPaletteNode.java |
25 |
| Invalid numeric literal: 3 |
Consider defining a constant name to represent the value. |
High |
SetPaletteNode.java |
26 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
InteractionCommandsNode.java |
15 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
RemainderNode.java |
16 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
TwoChildMathOperationsNode.java |
15 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
SetPositionNode.java |
17 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
SetTowardsNode.java |
17 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
MakeNode.java |
16 |
| Invalid numeric literal: 2d |
Consider defining a constant name to represent the value. |
High |
SLogoPen.java |
49 |
| Invalid numeric literal: 21d |
Consider defining a constant name to represent the value. |
High |
SLogoPen.java |
49 |
| Invalid numeric literal: 25d |
Consider defining a constant name to represent the value. |
High |
SLogoPen.java |
51 |
| Invalid numeric literal: 10d |
Consider defining a constant name to represent the value. |
High |
SLogoPen.java |
51 |
| Invalid numeric literal: 30 |
Consider defining a constant name to represent the value. |
High |
Turtle.java |
90 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
Turtle.java |
96 |
| Invalid numeric literal: 100 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
17 |
| Invalid numeric literal: 100 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
17 |
| Invalid numeric literal: 90 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
19 |
| Invalid numeric literal: 270.0 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
21 |
| Invalid numeric literal: 0.001 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
21 |
| Invalid numeric literal: 180 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
22 |
| Invalid numeric literal: 0.001 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
22 |
| Invalid numeric literal: 100 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
27 |
| Invalid numeric literal: 100 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
27 |
| Invalid numeric literal: 90 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
29 |
| Invalid numeric literal: 90.0 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
31 |
| Invalid numeric literal: 0.001 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
31 |
| Invalid numeric literal: 0.001 |
Consider defining a constant name to represent the value. |
High |
ActionTest.java |
32 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
28 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
29 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
39 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
40 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
50 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
51 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
61 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
BooleanOperationTest.java |
62 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
16 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
18 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
24 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
26 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
32 |
| Invalid numeric literal: 5 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
33 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
34 |
| Invalid numeric literal: 3.0 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
38 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
44 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
46 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
53 |
| Invalid numeric literal: 10.0 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
55 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
61 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
62 |
| Invalid numeric literal: 100.0 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
65 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
71 |
| Invalid numeric literal: 2 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
72 |
| Invalid numeric literal: 7 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
73 |
| Invalid numeric literal: 140.0 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
77 |
| Invalid numeric literal: 10 |
Consider defining a constant name to represent the value. |
High |
MathOperationsTest.java |
83 |
| Invalid numeric literal: 100 |
Consider defining a constant name to represent the value. |
High |
TurtleTester.java |
15 |
| Invalid numeric literal: 100 |
Consider defining a constant name to represent the value. |
High |
TurtleTester.java |
16 |
| Invalid numeric literal: 250 |
Consider defining a constant name to represent the value. |
High |
TurtleTester.java |
17 |
| Invalid numeric literal: 150 |
Consider defining a constant name to represent the value. |
High |
TurtleTester.java |
18 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid package name: "Actions" starts with uppercase letters |
Rename the package. |
Medium |
AbstractAction.java |
7 |
| Invalid package name: "Constants" starts with uppercase letters |
Rename the package. |
Medium |
ExceptionConstants.java |
1 |
| Invalid package name: "Errors" starts with uppercase letters |
Rename the package. |
Medium |
DivideByZero.java |
1 |
| Invalid package name: "GUIfeatures" starts with uppercase letters |
Rename the package. |
Medium |
AddTurtleToWorkspaceButton.java |
1 |
| Invalid package name: "BooleanOperations" starts with uppercase letters |
Rename the package. |
Medium |
AndNode.java |
1 |
| Invalid package name: "ControlStructures" starts with uppercase letters |
Rename the package. |
Medium |
ControlStructuresNode.java |
1 |
| Invalid package name: "DisplayCommands" starts with uppercase letters |
Rename the package. |
Medium |
ClearStampsNode.java |
7 |
| Invalid package name: "InteractionCommands" starts with uppercase letters |
Rename the package. |
Medium |
InteractionCommandsNode.java |
7 |
| Invalid package name: "MathOperations" starts with uppercase letters |
Rename the package. |
Medium |
AtanNode.java |
6 |
| Invalid package name: "MultipleTurtleCommands" starts with uppercase letters |
Rename the package. |
Medium |
package-info.java |
8 |
| Invalid package name: "Number" starts with uppercase letters |
Rename the package. |
Medium |
ConstantNode.java |
6 |
| Invalid package name: "PunctuationNode" starts with uppercase letters |
Rename the package. |
Medium |
GroupEndNode.java |
6 |
| Invalid package name: "TurtleCommands" starts with uppercase letters |
Rename the package. |
Medium |
BackwardNode.java |
6 |
| Invalid package name: "TurtleQueries" starts with uppercase letters |
Rename the package. |
Medium |
TurtleQueriesNode.java |
1 |
| Invalid package name: "Variables" starts with uppercase letters |
Rename the package. |
Medium |
MakeNode.java |
6 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
AbstractAction.java |
7 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
ExceptionConstants.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
Controller.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
DivideByZero.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
AddTurtleToWorkspaceButton.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
AndNode.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
ControlStructuresNode.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
ClearStampsNode.java |
7 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
InteractionCommandsNode.java |
7 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
AtanNode.java |
6 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
package-info.java |
8 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
ConstantNode.java |
6 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
GroupEndNode.java |
6 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
SLogoNode.java |
7 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
BackwardNode.java |
6 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
TurtleQueriesNode.java |
1 |
| Invalid package prefix: "application" |
Rename the package. |
Medium |
MakeNode.java |
6 |
| Invalid package prefix: "test" |
Rename the package. |
Medium |
AbstractDrawerTest.java |
1 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Short name found: r |
Rename. |
High |
SetPaletteNode.java |
24 |
| Short name found: g |
Rename. |
High |
SetPaletteNode.java |
25 |
| Short name found: b |
Rename. |
High |
SetPaletteNode.java |
26 |
| Short name found: c |
Rename. |
High |
ActionTest.java |
17 |
| Short name found: c |
Rename. |
High |
ActionTest.java |
27 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Missing space before operator "/" |
Insert the missing space. |
Low |
Model.java |
160 |
| Missing space after operator "/" |
Insert the missing space. |
Low |
Model.java |
160 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
AndNode.java |
17 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
AndNode.java |
17 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
IfElseNode.java |
32 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
IfElseNode.java |
32 |
| Missing space before operator "==" |
Insert the missing space. |
Low |
IfNode.java |
22 |
| Missing space after operator "==" |
Insert the missing space. |
Low |
IfNode.java |
22 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
RepeatNode.java |
20 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
RepeatNode.java |
20 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
DifferenceNode.java |
27 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
DifferenceNode.java |
27 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
ProductNode.java |
27 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
ProductNode.java |
27 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
QuotientNode.java |
27 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
QuotientNode.java |
27 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
SumNode.java |
27 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
SumNode.java |
27 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
TwoChildMathOperationsNode.java |
22 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
TwoChildMathOperationsNode.java |
22 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
GroupNode.java |
17 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
GroupNode.java |
17 |
| Missing space before operator ">" |
Insert the missing space. |
Low |
GroupStartNode.java |
17 |
| Missing space after operator ">" |
Insert the missing space. |
Low |
GroupStartNode.java |
17 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
GroupStartNode.java |
18 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
GroupStartNode.java |
18 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
ListNode.java |
17 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
ListNode.java |
17 |
| Missing space before operator ">" |
Insert the missing space. |
Low |
ListStartNode.java |
17 |
| Missing space after operator ">" |
Insert the missing space. |
Low |
ListStartNode.java |
17 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
ListStartNode.java |
18 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
ListStartNode.java |
18 |
| Missing space before operator ">" |
Insert the missing space. |
Low |
PunctuationNode.java |
22 |
| Missing space after operator ">" |
Insert the missing space. |
Low |
PunctuationNode.java |
22 |
| Missing space before operator "-" |
Insert the missing space. |
Low |
PunctuationNode.java |
23 |
| Missing space after operator "-" |
Insert the missing space. |
Low |
PunctuationNode.java |
23 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
PunctuationNode.java |
29 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
PunctuationNode.java |
29 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
TurtleCommandsNode.java |
22 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
TurtleCommandsNode.java |
22 |
| Missing space before operator "<" |
Insert the missing space. |
Low |
MakeNode.java |
24 |
| Missing space after operator "<" |
Insert the missing space. |
Low |
MakeNode.java |
24 |
| Missing space before operator "+" |
Insert the missing space. |
Low |
SLogoNodeFactory.java |
35 |
| Missing space after operator "+" |
Insert the missing space. |
Low |
SLogoNodeFactory.java |
35 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Invalid string literal: "Action: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
AbstractAction.java |
30 |
| Invalid string literal: "Action" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Action.java |
16 |
| Invalid string literal: "Backward" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BackwardAction.java |
11 |
| Invalid string literal: "ClearScreen" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ClearScreenAction.java |
15 |
| Invalid string literal: "Constant" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ConstantAction.java |
15 |
| Invalid string literal: "DisplayTurtle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
DisplayTurtleAction.java |
21 |
| Invalid string literal: "Forward" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ForwardAction.java |
11 |
| Invalid string literal: "Home" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
HomeAction.java |
15 |
| Invalid string literal: "Left" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
LeftAction.java |
16 |
| Invalid string literal: "PenUpDown" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PenUpDownAction.java |
20 |
| Invalid string literal: "Right" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RightAction.java |
10 |
| Invalid string literal: "Rotate" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RotateAction.java |
16 |
| Invalid string literal: "SetBackground" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetBackgroundAction.java |
11 |
| Invalid string literal: "SetHeading" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetHeadingAction.java |
16 |
| Invalid string literal: "SetPalette" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPaletteAction.java |
20 |
| Invalid string literal: "SetPenColor" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPenColorAction.java |
11 |
| Invalid string literal: "SetPenSize" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPenSizeAction.java |
11 |
| Invalid string literal: "Position" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPositionAction.java |
16 |
| Invalid string literal: "Towards" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetTowardsAction.java |
16 |
| Invalid string literal: "XCor" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
XCorAction.java |
11 |
| Invalid string literal: "invalidSLogoSyntax" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ExceptionConstants.java |
4 |
| Invalid string literal: "nullInput" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ExceptionConstants.java |
5 |
| Invalid string literal: "divideByZero" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ExceptionConstants.java |
6 |
| Invalid string literal: "fxmlException" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ExceptionConstants.java |
7 |
| Invalid string literal: "propertiesException" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ExceptionConstants.java |
8 |
| Invalid string literal: "SLogo" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
15 |
| Invalid string literal: "RootLayout.fxml" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
16 |
| Invalid string literal: "final_turtle.png" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
17 |
| Invalid string literal: "highlighted_turtle.png" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
18 |
| Invalid string literal: "backgroundButtonText" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
19 |
| Invalid string literal: "turtleImageButtonText" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
20 |
| Invalid string literal: "addWorkspaceButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
21 |
| Invalid string literal: "workspaceTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
22 |
| Invalid string literal: "penColorButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
23 |
| Invalid string literal: "penSliderTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
24 |
| Invalid string literal: "submitButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
25 |
| Invalid string literal: "gridLinesButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
26 |
| Invalid string literal: "helpPageURL" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
27 |
| Invalid string literal: "dashedPenButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
28 |
| Invalid string literal: "dottedPenButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
29 |
| Invalid string literal: "solidPenButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
30 |
| Invalid string literal: "helpPageTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
31 |
| Invalid string literal: "showTurtleButtonTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
34 |
| Invalid string literal: "addTurtleTitle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
35 |
| Invalid string literal: "resources.languages/" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GUIconstants.java |
36 |
| Invalid string literal: "resources.languages/English" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ModelConstants.java |
5 |
| Invalid string literal: "Help Page" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
HelpPageButton.java |
31 |
| Invalid string literal: "Model Created" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
29 |
| Invalid string literal: ":slogo" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
33 |
| Invalid string literal: "100" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
33 |
| Invalid string literal: "Factory loaded" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
39 |
| Invalid string literal: "An Error occured in the loading of the properties File!" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
42 |
| Invalid string literal: "\\s+" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
67 |
| Invalid string literal: "Input String: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
73 |
| Invalid string literal: "Split input: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
74 |
| Invalid string literal: "My Commands: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
75 |
| Invalid string literal: "\n\n\n*****List: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
81 |
| Invalid string literal: "\n\n****" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
81 |
| Invalid string literal: "Action List: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Model.java |
84 |
| Invalid string literal: "Properties created." |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
44 |
| Invalid string literal: "Got through basic property set up." |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
52 |
| Invalid string literal: "Forward" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
55 |
| Invalid string literal: "Backward" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
55 |
| Invalid string literal: "Left" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
55 |
| Invalid string literal: "Right" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
55 |
| Invalid string literal: "SetHeading" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
56 |
| Invalid string literal: "SetTowards" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
56 |
| Invalid string literal: "SetPosition" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
56 |
| Invalid string literal: "PenDown" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
56 |
| Invalid string literal: "PenUp" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
56 |
| Invalid string literal: "ShowTurtle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
57 |
| Invalid string literal: "HideTurtle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
57 |
| Invalid string literal: "Home" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
57 |
| Invalid string literal: "ClearScreen" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
57 |
| Invalid string literal: "XCoordinate" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
58 |
| Invalid string literal: "YCoordinate" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
58 |
| Invalid string literal: "Heading" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
58 |
| Invalid string literal: "IsPenDown" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
59 |
| Invalid string literal: "IsShowing" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PropertiesFactory.java |
59 |
| Invalid string literal: "AND" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
AndNode.java |
11 |
| Invalid string literal: "LESS" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
18 |
| Invalid string literal: "GREATER" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
18 |
| Invalid string literal: "EQUAL" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
18 |
| Invalid string literal: "NOTEQUAL" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
18 |
| Invalid string literal: "AND" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
18 |
| Invalid string literal: "NOT" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
18 |
| Invalid string literal: "BooleanOperation" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
22 |
| Invalid string literal: " Operation:" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BooleanOperations.java |
28 |
| Invalid string literal: "EQUAL" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
EqualToNode.java |
9 |
| Invalid string literal: "GREATER" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GreaterThanNode.java |
10 |
| Invalid string literal: "LESS" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
LessThanNode.java |
10 |
| Invalid string literal: "NOTEQUAL" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
NotEqualToNode.java |
10 |
| Invalid string literal: "NOT" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
NotNode.java |
16 |
| Invalid string literal: "ControlStructures" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ControlStructuresNode.java |
10 |
| Invalid string literal: "Control Structure:" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ControlStructuresNode.java |
15 |
| Invalid string literal: "DoTimes" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
DoTimesNode.java |
12 |
| Invalid string literal: "For" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ForNode.java |
13 |
| Invalid string literal: "IfElse" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
IfElseNode.java |
12 |
| Invalid string literal: "Repeat" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RepeatNode.java |
12 |
| Invalid string literal: "ClearStamps" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ClearStampsNode.java |
16 |
| Invalid string literal: "DisplayCommands" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
DisplayCommandsNode.java |
18 |
| Invalid string literal: "PenColor" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PenColorNode.java |
17 |
| Invalid string literal: "SetBackground" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetBackgroundNode.java |
17 |
| Invalid string literal: "SetPalette" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPaletteNode.java |
18 |
| Invalid string literal: "SetPenColor" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPenColorNode.java |
18 |
| Invalid string literal: "SetSizeColor" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPenSizeNode.java |
17 |
| Invalid string literal: "SetShape" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetShapeNode.java |
16 |
| Invalid string literal: "Shape" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ShapeNode.java |
16 |
| Invalid string literal: "Stamp" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
StampNode.java |
15 |
| Invalid string literal: "InteractionCommands" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
InteractionCommandsNode.java |
16 |
| Invalid string literal: "OnClick" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
OnClickNode.java |
16 |
| Invalid string literal: "OnKey" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
OnKeyNode.java |
16 |
| Invalid string literal: "OnClick" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
OnMoveNode.java |
17 |
| Invalid string literal: "Atan" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
AtanNode.java |
16 |
| Invalid string literal: "Cos" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
CosNode.java |
16 |
| Invalid string literal: "Difference" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
DifferenceNode.java |
19 |
| Invalid string literal: "Log" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
LogNode.java |
16 |
| Invalid string literal: "MathOperation" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
MathOperationsNode.java |
15 |
| Invalid string literal: "Minus" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
MinusNode.java |
17 |
| Invalid string literal: "Pow" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PowNode.java |
17 |
| Invalid string literal: "Product" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ProductNode.java |
19 |
| Invalid string literal: "Quotient" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
QuotientNode.java |
19 |
| Invalid string literal: "Random" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RandomNode.java |
16 |
| Invalid string literal: "Remainder" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RemainderNode.java |
17 |
| Invalid string literal: "Sin" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SinNode.java |
16 |
| Invalid string literal: "Sum" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SumNode.java |
19 |
| Invalid string literal: "Tan" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
TanNode.java |
16 |
| Invalid string literal: "Constant" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ConstantNode.java |
16 |
| Invalid string literal: "Number" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
NumberNode.java |
15 |
| Invalid string literal: "RightParentheses" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GroupEndNode.java |
12 |
| Invalid string literal: "Parentheses" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GroupNode.java |
12 |
| Invalid string literal: "LeftParentheses" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
GroupStartNode.java |
12 |
| Invalid string literal: "RightBracket" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ListEndNode.java |
12 |
| Invalid string literal: "Bracket" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ListNode.java |
12 |
| Invalid string literal: "LeftBracket" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ListStartNode.java |
12 |
| Invalid string literal: "Punctuation" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PunctuationNode.java |
17 |
| Invalid string literal: "SLogoNode: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SLogoNode.java |
80 |
| Invalid string literal: "MoveBackward" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
BackwardNode.java |
19 |
| Invalid string literal: "ClearScreen" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ClearScreenNode.java |
18 |
| Invalid string literal: "DisplayTurtle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
DisplayTurtleNode.java |
18 |
| Invalid string literal: "MoveForward" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ForwardNode.java |
18 |
| Invalid string literal: "ShowTurtle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
HideTurtleNode.java |
16 |
| Invalid string literal: "Home" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
HomeNode.java |
18 |
| Invalid string literal: "RotateLeft" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
LeftNode.java |
17 |
| Invalid string literal: "Move" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
MoveNode.java |
17 |
| Invalid string literal: "PenDown" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PenDownNode.java |
17 |
| Invalid string literal: "PenNode" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PenNode.java |
19 |
| Invalid string literal: "PenUp" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
PenUpNode.java |
17 |
| Invalid string literal: "RotateRight" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RightNode.java |
17 |
| Invalid string literal: "Rotate" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
RotateNode.java |
12 |
| Invalid string literal: "SetHeading" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetHeadingNode.java |
19 |
| Invalid string literal: "SetPosition" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetPositionNode.java |
18 |
| Invalid string literal: "Towards" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SetTowardsNode.java |
18 |
| Invalid string literal: "ShowTurtle" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
ShowTurtleNode.java |
16 |
| Invalid string literal: "TurtleCommands" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
TurtleCommandsNode.java |
17 |
| Invalid string literal: "TurtleQueries" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
TurtleQueriesNode.java |
10 |
| Invalid string literal: " Turtle Query: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
TurtleQueriesNode.java |
15 |
| Invalid string literal: "XCor" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
XCorNode.java |
10 |
| Invalid string literal: "Make" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
MakeNode.java |
17 |
| Invalid string literal: "ClassType" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SLogoNodeFactory.java |
35 |
| Invalid string literal: "\nERROR! THIS COMMAND DOES NOT EXIST: " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
SLogoNodeFactory.java |
43 |
| Invalid string literal: "Turtle image not found.\n" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Turtle.java |
209 |
| Invalid string literal: "Turtle image not found.\n" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
Turtle.java |
212 |
| Invalid string literal: "There are no elements size = " |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
TurtleCollection.java |
85 |
| Invalid string literal: "slogo" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
CommandsTest.java |
23 |
| Invalid string literal: "100" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
CommandsTest.java |
23 |
| Invalid string literal: "100" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
CommandsTest.java |
26 |
| Invalid string literal: "100" |
Consider defining a constant to represent the string or move it to a resource bundle. |
High |
CommandsTest.java |
26 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/Constants/GUIconstants.java" |
|
High |
GUIconstants.java |
1 |
| Maximum violation count of 20 exceeded for the rule "String Literals" in "/TempCodeProProject1/src1/application/Constants/GUIconstants.java" |
|
High |
GUIconstants.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/Controller.java" |
|
High |
Controller.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/ChooseTurtleImageButton.java" |
|
High |
ChooseTurtleImageButton.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/HelpPageButton.java" |
|
High |
HelpPageButton.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/HistoryFeature.java" |
|
High |
HistoryFeature.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/KeyControlFeature.java" |
|
High |
KeyControlFeature.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/NewWorkspaceButton.java" |
|
High |
NewWorkspaceButton.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/PenSizeSlider.java" |
|
High |
PenSizeSlider.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/ShowAndHideTurtleButton.java" |
|
High |
ShowAndHideTurtleButton.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/GUIfeatures/SLogoFileOpener.java" |
|
High |
SLogoFileOpener.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/Model.java" |
|
High |
Model.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/PropertiesFactory.java" |
|
High |
PropertiesFactory.java |
1 |
| Maximum violation count of 20 exceeded for the rule "String Literals" in "/TempCodeProProject1/src1/application/PropertiesFactory.java" |
|
High |
PropertiesFactory.java |
1 |
| Maximum violation count of 20 exceeded for the rule "Indent Code Within Blocks" in "/TempCodeProProject1/src1/application/PropertiesFactory.java" |
|
High |
PropertiesFactory.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/SLogoCanvas.java" |
|
High |
SLogoCanvas.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/slogonode/ControlStructures/ForNode.java" |
|
High |
ForNode.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/slogonode/ControlStructures/IfElseNode.java" |
|
High |
IfElseNode.java |
1 |
| Maximum violation count of 2000 exceeded for the rule "White Space Usage" |
|
High |
ProductNode.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/slogonode/SLogoNode.java" |
|
High |
SLogoNode.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/SLogoNodeFactory.java" |
|
High |
SLogoNodeFactory.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/SLogoPen.java" |
|
High |
SLogoPen.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/Turtle.java" |
|
High |
Turtle.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/TurtleCollection.java" |
|
High |
TurtleCollection.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/View.java" |
|
High |
View.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/application/Workspace.java" |
|
High |
Workspace.java |
1 |
| Maximum violation count of 20 exceeded for the rule "Indent Code Within Blocks" in "/TempCodeProProject1/src1/test/AbstractDrawerTest.java" |
|
High |
AbstractDrawerTest.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/test/BooleanOperationTest.java" |
|
High |
BooleanOperationTest.java |
1 |
| Maximum violation count of 20 exceeded for the rule "White Space Usage" in "/TempCodeProProject1/src1/test/MathOperationsTest.java" |
|
High |
MathOperationsTest.java |
1 |
| Maximum violation count of 20 exceeded for the rule "Numeric Literals" in "/TempCodeProProject1/src1/test/MathOperationsTest.java" |
|
High |
MathOperationsTest.java |
1 |
| |
|
|
|
| Violation |
Recommendation |
Severity |
Resource |
Line |
| Missing Javadoc comment for type AbstractAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
AbstractAction.java |
13 |
| Missing Javadoc comment for type Action |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
Action.java |
12 |
| Missing Javadoc comment for type ActionFactory |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ActionFactory.java |
4 |
| Missing Javadoc comment for type BackwardAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
BackwardAction.java |
7 |
| Missing Javadoc comment for type ClearScreenAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ClearScreenAction.java |
12 |
| Missing Javadoc comment for type ConstantAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ConstantAction.java |
11 |
| Missing Javadoc comment for type DisplayTurtleAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
DisplayTurtleAction.java |
12 |
| Missing Javadoc comment for type ForwardAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ForwardAction.java |
7 |
| Missing Javadoc comment for type HomeAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
HomeAction.java |
12 |
| Missing Javadoc comment for type LeftAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
LeftAction.java |
12 |
| Missing Javadoc comment for type PenUpDownAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
PenUpDownAction.java |
12 |
| Missing Javadoc comment for type RightAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
RightAction.java |
6 |
| Missing Javadoc comment for type RotateAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
RotateAction.java |
12 |
| Missing Javadoc comment for type SetBackgroundAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SetBackgroundAction.java |
7 |
| Missing Javadoc comment for type SetHeadingAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SetHeadingAction.java |
12 |
| Missing @version tag for type SetPaletteAction |
Run the Javadoc Repair tool on this type to add the missing @version tag. |
Medium |
SetPaletteAction.java |
8 |
| Missing Javadoc comment for type SetPenColorAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SetPenColorAction.java |
7 |
| Missing Javadoc comment for type SetPenSizeAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SetPenSizeAction.java |
7 |
| Missing Javadoc comment for type SetPositionAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SetPositionAction.java |
12 |
| Missing Javadoc comment for type SetTowardsAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SetTowardsAction.java |
12 |
| Missing Javadoc comment for type VariableAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
VariableAction.java |
3 |
| Missing Javadoc comment for type XCorAction |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
XCorAction.java |
7 |
| Missing Javadoc comment for type ExceptionConstants |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ExceptionConstants.java |
3 |
| Missing Javadoc comment for type GUIconstants |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
GUIconstants.java |
3 |
| Missing Javadoc comment for type ModelConstants |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ModelConstants.java |
3 |
| Missing @version tag for type Controller |
Run the Javadoc Repair tool on this type to add the missing @version tag. |
Medium |
Controller.java |
15 |
| Missing Javadoc comment for type DivideByZero |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
DivideByZero.java |
7 |
| Missing Javadoc comment for type FXMLException |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
FXMLException.java |
6 |
| Missing Javadoc comment for type InvalidSLogoSyntax |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
InvalidSLogoSyntax.java |
5 |
| Missing Javadoc comment for type NullInput |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
NullInput.java |
7 |
| Missing Javadoc comment for type PropertiesError |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
PropertiesError.java |
6 |
| Missing Javadoc comment for type SLogoException |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SLogoException.java |
3 |
| Missing Javadoc comment for type AddTurtleToWorkspaceButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
AddTurtleToWorkspaceButton.java |
8 |
| Missing Javadoc comment for type BackgroundColorPickerAndButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
BackgroundColorPickerAndButton.java |
13 |
| Missing Javadoc comment for type ChooseTurtleImageButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ChooseTurtleImageButton.java |
11 |
| Missing Javadoc comment for type ColorPickerAndButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ColorPickerAndButton.java |
7 |
| Missing Javadoc comment for type DashPenButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
DashPenButton.java |
10 |
| Missing Javadoc comment for type DotPenButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
DotPenButton.java |
10 |
| Missing @version tag for type GenericGUIFeature |
Run the Javadoc Repair tool on this type to add the missing @version tag. |
Medium |
GenericGUIFeature.java |
7 |
| Missing @version tag for type GUIButtonFeature |
Run the Javadoc Repair tool on this type to add the missing @version tag. |
Medium |
GUIButtonFeature.java |
6 |
| Missing Javadoc comment for type HelpPageButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
HelpPageButton.java |
13 |
| Missing Javadoc comment for type HistoryFeature |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
HistoryFeature.java |
8 |
| Missing Javadoc comment for type KeyControlFeature |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
KeyControlFeature.java |
22 |
| Missing Javadoc comment for type NewWorkspaceButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
NewWorkspaceButton.java |
13 |
| Missing Javadoc comment for type PenColorPickerAndButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
PenColorPickerAndButton.java |
9 |
| Missing Javadoc comment for type PenSizeSlider |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
PenSizeSlider.java |
13 |
| Missing Javadoc comment for type ShowAndHideTurtleButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ShowAndHideTurtleButton.java |
9 |
| Missing Javadoc comment for type SLogoFileOpener |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SLogoFileOpener.java |
13 |
| Missing Javadoc comment for type SolidPenButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SolidPenButton.java |
9 |
| Missing Javadoc comment for type SubmitTextButtonAndField |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
SubmitTextButtonAndField.java |
13 |
| Missing Javadoc comment for type TestButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
TestButton.java |
11 |
| Missing Javadoc comment for type ToggleReferenceGridButton |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ToggleReferenceGridButton.java |
7 |
| Missing Javadoc comment for type Window |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
Window.java |
7 |
| Missing Javadoc comment for type WorkspaceTabs |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
WorkspaceTabs.java |
5 |
| Missing Javadoc comment for type Main |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
Main.java |
6 |
| Missing Javadoc comment for type Model |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
Model.java |
22 |
| Missing Javadoc comment for type PropertiesFactory |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
PropertiesFactory.java |
12 |
| Missing @version tag for type SLogoCanvas |
Run the Javadoc Repair tool on this type to add the missing @version tag. |
Medium |
SLogoCanvas.java |
16 |
| Missing Javadoc comment for type AndNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
AndNode.java |
8 |
| Missing Javadoc comment for type BooleanOperations |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
BooleanOperations.java |
16 |
| Missing Javadoc comment for type EqualToNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
EqualToNode.java |
6 |
| Missing Javadoc comment for type GreaterThanNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
GreaterThanNode.java |
7 |
| Missing Javadoc comment for type LessThanNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
LessThanNode.java |
7 |
| Missing Javadoc comment for type NotEqualToNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
NotEqualToNode.java |
7 |
| Missing Javadoc comment for type NotNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
NotNode.java |
13 |
| Missing Javadoc comment for type OrNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
OrNode.java |
8 |
| Missing Javadoc comment for type ControlStructuresNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ControlStructuresNode.java |
5 |
| Missing Javadoc comment for type DoTimesNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
DoTimesNode.java |
8 |
| Missing Javadoc comment for type ForNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ForNode.java |
9 |
| Missing Javadoc comment for type IfElseNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
IfElseNode.java |
8 |
| Missing Javadoc comment for type IfNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
IfNode.java |
9 |
| Missing Javadoc comment for type RepeatNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
RepeatNode.java |
8 |
| Missing Javadoc comment for type ClearStampsNode |
Run the Javadoc Repair tool on this type to add missing type Javadoc. |
Medium |
ClearStampsNode.java |
12 |