Checkstyle Results
The following document contains the results of Checkstyle 10.21.3 with config/dsol-checks.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 22 | 3 | 57 | 0 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock | 4 | |
| NeedBraces | 24 | ||
| coding | HiddenField
|
4 | |
| design | VisibilityModifier | 6 | |
| javadoc | JavadocMethod | 8 | |
| misc | TodoComment | 4 | |
| modifier | ModifierOrder | 2 | |
| RedundantModifier | 3 | ||
| naming | ParameterName | 2 | |
| sizes | MethodLength | 1 | |
| ParameterNumber | 2 |
Details
org/opentrafficsim/web/OtsWebModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'animationPanel' hides a field. | 246 |
org/opentrafficsim/web/OtsWebServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'webServer' must be private and have accessor methods. | 255 | |
| sizes | MethodLength | Method handle length is 312 lines (max allowed is 150). | 272 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 305 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 320 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 322 | |
| blocks | NeedBraces | 'for' construct must use '{}'s. | 432 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 452 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 465 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 527 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 536 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 538 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 543 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 545 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 570 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 572 |
org/opentrafficsim/web/TestDemoServer.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'sessionModelMap' must be private and have accessor methods. | 67 | |
| design | VisibilityModifier | Variable 'sessionWebModelMap' must be private and have accessor methods. | 70 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 97 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 177 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 179 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 181 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 183 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 393 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 395 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 410 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 412 | |
| blocks | EmptyBlock | Must have at least one statement. | 418 | |
| blocks | EmptyBlock | Must have at least one statement. | 422 | |
| blocks | EmptyBlock | Must have at least one statement. | 426 | |
| blocks | EmptyBlock | Must have at least one statement. | 430 | |
| javadoc | JavadocMethod | Expected @param tag for '<U>'. | 439 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 440 | |
| javadoc | JavadocMethod | Expected @param tag for '<U>'. | 449 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 450 | |
| javadoc | JavadocMethod | Expected @param tag for '<U>'. | 464 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 465 | |
| javadoc | JavadocMethod | Expected @param tag for '<U>'. | 474 | |
| javadoc | JavadocMethod | Expected @param tag for '<T>'. | 475 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 550 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 555 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 568 |
org/opentrafficsim/web/animation/HtmlGraphics2d.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 71 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 77 | |
| naming | ParameterName | Name 'Tx' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 753 | |
| naming | ParameterName | Name 'Tx' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 759 | |
| coding | HiddenField | 'color' hides a field. | 787 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 824 | |
| sizes | ParameterNumber | More than 7 parameters (found 10). | 1024 | |
| sizes | ParameterNumber | More than 7 parameters (found 11). | 1032 |
org/opentrafficsim/web/animation/HtmlGraphicsConfiguration.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'htmlDevice' must be private and have accessor methods. | 24 | |
| design | VisibilityModifier | Variable 'identityTransform' must be private and have accessor methods. | 27 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 29 | |
| design | VisibilityModifier | Variable 'bounds' must be private and have accessor methods. | 30 | |
| coding | HiddenField | 'htmlDevice' hides a field. | 52 |
org/opentrafficsim/web/animation/WebAnimationToggles.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 108 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 108 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 118 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 118 |
org/opentrafficsim/web/animation/d2/HtmlGridPanel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'extent' hides a field. | 652 |
