The following document contains the results of Checkstyle 6.18 with development/dsol-checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| coding | HiddenField
|
1 | |
| InnerAssignment | 1 | ||
| MissingSwitchDefault | 2 | ||
| design | DesignForExtension | 24 | |
| VisibilityModifier | 28 | ||
| javadoc | JavadocStyle
|
5 | |
| misc | FinalParameters | 61 | |
| modifier | ModifierOrder | 35 | |
| RedundantModifier | 21 | ||
| naming | ConstantName | 2 | |
| StaticVariableName | 3 | ||
| sizes | FileLength | 1 | |
| MethodLength | 2 | ||
| whitespace | WhitespaceAfter | 1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 27 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 34 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 37 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 40 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 43 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 46 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 49 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 52 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 52 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 84 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'id' must be private and have accessor methods. | 49 | |
| design | VisibilityModifier | Variable 'simulator' must be private and have accessor methods. | 52 | |
| naming | StaticVariableName | Name 'PORT' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 55 | |
| design | VisibilityModifier | Variable 'PORT' must be private and have accessor methods. | 55 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 58 | |
| design | VisibilityModifier | Variable 'serverSocket' must be private and have accessor methods. | 61 | |
| design | VisibilityModifier | Variable 'acceptThread' must be private and have accessor methods. | 73 | |
| design | DesignForExtension | Method 'setClientSocket' is not designed for extension - needs to be abstract, final or empty. | 129 | |
| design | DesignForExtension | Method 'getClientSocket' is not designed for extension - needs to be abstract, final or empty. | 161 | |
| design | DesignForExtension | Method 'notify' is not designed for extension - needs to be abstract, final or empty. | 192 | |
| misc | FinalParameters | Parameter event should be final. | 193 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 227 | |
| design | DesignForExtension | Method 'getFullId' is not designed for extension - needs to be abstract, final or empty. | 234 | |
| design | DesignForExtension | Method 'updateDetector' is not designed for extension - needs to be abstract, final or empty. | 241 | |
| design | DesignForExtension | Method 'clone' is not designed for extension - needs to be abstract, final or empty. | 250 | |
| misc | FinalParameters | Parameter newSimulator should be final. | 251 | |
| misc | FinalParameters | Parameter newNetwork should be final. | 251 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 48 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 51 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 54 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 57 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 60 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 63 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 66 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 69 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 72 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 75 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 78 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 81 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 84 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 87 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 90 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 93 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 96 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 99 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 102 | |
| design | VisibilityModifier | Variable 'streams' must be private and have accessor methods. | 105 | |
| design | VisibilityModifier | Variable 'routes' must be private and have accessor methods. | 108 | |
| misc | FinalParameters | Parameter o1 should be final. | 122 | |
| misc | FinalParameters | Parameter o2 should be final. | 122 | |
| whitespace | WhitespaceAfter | ',' is not followed by whitespace. | 141 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 278 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 310 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 333 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 355 | |
| modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 398 | |
| modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 401 | |
| modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 404 | |
| modifier | ModifierOrder | 'private' modifier out of order with the JLS suggestions. | 407 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 414 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 430 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 451 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 468 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 540 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 550 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 561 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 628 | |
| misc | FinalParameters | Parameter steps should be final. | 634 | |
| coding | InnerAssignment | Inner assignments should be avoided. | 638 | |
| design | DesignForExtension | Method 'render' is not designed for extension - needs to be abstract, final or empty. | 715 | |
| misc | FinalParameters | Parameter e should be final. | 863 | |
| design | VisibilityModifier | Variable 'testPanel' must be private and have accessor methods. | 872 | |
| design | VisibilityModifier | Variable 'checkBoxPanel' must be private and have accessor methods. | 875 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | FileLength | File length is 2,821 lines (max allowed is 2,000). | 1 | |
| design | VisibilityModifier | Variable 'controllerName' must be private and have accessor methods. | 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 | |
| design | VisibilityModifier | Variable 'trafcodRules' must be private and have accessor methods. | 82 | |
| design | VisibilityModifier | Variable 'tokenisedRules' must be private and have accessor methods. | 85 | |
| design | VisibilityModifier | Variable 'variables' must be private and have accessor methods. | 88 | |
| design | VisibilityModifier | Variable 'variablesInDefinitionOrder' must be private and have accessor methods. | 91 | |
| design | VisibilityModifier | Variable 'detectors' must be private and have accessor methods. | 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 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 106 | |
| design | VisibilityModifier | Variable 'numberOfConflictGroups' must be private and have accessor methods. | 109 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 132 | |
| misc | FinalParameters | Parameter controllerName should be final. | 154 | |
| misc | FinalParameters | Parameter display should be final. | 155 | |
| misc | FinalParameters | Parameter controllerName should be final. | 229 | |
| misc | FinalParameters | Parameter display should be final. | 229 | |
| sizes | MethodLength | Method length is 219 lines (max allowed is 150). | 247 | |
| design | DesignForExtension | Method 'checkConsistency' is not designed for extension - needs to be abstract, final or empty. | 472 | |
| misc | FinalParameters | Parameter sensors should be final. | 511 | |
| naming | StaticVariableName | Name 'BIND_UNARY_MINUS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 908 | |
| misc | FinalParameters | Parameter tokens should be final. | 1164 | |
| sizes | MethodLength | Method length is 269 lines (max allowed is 150). | 1375 | |
| misc | FinalParameters | Parameter name should be final. | 1686 | |
| misc | FinalParameters | Parameter stream should be final. | 1686 | |
| misc | FinalParameters | Parameter flags should be final. | 1686 | |
| misc | FinalParameters | Parameter location should be final. | 1686 | |
| design | DesignForExtension | Method 'getSimulator' is not designed for extension - needs to be abstract, final or empty. | 1724 | |
| design | DesignForExtension | Method 'getStructureNumber' is not designed for extension - needs to be abstract, final or empty. | 1733 | |
| design | DesignForExtension | Method 'updateDetector' is not designed for extension - needs to be abstract, final or empty. | 1739 | |
| misc | FinalParameters | Parameter detectorId should be final. | 1740 | |
| misc | FinalParameters | Parameter detectingGTU should be final. | 1740 | |
| design | DesignForExtension | Method 'traceVariablesOfStream' is not designed for extension - needs to be abstract, final or empty. | 1756 | |
| design | DesignForExtension | Method 'traceVariable' is not designed for extension - needs to be abstract, final or empty. | 1781 | |
| design | DesignForExtension | Method 'notify' is not designed for extension - needs to be abstract, final or empty. | 1800 | |
| misc | FinalParameters | Parameter event should be final. | 1801 | |
| design | DesignForExtension | Method 'fireTrafCODEvent' is not designed for extension - needs to be abstract, final or empty. | 1871 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 1877 | |
| design | DesignForExtension | Method 'getFullId' is not designed for extension - needs to be abstract, final or empty. | 1884 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 1968 | |
| design | VisibilityModifier | Variable 'flags' must be private and have accessor methods. | 2094 | |
| design | VisibilityModifier | Variable 'value' must be private and have accessor methods. | 2097 | |
| design | VisibilityModifier | Variable 'timerMax10' must be private and have accessor methods. | 2100 | |
| design | VisibilityModifier | Variable 'color' must be private and have accessor methods. | 2103 | |
| design | VisibilityModifier | Variable 'name' must be private and have accessor methods. | 2106 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 2108 | |
| design | VisibilityModifier | Variable 'stream' must be private and have accessor methods. | 2109 | |
| design | VisibilityModifier | Variable 'refCount' must be private and have accessor methods. | 2112 | |
| design | VisibilityModifier | Variable 'updateTime10' must be private and have accessor methods. | 2115 | |
| design | VisibilityModifier | Variable 'startSource' must be private and have accessor methods. | 2118 | |
| design | VisibilityModifier | Variable 'endSource' must be private and have accessor methods. | 2121 | |
| naming | StaticVariableName | Name 'ROWLETTERS' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 2127 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 2165 | |
| misc | FinalParameters | Parameter trafCOD should be final. | 2165 | |
| misc | FinalParameters | Parameter sensor should be final. | 2199 | |
| misc | FinalParameters | Parameter newValue should be final. | 2321 | |
| misc | FinalParameters | Parameter timeStamp10 should be final. | 2321 | |
| misc | FinalParameters | Parameter cause should be final. | 2321 | |
| misc | FinalParameters | Parameter trafCOD should be final. | 2321 | |
| coding | HiddenField | 'trafCOD' hides a field. | 2321 | |
| misc | FinalParameters | Parameter colorValue should be final. | 2475 | |
| misc | FinalParameters | Parameter value10 should be final. | 2524 | |
| misc | FinalParameters | Parameter startSource should be final. | 2548 | |
| misc | FinalParameters | Parameter endSource should be final. | 2572 | |
| misc | FinalParameters | Parameter printFlags should be final. | 2603 | |
| misc | FinalParameters | Parameter event should be final. | 2712 | |
| design | VisibilityModifier | Variable 'cause' must be private and have accessor methods. | 2733 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 2739 |
| 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. | 69 | |
| design | DesignForExtension | Method 'addTrafCODObject' is not designed for extension - needs to be abstract, final or empty. | 84 | |
| design | DesignForExtension | Method 'mouseDragged' is not designed for extension - needs to be abstract, final or empty. | 90 | |
| misc | FinalParameters | Parameter e should be final. | 91 | |
| design | DesignForExtension | Method 'mouseMoved' is not designed for extension - needs to be abstract, final or empty. | 97 | |
| misc | FinalParameters | Parameter e should be final. | 98 | |
| misc | FinalParameters | Parameter e should be final. | 115 | |
| misc | FinalParameters | Parameter e should be final. | 122 | |
| misc | FinalParameters | Parameter e should be final. | 129 | |
| design | DesignForExtension | Method 'mouseEntered' is not designed for extension - needs to be abstract, final or empty. | 135 | |
| misc | FinalParameters | Parameter e should be final. | 136 | |
| design | DesignForExtension | Method 'mouseExited' is not designed for extension - needs to be abstract, final or empty. | 142 | |
| misc | FinalParameters | Parameter e should be final. | 143 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 156 | |
| naming | ConstantName | Name 'xOffset' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 197 | |
| naming | ConstantName | Name 'yOffset' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 200 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 208 | |
| misc | FinalParameters | Parameter center should be final. | 208 | |
| misc | FinalParameters | Parameter description should be final. | 208 | |
| misc | FinalParameters | Parameter g2 should be final. | 219 | |
| misc | FinalParameters | Parameter event should be final. | 229 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 283 | |
| misc | FinalParameters | Parameter testX should be final. | 294 | |
| misc | FinalParameters | Parameter testY should be final. | 294 | |
| misc | FinalParameters | Parameter listener should be final. | 369 | |
| misc | FinalParameters | Parameter eventType should be final. | 369 | |
| misc | FinalParameters | Parameter listener should be final. | 376 | |
| misc | FinalParameters | Parameter eventType should be final. | 376 | |
| misc | FinalParameters | Parameter weak should be final. | 376 | |
| misc | FinalParameters | Parameter listener should be final. | 383 | |
| misc | FinalParameters | Parameter eventType should be final. | 383 | |
| misc | FinalParameters | Parameter position should be final. | 383 | |
| misc | FinalParameters | Parameter listener should be final. | 390 | |
| misc | FinalParameters | Parameter eventType should be final. | 390 | |
| misc | FinalParameters | Parameter position should be final. | 390 | |
| misc | FinalParameters | Parameter weak should be final. | 390 | |
| misc | FinalParameters | Parameter listener should be final. | 398 | |
| misc | FinalParameters | Parameter eventType should be final. | 398 | |
| misc | FinalParameters | Parameter trafficLightColor should be final. | 412 | |
| misc | FinalParameters | Parameter g2 should be final. | 423 |