src/main/java/org/opentripplanner/graph_builder

Severity

Blocker 0
Critical 2
Major 66
Minor 17
Info 1

Rule

Control structures should use curly braces 12
Exception handlers should preserve the original exceptions 9
Statements should be on separate lines 6
Strings literals should be placed on the left side when checking for equality 4
Field names should comply with a naming convention 4
Unused "private" fields should be removed 3
Package names should comply with a naming convention 3
Generic exceptions should never be thrown 3
Methods should not be too complex 3
Sections of code should not be "commented out" 3
Branches should have sufficient coverage by unit tests 3
Standard outputs should not be used directly to log anything 2
Class variable fields should not have public accessibility 2
Loggers should be "private static final" and should share a naming convention 2
Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList" 2
Collapsible "if" statements should be merged 1
A close curly brace should be located at the beginning of a line 1
"switch case" clauses should not have too many lines 1
Methods should not be empty 1
"switch" statements should end with "default" clauses 1