src/main/java/org/opentripplanner/common

Severity

Blocker 1
Critical 3
Major 114
Minor 62
Info 4

Rule

Control structures should use curly braces 32
Class variable fields should not have public accessibility 22
Branches should have sufficient coverage by unit tests 8
Declarations should use Java collection interfaces such as "List" rather than specific implementation classes such as "LinkedList" 7
Local variable and method parameter names should comply with a naming convention 7
Statements should be on separate lines 7
Exception handlers should preserve the original exceptions 4
Methods should not be too complex 4
Standard outputs should not be used directly to log anything 3
Utility classes should not have public constructors 3
Loops should not contain more than a single "break" or "continue" statement 2
A close curly brace should be located at the beginning of a line 2
Constant names should comply with a naming convention 2
Field names should comply with a naming convention 2
Sections of code should not be "commented out" 2
"switch" statements should end with "default" clauses 1
Loggers should be "private static final" and should share a naming convention 1
"FIXME" tags should be handled 1
Strings literals should be placed on the left side when checking for equality 1
Unused "private" fields should be removed 1
Generic exceptions should never be thrown 1
Method names should comply with a naming convention 1