src/main/java/com/google/transit/realtime

Severity

Blocker 0
Critical 0
Major 1,194
Minor 762
Info 0

Rule

Useless parentheses around expressions should be removed to prevent any misunderstanding 305
Sections of code should not be "commented out" 180
Field names should comply with a naming convention 169
Nested code blocks should not be used 75
A close curly brace should be located at the beginning of a line 73
Statements should be on separate lines 45
Control structures should use curly braces 45
Local variable and method parameter names should comply with a naming convention 44
"switch case" clauses should not have too many lines 42
Methods should not be too complex 32
Exception handlers should preserve the original exceptions 30
Collapsible "if" statements should be merged 26
Local variables should not shadow class fields 15
Class variable fields should not have public accessibility 15
"public static" fields should be constant 15
Unused method parameters should be removed 15
Classes that override "clone" should be "Cloneable" and call "super.clone()" 15
"switch" statements should end with "default" clauses 15
Unused local variables should be removed 12
Classes should not be too complex 10
Nested blocks of code should not be left empty 7
Generic wildcard types should not be used in return parameters 5
Methods should not be empty 1
Lambdas and anonymous classes should not have too many lines 1
Branches should have sufficient coverage by unit tests 1
Source files should not have any duplicated blocks 1
The members of an interface declaration or class should appear in a pre-defined order 430
Magic numbers should not be used 256
Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply 57
Local Variables should not be declared and then immediately returned or thrown 15
String literals should not be duplicated 4