src/test/java/org/opentripplanner/routing/core

Severity

Blocker 0
Critical 0
Major 49
Minor 312
Info 1

Rule

Unused local variables should be removed 12
Control structures should use curly braces 7
Standard outputs should not be used directly to log anything 6
Source files should not have any duplicated blocks 5
Useless parentheses around expressions should be removed to prevent any misunderstanding 3
Class variable fields should not have public accessibility 3
Octal values should not be used 2
Exception handlers should preserve the original exceptions 2
Collection.isEmpty() should be used to test for emptiness 2
Loops should not contain more than a single "break" or "continue" statement 1
Unused method parameters should be removed 1
Local variable and method parameter names should comply with a naming convention 1
Field names should comply with a naming convention 1
Generic exceptions should never be thrown 1
Methods should not have too many parameters 1
Methods should not be too complex 1
Magic numbers should not be used 298
String literals should not be duplicated 7
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 2
The members of an interface declaration or class should appear in a pre-defined order 1
Local Variables should not be declared and then immediately returned or thrown 1
"TODO" tags should be handled 1