src/main/java/org/opentripplanner/updater/stoptime

Severity

Blocker 0
Critical 0
Major 32
Minor 16
Info 4

Rule

Branches should have sufficient coverage by unit tests 5
Class variable fields should not have public accessibility 3
Unused method parameters should be removed 3
Local variables should not shadow class fields 3
Strings literals should be placed on the left side when checking for equality 2
Sections of code should not be "commented out" 2
Loggers should be "private static final" and should share a naming convention 2
Methods should not be empty 2
Collection.isEmpty() should be used to test for emptiness 2
"switch" statements should end with "default" clauses 1
Loops should not contain more than a single "break" or "continue" statement 1
Lambdas and anonymous classes should not have too many lines 1
Exception handlers should preserve the original exceptions 1
"FIXME" tags should be handled 1
Statements should be on separate lines 1
Control structures should use curly braces 1
Methods should not be too complex 1
Magic numbers should not be used 4
Useless imports should be removed 4
Tabulation characters should not be used 3
Comments should not be located at the end of lines of code 3
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 4