src/main/java/org/opentripplanner/updater

Severity

Blocker 0
Critical 0
Major 35
Minor 22
Info 5

Rule

Close curly brace and the next "else", "catch" and "finally" keywords should be located on the same line 8
Strings literals should be placed on the left side when checking for equality 8
Loggers should be "private static final" and should share a naming convention 3
Branches should have sufficient coverage by unit tests 2
Control structures should use curly braces 2
Try-catch blocks should not be nested 2
Sections of code should not be "commented out" 2
Generic wildcard types should not be used in return parameters 1
Public methods should throw at most one checked exception 1
Collection.isEmpty() should be used to test for emptiness 1
"FIXME" tags should be handled 1
Utility classes should not have public constructors 1
Statements should be on separate lines 1
Generic exceptions should never be thrown 1
Methods should not be too complex 1
Useless imports should be removed 10
Modifiers should be declared in the correct order 4
Magic numbers should not be used 4
The members of an interface declaration or class should appear in a pre-defined order 2
Control flow statements "if", "for", "while", "switch" and "try" should not be nested too deeply 1
Local Variables should not be declared and then immediately returned or thrown 1
"TODO" tags should be handled 5