src/main/java/org/opentripplanner/osm

Severity

Blocker 0
Critical 2
Major 106
Minor 40
Info 3

Rule

Exit methods should not be called 1
Throwable.printStackTrace(...) should not be called 1
Control structures should use curly braces 17
Class variable fields should not have public accessibility 15
Statements should be on separate lines 14
Branches should have sufficient coverage by unit tests 9
Sections of code should not be "commented out" 8
A close curly brace should be located at the beginning of a line 7
Methods should not be empty 7
Exception handlers should preserve the original exceptions 6
Generic exceptions should never be thrown 5
Empty statements should be removed 5
Local variables should not shadow class fields 2
Strings literals should be placed on the left side when checking for equality 2
Useless parentheses around expressions should be removed to prevent any misunderstanding 1
"switch" statements should end with "default" clauses 1
Loops should not contain more than a single "break" or "continue" statement 1
Loggers should be "private static final" and should share a naming convention 1
"FIXME" tags should be handled 1
Utility classes should not have public constructors 1
Collapsible "if" statements should be merged 1
Local variable and method parameter names should comply with a naming convention 1
Field names should comply with a naming convention 1
Magic numbers should not be used 22
Comments should not be located at the end of lines of code 15
Useless imports should be removed 3
"TODO" tags should be handled 3