src/main/java/org/opentripplanner/graph_builder

Severity

Blocker 0
Critical 2
Major 66
Minor 17
Info 1

Rule

Exit methods should not be called 2
Control structures should use curly braces 12
Exception handlers should preserve the original exceptions 9
Statements should be on separate lines 6
Field names should comply with a naming convention 4
Strings literals should be placed on the left side when checking for equality 4
Sections of code should not be "commented out" 3
Methods should not be too complex 3
Branches should have sufficient coverage by unit tests 3
Generic exceptions should never be thrown 3
Unused "private" fields should be removed 3
Package names should comply with a naming convention 3
Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList" 2
Loggers should be "private static final" and should share a naming convention 2
Standard outputs should not be used directly to log anything 2
Class variable fields should not have public accessibility 2
"switch" statements should end with "default" clauses 1
Methods should not be empty 1
"switch case" clauses should not have too many lines 1
Collapsible "if" statements should be merged 1
A close curly brace should be located at the beginning of a line 1
Magic numbers should not be used 8
Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply 3
String literals should not be duplicated 2
Tabulation characters should not be used 1
The members of an interface declaration or class should appear in a pre-defined order 1
Comments should not be located at the end of lines of code 1
Useless imports should be removed 1
"TODO" tags should be handled 1