src/main/java/org/opentripplanner/analyst/batch

Severity

Blocker 0
Critical 6
Major 109
Minor 60
Info 3

Rule

Class variable fields should not have public accessibility 35
Control structures should use curly braces 25
Branches should have sufficient coverage by unit tests 11
Exception handlers should preserve the original exceptions 7
Sections of code should not be "commented out" 6
Local variables should not shadow class fields 4
Generic exceptions should never be thrown 3
A close curly brace should be located at the beginning of a line 3
Methods should not be too complex 3
Unused "private" fields should be removed 3
Local variable and method parameter names should comply with a naming convention 2
Close curly brace and the next "else", "catch" and "finally" keywords should be located on the same line 2
Empty statements should be removed 1
Strings literals should be placed on the left side when checking for equality 1
Try-catch blocks should not be nested 1
Source files should not have any duplicated blocks 1
Loggers should be "private static final" and should share a naming convention 1