|
Instance methods should not write to \"static\" fields |
1
|
0
|
1
|
|
\"public static\" fields should be constant |
1
|
0
|
1
|
|
Throwable.printStackTrace(...) should not be called |
1
|
0
|
1
|
|
\"@Override\" annotation should be used on any method overriding (since Java 5) or implementing (since Java 6) another one |
45
|
0
|
45
|
|
Unused local variables should be removed |
6
|
0
|
6
|
|
Class variable fields should not have public accessibility |
1
|
0
|
1
|
|
Methods and field names should not be the same or differ only by capitalization |
1
|
0
|
1
|
|
Anonymous inner classes containing only one method should become lambdas |
41
|
0
|
41
|
|
Exception handlers should preserve the original exceptions |
2
|
0
|
2
|
|
Useless parentheses around expressions should be removed to prevent any misunderstanding |
2
|
0
|
2
|
|
Methods should not be too complex |
7
|
0
|
7
|
|
Sections of code should not be \"commented out\" |
11
|
0
|
11
|
|
\"switch case\" clauses should not have too many lines of code |
4
|
0
|
4
|
|
Inheritance tree of classes should not be too deep |
1
|
0
|
1
|
|
Utility classes should not have public constructors |
13
|
0
|
13
|
|
Declarations should use Java collection interfaces such as \"List\" rather than specific implementation classes such as \"LinkedList\" |
17
|
0
|
17
|
|
Classes with only \"static\" methods should not be instantiated |
1
|
0
|
1
|
|
Control flow statements \"if\", \"for\", \"while\", \"switch\" and \"try\" should not be nested too deeply |
9
|
0
|
9
|
|
Two branches in the same conditional structure should not have exactly the same implementation |
1
|
0
|
1
|
|
Methods should not have too many parameters |
2
|
0
|
2
|
|
Collapsible \"if\" statements should be merged |
1
|
0
|
1
|
|
Unused \"private\" fields should be removed |
3
|
0
|
3
|
|
Standard outputs should not be used directly to log anything |
38
|
0
|
38
|
|
Unused method parameters should be removed |
41
|
0
|
41
|
|
Dead stores should be removed |
33
|
0
|
33
|
|
Catches should be combined |
3
|
0
|
3
|
|
Local Variables should not be declared and then immediately returned or thrown |
12
|
0
|
12
|
|
Class names should comply with a naming convention |
1
|
0
|
1
|
|
Method names should comply with a naming convention |
52
|
0
|
52
|
|
Useless imports should be removed |
19
|
0
|
19
|
|
Array designators \"[]\" should be on the type, not the variable |
19
|
0
|
19
|
|
Method parameters, caught exceptions and foreach variables should not be reassigned |
3
|
0
|
3
|
|
Static non-final field names should comply with a naming convention |
1
|
0
|
1
|
|
Public types, methods and fields (API) should be documented with Javadoc |
53
|
0
|
53
|
|
Package names should comply with a naming convention |
13
|
0
|
13
|
|
Statements should be on separate lines |
45
|
0
|
45
|
|
Multiple variables should not be declared on the same line |
42
|
0
|
42
|
|
Local variable and method parameter names should comply with a naming convention |
50
|
0
|
50
|
|
The members of an interface declaration or class should appear in a pre-defined order |
136
|
0
|
136
|
|
Field names should comply with a naming convention |
59
|
0
|
59
|
|
\"TODO\" tags should be handled |
40
|
0
|
40
|