src/main/java/org/opentripplanner/openstreetmap/impl

Severity

Blocker 0
Critical 0
Major 70
Minor 34
Info 2

Rule

Field names should comply with a naming convention 20
Strings literals should be placed on the left side when checking for equality 13
Generic exceptions should never be thrown 7
Branches should have sufficient coverage by unit tests 6
Control structures should use curly braces 6
Public methods should throw at most one checked exception 4
A close curly brace should be located at the beginning of a line 2
Methods should not be too complex 2
Collapsible "if" statements should be merged 2
Empty statements should be removed 2
Source files should not have any duplicated blocks 2
Utility classes should not have public constructors 1
Exception handlers should preserve the original exceptions 1
Loggers should be "private static final" and should share a naming convention 1
Loops should not contain more than a single "break" or "continue" statement 1
Magic numbers should not be used 17
Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply 7
Comments should not be located at the end of lines of code 6
The members of an interface declaration or class should appear in a pre-defined order 2
Tabulation characters should not be used 1
Local Variables should not be declared and then immediately returned or thrown 1
"TODO" tags should be handled 2