The following document contains the results of Checkstyle 6.2 with development/dsol-checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | AvoidNestedBlocks
|
2 | |
| EmptyBlock | 9 | ||
| NeedBraces | 8 | ||
| coding | HiddenField
|
2 | |
| design | DesignForExtension | 51 | |
| HideUtilityClassConstructor | 1 | ||
| VisibilityModifier | 8 | ||
| imports | UnusedImports | 5 | |
| javadoc | JavadocMethod | 1 | |
| JavadocVariable | 2 | ||
| misc | FinalParameters | 23 | |
NewlineAtEndOfFile
|
7 | ||
| TodoComment | 6 | ||
| modifier | RedundantModifier | 2 | |
| sizes | LineLength
|
17 | |
| MethodLength | 4 | ||
| ParameterNumber | 7 | ||
| whitespace | MethodParamPad | 1 | |
| 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. | 76 | |
| design | VisibilityModifier | Variable 'initialLongitudinalPositions' must be private and have accessor methods. | 79 | |
| design | VisibilityModifier | Variable 'roomChecker' must be private and have accessor methods. | 82 | |
| design | VisibilityModifier | Variable 'gtuColorer' must be private and have accessor methods. | 85 | |
| sizes | ParameterNumber | More than 7 parameters (found 10). | 105 | |
| misc | FinalParameters | Parameter id should be final. | 105 | |
| misc | FinalParameters | Parameter roomChecker should be final. | 108 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 253 | |
| design | DesignForExtension | Method 'getGeneratedGTUs' is not designed for extension - needs to be abstract, final or empty. | 261 | |
| design | DesignForExtension | Method 'setGeneratedGTUs' is not designed for extension - needs to be abstract, final or empty. | 269 | |
| misc | FinalParameters | Parameter generatedGTUs should be final. | 269 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 278 | |
| design | DesignForExtension | Method 'getEndTime' is not designed for extension - needs to be abstract, final or empty. | 287 | |
| design | DesignForExtension | Method 'getMaxGTUs' is not designed for extension - needs to be abstract, final or empty. | 296 | |
| design | DesignForExtension | Method 'getGtuColorer' is not designed for extension - needs to be abstract, final or empty. | 305 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 327 |
| 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 156 lines (max allowed is 150). | 501 | |
| blocks | EmptyBlock | Must have at least one statement. | 858 | |
| blocks | EmptyBlock | Must have at least one statement. | 873 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.road.gtu.strategical.LaneBasedStrategicalPlanner. | 7 |
| 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 |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 135). | 75 | |
| blocks | EmptyBlock | Must have at least one statement. | 103 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.road.network.speed.SpeedLimitProspect. | 10 |
| 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 |
|---|---|---|---|---|
| misc | FinalParameters | Parameter operationalPlanSegmentList should be final. | 53 | |
| sizes | LineLength | Line is longer than 128 characters (found 137). | 69 | |
| design | DesignForExtension | Method 'isLaneChange' is not designed for extension - needs to be abstract, final or empty. | 82 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - javax.management.RuntimeErrorException. | 8 | |
| 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). | 764 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 207 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 318 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 353 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 371 | |
| sizes | MethodLength | Method length is 165 lines (max allowed is 150). | 414 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 629 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 135). | 27 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'generateOperationalPlan' is not designed for extension - needs to be abstract, final or empty. | 57 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 58 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| whitespace | MethodParamPad | '(' is preceded with whitespace. | 77 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 44 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'duration' must be private and have accessor methods. | 33 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'stepSize' hides a field. | 107 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'stepSize' hides a field. | 114 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 15 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.core.gtu.behavioralcharacteristics.ParameterTypes. | 13 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method length is 214 lines (max allowed is 150). | 91 | |
| sizes | MethodLength | Method length is 175 lines (max allowed is 150). | 319 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 842 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 865 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 950 |
| 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 |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 171 |
| 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. | 1358 | |
| design | DesignForExtension | Method 'removeDirectionality' is not designed for extension - needs to be abstract, final or empty. | 1371 |
| 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 |
|---|---|---|---|---|
| design | DesignForExtension | Method 'clone' is not designed for extension - needs to be abstract, final or empty. | 59 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 82 | |
| blocks | EmptyBlock | Must have at least one statement. | 130 | |
| blocks | EmptyBlock | Must have at least one statement. | 139 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 91 | |
| design | DesignForExtension | Method 'clone' is not designed for extension - needs to be abstract, final or empty. | 98 |
| 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 |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 335 |