hitul

Severity

Blocker 0
Critical 0
Major 552
Minor 554
Info 14

Rule

An open curly brace should be located at the end of a line 147
Sections of code should not be "commented out" 48
Standard outputs should not be used directly to log anything 48
Control structures should use curly braces 36
Local variable and method parameter names should comply with a naming convention 31
Synchronized classes Vector, Hashtable, Stack and StringBuffer should not be used 28
Field names should comply with a naming convention 27
Generic exceptions should never be thrown 26
Exception handlers should preserve the original exceptions 17
Close curly brace and the next "else", "catch" and "finally" keywords should be located on the same line 16
Useless parentheses around expressions should be removed to prevent any misunderstanding 15
Empty statements should be removed 12
Unused local variables should be removed 11
Methods should not be too complex 10
Source files should not have any duplicated blocks 9
A close curly brace should be located at the beginning of a line 8
Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList" 8
Constant names should comply with a naming convention 7
Utility classes should not have public constructors 7
Strings literals should be placed on the left side when checking for equality 7
Class names should comply with a naming convention 4
Method names should comply with a naming convention 4
Statements should be on separate lines 4
Public methods should throw at most one checked exception 4
Unused "private" fields should be removed 3
Loops should not contain more than a single "break" or "continue" statement 2
Unused method parameters should be removed 2
Collection.isEmpty() should be used to test for emptiness 2
"switch" statements should end with "default" clauses 2
Class variable fields should not have public accessibility 2
Methods should not be empty 1
A field should not duplicate the name of its containing class 1
Methods should not have too many parameters 1
Local variables should not shadow class fields 1
Method parameters, caught exceptions and foreach variables should not be reassigned 1
Magic numbers should not be used 359
The members of an interface declaration or class should appear in a pre-defined order 56
Comments should not be located at the end of lines of code 46
Tabulation characters should not be used 28
Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply 17
Useless imports should be removed 16
String literals should not be duplicated 14
Modifiers should be declared in the correct order 14
Local Variables should not be declared and then immediately returned or thrown 2
Array designators "[]" should be on the type, not the variable 1
Overriding methods should do more than simply call the same method in the super class 1
"TODO" tags should be handled 14