src/main/java/org/opentripplanner/graph_builder/module

Severity

Blocker 0
Critical 2
Major 123
Minor 37
Info 6

Rule

Control structures should use curly braces 23
Package names should comply with a naming convention 16
Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList" 13
Branches should have sufficient coverage by unit tests 13
Field names should comply with a naming convention 10
Statements should be on separate lines 10
Methods should not be too complex 6
Class variable fields should not have public accessibility 6
"FIXME" tags should be handled 5
Generic exceptions should never be thrown 4
Collapsible "if" statements should be merged 3
Loggers should be "private static final" and should share a naming convention 3
Sections of code should not be "commented out" 2
Local variable and method parameter names should comply with a naming convention 1
A close curly brace should be located at the beginning of a line 1
Empty statements should be removed 1
Unused "private" fields should be removed 1
Collection.isEmpty() should be used to test for emptiness 1
Unused method parameters should be removed 1
Methods should not be empty 1
Source files should not have any duplicated blocks 1
Loops should not contain more than a single "break" or "continue" statement 1