The following document contains the results of Checkstyle 6.2 with development/dsol-checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
coding | HiddenField
|
1 | ![]() |
InnerAssignment | 1 | ![]() | |
MissingSwitchDefault | 2 | ![]() | |
design | DesignForExtension | 16 | ![]() |
VisibilityModifier | 24 | ![]() | |
javadoc | JavadocMethod | 1 | ![]() |
JavadocStyle
|
5 | ![]() | |
misc | FinalParameters | 56 | ![]() |
modifier | ModifierOrder | 34 | ![]() |
RedundantModifier | 8 | ![]() | |
naming | ConstantName | 2 | ![]() |
StaticVariableName | 2 | ![]() | |
sizes | FileLength | 1 | ![]() |
LineLength
|
4 | ![]() | |
MethodLength | 2 | ![]() | |
whitespace | WhitespaceAfter | 1 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
modifier | RedundantModifier | Redundant 'public' modifier. | 24 |
![]() |
modifier | RedundantModifier | Redundant 'public' modifier. | 31 |
![]() |
modifier | RedundantModifier | Redundant 'final' modifier. | 34 |
![]() |
modifier | RedundantModifier | Redundant 'final' modifier. | 37 |
![]() |
modifier | RedundantModifier | Redundant 'final' modifier. | 40 |
![]() |
modifier | RedundantModifier | Redundant 'final' modifier. | 43 |
![]() |
modifier | RedundantModifier | Redundant 'public' modifier. | 46 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 46 |
![]() |
modifier | RedundantModifier | Redundant 'public' modifier. | 78 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 130). | 82 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 133). | 90 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 49 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 52 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 55 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 58 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 61 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 64 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 67 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 70 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 73 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 76 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 79 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 82 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 85 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 88 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 91 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 94 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 97 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 100 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 103 |
![]() |
design | VisibilityModifier | Variable 'streams' must be private and have accessor methods. | 106 |
![]() |
design | VisibilityModifier | Variable 'routes' must be private and have accessor methods. | 109 |
![]() |
misc | FinalParameters | Parameter o1 should be final. | 123 |
![]() |
misc | FinalParameters | Parameter o2 should be final. | 123 |
![]() |
whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 142 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 279 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 311 |
![]() |
coding | MissingSwitchDefault | switch without "default" clause. | 334 |
![]() |
coding | MissingSwitchDefault | switch without "default" clause. | 356 |
![]() |
modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 399 |
![]() |
modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 402 |
![]() |
modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 405 |
![]() |
modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 408 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 627 |
![]() |
misc | FinalParameters | Parameter steps should be final. | 633 |
![]() |
coding | InnerAssignment | Inner assignments should be avoided. | 637 |
![]() |
design | DesignForExtension | Method 'render' is not designed for extension - needs to be abstract, final or empty. | 714 |
![]() |
misc | FinalParameters | Parameter e should be final. | 862 |
![]() |
design | VisibilityModifier | Variable 'testPanel' must be private and have accessor methods. | 871 |
![]() |
design | VisibilityModifier | Variable 'checkBoxPanel' must be private and have accessor methods. | 874 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | FileLength | File length is 2,688 lines (max allowed is 2,000). | 1 |
![]() |
design | VisibilityModifier | Variable 'controllerName' must be private and have accessor methods. | 59 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 62 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 65 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 68 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 71 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 74 |
![]() |
design | VisibilityModifier | Variable 'trafcodRules' must be private and have accessor methods. | 77 |
![]() |
design | VisibilityModifier | Variable 'tokenisedRules' must be private and have accessor methods. | 80 |
![]() |
design | VisibilityModifier | Variable 'variables' must be private and have accessor methods. | 83 |
![]() |
design | VisibilityModifier | Variable 'variablesInDefinitionOrder' must be private and have accessor methods. | 86 |
![]() |
design | VisibilityModifier | Variable 'detectors' must be private and have accessor methods. | 89 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 92 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 95 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 98 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 101 |
![]() |
design | VisibilityModifier | Variable 'numberOfConflictGroups' must be private and have accessor methods. | 104 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 127 |
![]() |
misc | FinalParameters | Parameter controllerName should be final. | 149 |
![]() |
misc | FinalParameters | Parameter display should be final. | 151 |
![]() |
sizes | MethodLength | Method length is 219 lines (max allowed is 150). | 229 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 129). | 389 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 129). | 394 |
![]() |
design | DesignForExtension | Method 'checkConsistency' is not designed for extension - needs to be abstract, final or empty. | 454 |
![]() |
misc | FinalParameters | Parameter sensors should be final. | 493 |
![]() |
naming | StaticVariableName | Name 'BIND_UNARY_MINUS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 891 |
![]() |
misc | FinalParameters | Parameter tokens should be final. | 1147 |
![]() |
sizes | MethodLength | Method length is 271 lines (max allowed is 150). | 1358 |
![]() |
misc | FinalParameters | Parameter name should be final. | 1671 |
![]() |
misc | FinalParameters | Parameter stream should be final. | 1671 |
![]() |
misc | FinalParameters | Parameter flags should be final. | 1671 |
![]() |
misc | FinalParameters | Parameter location should be final. | 1671 |
![]() |
design | DesignForExtension | Method 'getSimulator' is not designed for extension - needs to be abstract, final or empty. | 1709 |
![]() |
design | DesignForExtension | Method 'getStructureNumber' is not designed for extension - needs to be abstract, final or empty. | 1718 |
![]() |
design | DesignForExtension | Method 'updateDetector' is not designed for extension - needs to be abstract, final or empty. | 1724 |
![]() |
misc | FinalParameters | Parameter detectorId should be final. | 1725 |
![]() |
misc | FinalParameters | Parameter detectingGTU should be final. | 1725 |
![]() |
design | DesignForExtension | Method 'traceVariablesOfStream' is not designed for extension - needs to be abstract, final or empty. | 1742 |
![]() |
design | DesignForExtension | Method 'traceVariable' is not designed for extension - needs to be abstract, final or empty. | 1767 |
![]() |
design | DesignForExtension | Method 'notify' is not designed for extension - needs to be abstract, final or empty. | 1786 |
![]() |
misc | FinalParameters | Parameter event should be final. | 1787 |
![]() |
design | DesignForExtension | Method 'fireTrafCODEvent' is not designed for extension - needs to be abstract, final or empty. | 1857 |
![]() |
design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 1870 |
![]() |
design | VisibilityModifier | Variable 'flags' must be private and have accessor methods. | 2026 |
![]() |
design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 2029 |
![]() |
design | VisibilityModifier | Variable 'timerMax10' must be private and have accessor methods. | 2032 |
![]() |
design | VisibilityModifier | Variable 'color' must be private and have accessor methods. | 2035 |
![]() |
design | VisibilityModifier | Variable 'name' must be private and have accessor methods. | 2038 |
![]() |
design | VisibilityModifier | Variable 'listPos' must be private and have accessor methods. | 2041 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 2043 |
![]() |
design | VisibilityModifier | Variable 'stream' must be private and have accessor methods. | 2044 |
![]() |
design | VisibilityModifier | Variable 'refCount' must be private and have accessor methods. | 2047 |
![]() |
design | VisibilityModifier | Variable 'updateTime10' must be private and have accessor methods. | 2050 |
![]() |
design | VisibilityModifier | Variable 'startSource' must be private and have accessor methods. | 2053 |
![]() |
design | VisibilityModifier | Variable 'endSource' must be private and have accessor methods. | 2056 |
![]() |
naming | StaticVariableName | Name 'ROWLETTERS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2062 |
![]() |
misc | FinalParameters | Parameter trafCOD should be final. | 2070 |
![]() |
misc | FinalParameters | Parameter sensor should be final. | 2104 |
![]() |
misc | FinalParameters | Parameter newValue should be final. | 2226 |
![]() |
misc | FinalParameters | Parameter timeStamp10 should be final. | 2226 |
![]() |
misc | FinalParameters | Parameter cause should be final. | 2226 |
![]() |
misc | FinalParameters | Parameter trafCOD should be final. | 2226 |
![]() |
coding | HiddenField | 'trafCOD' hides a field. | 2226 |
![]() |
misc | FinalParameters | Parameter colorValue should be final. | 2343 |
![]() |
misc | FinalParameters | Parameter value10 should be final. | 2392 |
![]() |
misc | FinalParameters | Parameter startSource should be final. | 2416 |
![]() |
misc | FinalParameters | Parameter endSource should be final. | 2440 |
![]() |
misc | FinalParameters | Parameter printFlags should be final. | 2471 |
![]() |
misc | FinalParameters | Parameter event should be final. | 2580 |
![]() |
design | VisibilityModifier | Variable 'cause' must be private and have accessor methods. | 2601 |
![]() |
javadoc | JavadocMethod | Missing a Javadoc comment. | 2612 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'defaultInitialDelay' must be private and have accessor methods. | 54 |
![]() |
design | DesignForExtension | Method 'paintComponent' is not designed for extension - needs to be abstract, final or empty. | 68 |
![]() |
design | DesignForExtension | Method 'addTrafCODObject' is not designed for extension - needs to be abstract, final or empty. | 83 |
![]() |
design | DesignForExtension | Method 'mouseDragged' is not designed for extension - needs to be abstract, final or empty. | 89 |
![]() |
misc | FinalParameters | Parameter e should be final. | 90 |
![]() |
design | DesignForExtension | Method 'mouseMoved' is not designed for extension - needs to be abstract, final or empty. | 96 |
![]() |
misc | FinalParameters | Parameter e should be final. | 97 |
![]() |
misc | FinalParameters | Parameter e should be final. | 114 |
![]() |
misc | FinalParameters | Parameter e should be final. | 121 |
![]() |
misc | FinalParameters | Parameter e should be final. | 128 |
![]() |
design | DesignForExtension | Method 'mouseEntered' is not designed for extension - needs to be abstract, final or empty. | 134 |
![]() |
misc | FinalParameters | Parameter e should be final. | 135 |
![]() |
design | DesignForExtension | Method 'mouseExited' is not designed for extension - needs to be abstract, final or empty. | 141 |
![]() |
misc | FinalParameters | Parameter e should be final. | 142 |
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 155 |
![]() |
naming | ConstantName | Name 'xOffset' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 196 |
![]() |
naming | ConstantName | Name 'yOffset' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 199 |
![]() |
misc | FinalParameters | Parameter center should be final. | 207 |
![]() |
misc | FinalParameters | Parameter description should be final. | 207 |
![]() |
misc | FinalParameters | Parameter g2 should be final. | 218 |
![]() |
misc | FinalParameters | Parameter event should be final. | 228 |
![]() |
misc | FinalParameters | Parameter testX should be final. | 293 |
![]() |
misc | FinalParameters | Parameter testY should be final. | 293 |
![]() |
misc | FinalParameters | Parameter listener should be final. | 361 |
![]() |
misc | FinalParameters | Parameter eventType should be final. | 361 |
![]() |
misc | FinalParameters | Parameter listener should be final. | 368 |
![]() |
misc | FinalParameters | Parameter eventType should be final. | 368 |
![]() |
misc | FinalParameters | Parameter weak should be final. | 368 |
![]() |
misc | FinalParameters | Parameter listener should be final. | 375 |
![]() |
misc | FinalParameters | Parameter eventType should be final. | 375 |
![]() |
misc | FinalParameters | Parameter position should be final. | 375 |
![]() |
misc | FinalParameters | Parameter listener should be final. | 382 |
![]() |
misc | FinalParameters | Parameter eventType should be final. | 382 |
![]() |
misc | FinalParameters | Parameter position should be final. | 382 |
![]() |
misc | FinalParameters | Parameter weak should be final. | 382 |
![]() |
misc | FinalParameters | Parameter listener should be final. | 390 |
![]() |
misc | FinalParameters | Parameter eventType should be final. | 390 |
![]() |
misc | FinalParameters | Parameter trafficLightColor should be final. | 404 |
![]() |
misc | FinalParameters | Parameter g2 should be final. | 415 |