The following document contains the results of Checkstyle 6.2 with development/dsol-checks.xml ruleset. 
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | AvoidNestedBlocks
|
2 | |
| EmptyBlock | 5 | ||
| NeedBraces | 8 | ||
| design | DesignForExtension | 43 | |
| HideUtilityClassConstructor | 1 | ||
| VisibilityModifier | 5 | ||
| imports | UnusedImports | 12 | |
| javadoc | JavadocMethod | 1 | |
| JavadocVariable | 4 | ||
| misc | FinalParameters | 19 | |
NewlineAtEndOfFile
|
6 | ||
| TodoComment | 7 | ||
| modifier | RedundantModifier | 2 | |
| sizes | LineLength
|
10 | |
| MethodLength | 9 | ||
| ParameterNumber | 8 | ||
| 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 |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 256 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 313 | |
| blocks | AvoidNestedBlocks | Avoid nested blocks. | 340 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 373 | |
| blocks | AvoidNestedBlocks | Avoid nested blocks. | 400 |
| 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. | 80 | |
| design | VisibilityModifier | Variable 'initialLongitudinalPositions' must be private and have accessor methods. | 83 | |
| design | VisibilityModifier | Variable 'roomChecker' must be private and have accessor methods. | 86 | |
| design | VisibilityModifier | Variable 'gtuColorer' must be private and have accessor methods. | 89 | |
| sizes | ParameterNumber | More than 7 parameters (found 10). | 110 | |
| misc | FinalParameters | Parameter id should be final. | 110 | |
| misc | FinalParameters | Parameter roomChecker should be final. | 113 | |
| design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 286 | |
| design | DesignForExtension | Method 'getGeneratedGTUs' is not designed for extension - needs to be abstract, final or empty. | 294 | |
| design | DesignForExtension | Method 'setGeneratedGTUs' is not designed for extension - needs to be abstract, final or empty. | 302 | |
| misc | FinalParameters | Parameter generatedGTUs should be final. | 302 | |
| design | DesignForExtension | Method 'getId' is not designed for extension - needs to be abstract, final or empty. | 311 | |
| design | DesignForExtension | Method 'getEndTime' is not designed for extension - needs to be abstract, final or empty. | 320 | |
| design | DesignForExtension | Method 'getMaxGTUs' is not designed for extension - needs to be abstract, final or empty. | 329 | |
| design | DesignForExtension | Method 'getGtuColorer' is not designed for extension - needs to be abstract, final or empty. | 338 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 360 |
| 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). | 593 | |
| blocks | EmptyBlock | Must have at least one statement. | 960 | |
| blocks | EmptyBlock | Must have at least one statement. | 975 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.core.network.route.RouteGenerator. | 8 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 491 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.simulationengine.SimpleSimulator. | 18 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.road.network.lane.Lane. | 19 |
| 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 | MethodLength | Method length is 211 lines (max allowed is 150). | 109 |
| 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). | 200 | |
| sizes | LineLength | Line is longer than 128 characters (found 131). | 797 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.djunits.unit.LengthUnit. | 11 | |
| sizes | MethodLength | Method length is 193 lines (max allowed is 150). | 264 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 235 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 381 | |
| sizes | MethodLength | Method length is 168 lines (max allowed is 150). | 443 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 661 |
| 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 | MethodLength | Method length is 169 lines (max allowed is 150). | 505 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method length is 166 lines (max allowed is 150). | 158 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.core.gtu.behavioralcharacteristics.BehavioralCharacteristics. | 4 |
| 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 | ParameterNumber | More than 7 parameters (found 9). | 180 |
| 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 |
|---|---|---|---|---|
| misc | FinalParameters | Parameter position should be final. | 986 | |
| design | DesignForExtension | Method 'addDirectionality' is not designed for extension - needs to be abstract, final or empty. | 1493 | |
| design | DesignForExtension | Method 'removeDirectionality' is not designed for extension - needs to be abstract, final or empty. | 1506 |
| 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 |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.djunits.unit.LengthUnit. | 8 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | NewlineAtEndOfFile | File does not end with a newline. |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.core.gtu.GTUDirectionality. | 6 |
| 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 | FinalParameters | Parameter dummy1 should be final. | 63 | |
| misc | FinalParameters | Parameter dummy2 should be final. | 63 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method length is 188 lines (max allowed is 150). | 254 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 329 | |
| sizes | LineLength | Line is longer than 128 characters (found 129). | 604 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.core.gtu.behavioralcharacteristics.ParameterTypes. | 19 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.djunits.unit.SpeedUnit. | 3 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.djunits.value.ValueException. | 8 | |
| imports | UnusedImports | Unused import - org.djunits.value.vfloat.vector.FloatLengthVector. | 12 |
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | LineLength | Line is longer than 128 characters (found 131). | 335 |