|
Resources should be closed |
0
|
0
|
2
|
|
Instance methods should not write to \"static\" fields |
0
|
0
|
1
|
|
\"public static\" fields should be constant |
0
|
0
|
7
|
|
Generic exceptions should never be thrown |
0
|
0
|
1
|
|
Throwable.printStackTrace(...) should not be called |
0
|
0
|
6
|
|
Math operands should be cast before assignment |
0
|
0
|
2
|
|
Try-with-resources should be used |
0
|
0
|
1
|
|
The diamond operator (\"<>\") should be used |
0
|
0
|
2
|
|
Class variable fields should not have public accessibility |
0
|
0
|
7
|
|
Anonymous inner classes containing only one method should become lambdas |
0
|
0
|
2
|
|
Exception handlers should preserve the original exceptions |
0
|
0
|
8
|
|
Useless parentheses around expressions should be removed to prevent any misunderstanding |
0
|
0
|
4
|
|
Unused \"private\" methods should be removed |
0
|
0
|
2
|
|
Methods should not be too complex |
0
|
0
|
3
|
|
Sections of code should not be \"commented out\" |
0
|
0
|
5
|
|
Utility classes should not have public constructors |
0
|
0
|
5
|
|
Control flow statements \"if\", \"for\", \"while\", \"switch\" and \"try\" should not be nested too deeply |
0
|
0
|
9
|
|
Source files should not have any duplicated blocks |
0
|
0
|
1
|
|
Two branches in the same conditional structure should not have exactly the same implementation |
0
|
0
|
1
|
|
\"for\" loop incrementers should modify the variable being tested in the loop\'s stop condition |
0
|
0
|
2
|
|
\"for\" loop stop conditions should be invariant |
0
|
0
|
4
|
|
Local variables should not shadow class fields |
0
|
0
|
1
|
|
Methods should not have too many parameters |
0
|
0
|
4
|
|
Methods should not be empty |
0
|
0
|
3
|
|
Collapsible \"if\" statements should be merged |
0
|
0
|
1
|
|
Nested blocks of code should not be left empty |
0
|
0
|
2
|
|
Unused \"private\" fields should be removed |
0
|
0
|
5
|
|
Exit methods should not be called |
0
|
0
|
1
|
|
Standard outputs should not be used directly to log anything |
0
|
0
|
16
|
|
Unused method parameters should be removed |
0
|
0
|
1
|
|
Dead stores should be removed |
0
|
0
|
10
|
|
Strings literals should be placed on the left side when checking for equality |
0
|
0
|
6
|
|
Catches should be combined |
0
|
0
|
2
|
|
String literals should not be duplicated |
0
|
0
|
9
|
|
Useless imports should be removed |
0
|
0
|
4
|
|
Method parameters, caught exceptions and foreach variables should not be reassigned |
0
|
0
|
11
|
|
Static non-final field names should comply with a naming convention |
0
|
0
|
1
|
|
Public types, methods and fields (API) should be documented with Javadoc |
0
|
0
|
146
|
|
Package names should comply with a naming convention |
0
|
0
|
1
|
|
Statements should be on separate lines |
0
|
0
|
1
|
|
Multiple variables should not be declared on the same line |
0
|
0
|
34
|
|
Local variable and method parameter names should comply with a naming convention |
0
|
0
|
18
|
|
Field names should comply with a naming convention |
0
|
0
|
5
|
|
\"TODO\" tags should be handled |
0
|
0
|
5
|