The following document contains the results of Checkstyle 6.18 with development/dsol-checks.xml ruleset.
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | EmptyBlock | 2 | ![]() |
NeedBraces | 15 | ![]() | |
coding | InnerAssignment | 2 | ![]() |
design | DesignForExtension | 6 | ![]() |
HideUtilityClassConstructor | 3 | ![]() | |
VisibilityModifier | 27 | ![]() | |
javadoc | JavadocStyle
|
1 | ![]() |
misc | FinalParameters | 13 | ![]() |
TodoComment | 29 | ![]() | |
modifier | ModifierOrder | 2 | ![]() |
sizes | LineLength
|
8 | ![]() |
MethodLength | 3 | ![]() |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | FinalParameters | Parameter message should be final. | 33 |
![]() |
misc | FinalParameters | Parameter cause should be final. | 33 |
![]() |
misc | FinalParameters | Parameter message should be final. | 41 |
![]() |
misc | FinalParameters | Parameter cause should be final. | 49 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'factor' must be private and have accessor methods. | 47 |
![]() |
design | DesignForExtension | Method 'getCategory' is not designed for extension - needs to be abstract, final or empty. | 173 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 44 |
![]() |
design | VisibilityModifier | Variable 'origin' must be private and have accessor methods. | 47 |
![]() |
design | VisibilityModifier | Variable 'destination' must be private and have accessor methods. | 50 |
![]() |
design | VisibilityModifier | Variable 'interpolation' must be private and have accessor methods. | 53 |
![]() |
design | VisibilityModifier | Variable 'category' must be private and have accessor methods. | 56 |
![]() |
design | VisibilityModifier | Variable 'categoryName' must be private and have accessor methods. | 59 |
![]() |
design | VisibilityModifier | Variable 'demandType' must be private and have accessor methods. | 62 |
![]() |
design | VisibilityModifier | Variable 'timeVector' must be private and have accessor methods. | 65 |
![]() |
design | VisibilityModifier | Variable 'demandVector' must be private and have accessor methods. | 68 |
![]() |
design | VisibilityModifier | Variable 'factor' must be private and have accessor methods. | 71 |
![]() |
design | VisibilityModifier | Variable 'factors' must be private and have accessor methods. | 74 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'network' must be private and have accessor methods. | 71 |
![]() |
design | VisibilityModifier | Variable 'categorization' must be private and have accessor methods. | 77 |
![]() |
design | VisibilityModifier | Variable 'categories' must be private and have accessor methods. | 80 |
![]() |
design | VisibilityModifier | Variable 'globalTime' must be private and have accessor methods. | 83 |
![]() |
design | VisibilityModifier | Variable 'globalInterpolation' must be private and have accessor methods. | 86 |
![]() |
design | VisibilityModifier | Variable 'demand' must be private and have accessor methods. | 89 |
![]() |
design | DesignForExtension | Method 'getGTUType' is not designed for extension - needs to be abstract, final or empty. | 116 |
![]() |
design | DesignForExtension | Method 'apply' is not designed for extension - needs to be abstract, final or empty. | 138 |
![]() |
design | DesignForExtension | Method 'apply' is not designed for extension - needs to be abstract, final or empty. | 187 |
![]() |
sizes | MethodLength | Method length is 172 lines (max allowed is 150). | 255 |
![]() |
modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 435 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 61 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 69 |
![]() |
sizes | MethodLength | Method length is 159 lines (max allowed is 150). | 86 |
![]() |
misc | FinalParameters | Parameter nodeDirections should be final. | 86 |
![]() |
misc | FinalParameters | Parameter simulator should be final. | 87 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 174 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 208 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 215 |
![]() |
blocks | EmptyBlock | Must have at least one statement. | 219 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 220 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 221 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 232 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 243 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 244 |
![]() |
misc | FinalParameters | Parameter simulator should be final. | 257 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 265 |
![]() |
misc | FinalParameters | Parameter t should be final. | 274 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 284 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 298 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 299 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 305 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 311 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 318 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 325 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 336 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 359 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 364 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 367 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 372 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 398 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 403 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 429 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 430 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 436 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 437 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 444 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 445 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 452 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 453 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 459 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 460 |
![]() |
misc | FinalParameters | Parameter ocStr should be final. | 474 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
![]() |
blocks | NeedBraces | 'for' construct must use '{}'s. | 41 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 34 |
![]() |
blocks | NeedBraces | 'else' construct must use '{}'s. | 36 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 44 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 131). | 89 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'name' must be private and have accessor methods. | 30 |
![]() |
design | VisibilityModifier | Variable 'nodeStartTag' must be private and have accessor methods. | 33 |
![]() |
design | VisibilityModifier | Variable 'nodeEndTag' must be private and have accessor methods. | 36 |
![]() |
design | VisibilityModifier | Variable 'demandWeight' must be private and have accessor methods. | 39 |
![]() |
design | VisibilityModifier | Variable 'connector' must be private and have accessor methods. | 42 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 482 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 485 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 488 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
javadoc | JavadocStyle | First sentence should end with a period. | 31 |
![]() |
design | VisibilityModifier | Variable 'defaultColorer' must be private and have accessor methods. | 33 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 69 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 84 |
![]() |
blocks | NeedBraces | 'if' construct must use '{}'s. | 97 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'gtuDirection' must be private and have accessor methods. | 77 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 130). | 294 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 128 characters (found 131). | 80 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'priority' must be private and have accessor methods. | 127 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
coding | InnerAssignment | Inner assignments should be avoided. | 188 |
![]() |
coding | InnerAssignment | Inner assignments should be avoided. | 192 |
![]() |
misc | TodoComment | Comment matches to-do format 'TODO:'. | 253 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'coordinates' must be private and have accessor methods. | 33 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 128 characters (found 131). | 150 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 131). | 168 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 128 characters (found 131). | 87 |
![]() |
sizes | LineLength | Line is longer than 128 characters (found 129). | 105 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
sizes | LineLength | Line is longer than 128 characters (found 130). | 47 |
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
![]() |
design | VisibilityModifier | Variable 'xmlComments' must be private and have accessor methods. | 152 |
![]() |
misc | FinalParameters | Parameter interpretXMLComments should be final. | 195 |
![]() |
misc | FinalParameters | Parameter interpretXMLComments should be final. | 219 |
![]() |
misc | FinalParameters | Parameter interpretXMLComments should be final. | 244 |
![]() |
misc | FinalParameters | Parameter interpretXMLComments should be final. | 269 |
![]() |
blocks | EmptyBlock | Must have at least one statement. | 360 |
![]() |
sizes | MethodLength | Method length is 161 lines (max allowed is 150). | 406 |
![]() |
design | DesignForExtension | Method 'getXMLComments' is not designed for extension - needs to be abstract, final or empty. | 618 |
![]() |
design | DesignForExtension | Method 'toString' is not designed for extension - needs to be abstract, final or empty. | 632 |