The following document contains the results of Checkstyle 6.2 with development/dsol-checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | AvoidNestedBlocks
|
2 | |
| EmptyBlock | 7 | ||
| NeedBraces | 8 | ||
| design | DesignForExtension | 46 | |
| HideUtilityClassConstructor | 1 | ||
| VisibilityModifier | 7 | ||
| imports | UnusedImports | 7 | |
| javadoc | JavadocMethod | 1 | |
| JavadocVariable | 4 | ||
| misc | FinalParameters | 22 | |
NewlineAtEndOfFile
|
6 | ||
| TodoComment | 8 | ||
| modifier | RedundantModifier | 2 | |
| sizes | LineLength
|
14 | |
| MethodLength | 6 | ||
| ParameterNumber | 9 | ||
| whitespace | NoWhitespaceBefore | 1 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getRelativeBin' is not designed for extension - needs to be abstract, final or empty. | 75 | |
| design | DesignForExtension | Method 'adjustMaximumValue' is not designed for extension - needs to be abstract, final or empty. | 84 | |
| design | DesignForExtension | Method 'getValue' is not designed for extension - needs to be abstract, final or empty. | 95 | |
| design | DesignForExtension | Method 'getAggregatedBinCount' is not designed for extension - needs to be abstract, final or empty. | 103 | |
| design | DesignForExtension | Method 'getBinCount' is not designed for extension - needs to be abstract, final or empty. | 112 | |
| design | DesignForExtension | Method 'getCurrentGranularity' is not designed for extension - needs to be abstract, final or empty. | 121 | |
| design | DesignForExtension | Method 'setCurrentGranularity' is not designed for extension - needs to be abstract, final or empty. | 131 | |
| design | DesignForExtension | Method 'getMaximumValue' is not designed for extension - needs to be abstract, final or empty. | 148 | |
| design | DesignForExtension | Method 'setMaximumValue' is not designed for extension - needs to be abstract, final or empty. | 158 | |
| design | DesignForExtension | Method 'getMinimumValue' is not designed for extension - needs to be abstract, final or empty. | 171 | |
| design | DesignForExtension | Method 'getFormat' is not designed for extension - needs to be abstract, final or empty. | 189 | |
| design | DesignForExtension | Method 'getShortName' is not designed for extension - needs to be abstract, final or empty. | 198 | |
| design | DesignForExtension | Method 'getName' is not designed for extension - needs to be abstract, final or empty. | 207 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 123 | |
| blocks | EmptyBlock | Must have at least one statement. | 202 | |
| design | DesignForExtension | Method 'createChart' is not designed for extension - needs to be abstract, final or empty. | 267 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. | ||
| blocks | EmptyBlock | Must have at least one statement. | 71 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 77 | |
| whitespace | NoWhitespaceBefore | ';' is preceded with whitespace. | 84 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'flow' must be private and have accessor methods. | 139 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'sample' is not designed for extension - needs to be abstract, final or empty. | 182 | |
| blocks | EmptyBlock | Must have at least one statement. | 256 | |
| design | DesignForExtension | Method 'createChart' is not designed for extension - needs to be abstract, final or empty. | 291 | |
| misc | FinalParameters | Parameter gtu should be final. | 595 | |
| misc | FinalParameters | Parameter lane should be final. | 595 | |
| misc | FinalParameters | Parameter position should be final. | 595 | |
| misc | FinalParameters | Parameter item should be final. | 634 | |
| misc | FinalParameters | Parameter item should be final. | 641 | |
| misc | FinalParameters | Parameter item should be final. | 653 | |
| design | VisibilityModifier | Variable 'distance' must be private and have accessor methods. | 676 | |
| design | VisibilityModifier | Variable 'time' must be private and have accessor methods. | 679 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.road.gtu.lane.perception.LanePerception. | 32 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 309 | |
| blocks | AvoidNestedBlocks | Avoid nested blocks. | 337 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 370 | |
| blocks | AvoidNestedBlocks | Avoid nested blocks. | 398 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'checkEnoughSpace' is not designed for extension - needs to be abstract, final or empty. | 19 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'reTryInterval' must be private and have accessor methods. | 78 | |
| design | VisibilityModifier | Variable 'initialLongitudinalPositions' must be private and have accessor methods. | 81 | |
| design | VisibilityModifier | Variable 'roomChecker' must be private and have accessor methods. | 84 | |
| design | VisibilityModifier | Variable 'gtuColorer' must be private and have accessor methods. | 87 | |
| sizes | ParameterNumber | More than 7 parameters (found 10). | 108 | |
| misc | FinalParameters | Parameter id should be final. | 108 | |
| misc | FinalParameters | Parameter roomChecker should be final. | 111 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 281 | |
| design | DesignForExtension | Method 'getGeneratedGTUs' is not designed for extension - needs to be abstract, final or empty. | 289 | |
| design | DesignForExtension | Method 'setGeneratedGTUs' is not designed for extension - needs to be abstract, final or empty. | 297 | |
| misc | FinalParameters | Parameter generatedGTUs should be final. | 297 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 306 | |
| design | DesignForExtension | Method 'getEndTime' is not designed for extension - needs to be abstract, final or empty. | 315 | |
| design | DesignForExtension | Method 'getMaxGTUs' is not designed for extension - needs to be abstract, final or empty. | 324 | |
| design | DesignForExtension | Method 'getGtuColorer' is not designed for extension - needs to be abstract, final or empty. | 333 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 355 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 11). | 101 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 191 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 24 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method length is 162 lines (max allowed is 150). | 535 | |
| blocks | EmptyBlock | Must have at least one statement. | 898 | |
| blocks | EmptyBlock | Must have at least one statement. | 913 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 490 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocVariable | Missing a Javadoc comment. | 25 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - java.util.Set. | 5 | |
| imports | UnusedImports | Unused import - org.djunits.value.vdouble.scalar.Length. | 9 | |
| imports | UnusedImports | Unused import - org.opentrafficsim.road.network.lane.object.trafficlight.SimpleTrafficLight. | 20 | |
| blocks | EmptyBlock | Must have at least one statement. | 99 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.djunits.value.vdouble.scalar.Length. | 7 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 42 | |
| javadoc | JavadocVariable | Missing a Javadoc comment. | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 85 | |
| sizes | ParameterNumber | More than 7 parameters (found 9). | 199 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 796 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method length is 152 lines (max allowed is 150). | 263 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 240 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 388 | |
| sizes | MethodLength | Method length is 168 lines (max allowed is 150). | 450 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 668 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 54 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 15 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method length is 214 lines (max allowed is 150). | 90 | |
| sizes | MethodLength | Method length is 175 lines (max allowed is 150). | 318 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 841 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 864 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 949 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 28 | |
| misc | FinalParameters | Parameter args should be final. | 35 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 552 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 716 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.road.network.lane.object.trafficlight.TrafficLightColor. | 19 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'generateTacticalPlanner' is not designed for extension - needs to be abstract, final or empty. | 42 | |
| design | DesignForExtension | Method 'nextNode' is not designed for extension - needs to be abstract, final or empty. | 49 | |
| misc | FinalParameters | Parameter link should be final. | 50 | |
| misc | FinalParameters | Parameter direction should be final. | 50 | |
| misc | FinalParameters | Parameter gtuType should be final. | 50 | |
| design | DesignForExtension | Method 'nextLinkDirection' is not designed for extension - needs to be abstract, final or empty. | 56 | |
| misc | FinalParameters | Parameter link should be final. | 57 | |
| misc | FinalParameters | Parameter direction should be final. | 57 | |
| misc | FinalParameters | Parameter gtuType should be final. | 57 | |
| design | DesignForExtension | Method 'nextNode' is not designed for extension - needs to be abstract, final or empty. | 64 | |
| misc | FinalParameters | Parameter node should be final. | 65 | |
| misc | FinalParameters | Parameter previousLink should be final. | 65 | |
| misc | FinalParameters | Parameter gtuType should be final. | 65 | |
| design | DesignForExtension | Method 'nextLinkDirection' is not designed for extension - needs to be abstract, final or empty. | 71 | |
| misc | FinalParameters | Parameter node should be final. | 72 | |
| misc | FinalParameters | Parameter previousLink should be final. | 72 | |
| misc | FinalParameters | Parameter gtuType should be final. | 72 | |
| design | DesignForExtension | Method 'getRoute' is not designed for extension - needs to be abstract, final or empty. | 78 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 134). | 57 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| javadoc | JavadocMethod | Missing a Javadoc comment. | 490 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 587 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'generateTacticalPlanner' is not designed for extension - needs to be abstract, final or empty. | 82 | |
| design | DesignForExtension | Method 'nextNode' is not designed for extension - needs to be abstract, final or empty. | 89 | |
| design | DesignForExtension | Method 'nextLinkDirection' is not designed for extension - needs to be abstract, final or empty. | 97 | |
| design | DesignForExtension | Method 'nextNode' is not designed for extension - needs to be abstract, final or empty. | 106 | |
| design | DesignForExtension | Method 'nextLinkDirection' is not designed for extension - needs to be abstract, final or empty. | 114 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 141). | 86 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 179 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getLocation' is not designed for extension - needs to be abstract, final or empty. | 90 | |
| design | DesignForExtension | Method 'hashCode' is not designed for extension - needs to be abstract, final or empty. | 105 | |
| design | DesignForExtension | Method 'equals' is not designed for extension - needs to be abstract, final or empty. | 117 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 120 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 122 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 124 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 127 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 131 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 134 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 138 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 141 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 147 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'addDirectionality' is not designed for extension - needs to be abstract, final or empty. | 1350 | |
| design | DesignForExtension | Method 'removeDirectionality' is not designed for extension - needs to be abstract, final or empty. | 1363 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 56 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 129). | 83 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'clone' is not designed for extension - needs to be abstract, final or empty. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 8). | 74 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 62 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 89 | |
| sizes | MethodLength | Method length is 188 lines (max allowed is 150). | 180 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 255 | |
| blocks | EmptyBlock | Must have at least one statement. | 432 | |
| sizes | LineLength | Line is longer than 128 characters (found 129). | 498 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 421 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.djunits.unit.SpeedUnit. | 3 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 335 |