src/main/java/org/opentripplanner/analyst/core

Severity

Blocker 0
Critical 1
Major 63
Minor 132
Info 4

Rule

Throwable.printStackTrace(...) should not be called 1
Control structures should use curly braces 24
Sections of code should not be "commented out" 7
Class variable fields should not have public accessibility 7
Branches should have sufficient coverage by unit tests 6
Exception handlers should preserve the original exceptions 3
Unused "private" fields should be removed 3
Local variable and method parameter names should comply with a naming convention 2
Octal values should not be used 2
Method parameters, caught exceptions and foreach variables should not be reassigned 2
Useless parentheses around expressions should be removed to prevent any misunderstanding 1
Public constants and fields initialized at declaration should be "static final" rather than merely "final" 1
Empty arrays and collections should be returned instead of null 1
"FIXME" tags should be handled 1
Utility classes should not have public constructors 1
Field names should comply with a naming convention 1
Methods should not be too complex 1
Magic numbers should not be used 105
The members of an interface declaration or class should appear in a pre-defined order 12
Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply 6
Comments should not be located at the end of lines of code 6
Tabulation characters should not be used 1
Local Variables should not be declared and then immediately returned or thrown 1
Useless imports should be removed 1
"TODO" tags should be handled 4