Checkstyle Results
The following document contains the results of Checkstyle 10.17.0 with /config/dsol-checks.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 88 | 6 | 90 | 0 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock | 2 | |
| coding | HiddenField
|
39 | |
| MissingSwitchDefault | 1 | ||
| design | FinalClass | 1 | |
| HideUtilityClassConstructor | 1 | ||
| VisibilityModifier | 16 | ||
| imports | UnusedImports | 1 | |
| misc | TodoComment | 10 | |
| modifier | ModifierOrder | 4 | |
| RedundantModifier | 13 | ||
| sizes | FileLength | 2 | |
| MethodLength | 5 | ||
| ParameterNumber | 1 |
Details
org/opentrafficsim/editor/ApplicationStore.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'file' hides a field. | 119 | |
| coding | HiddenField | 'file' hides a field. | 141 |
org/opentrafficsim/editor/EvalWrapper.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 318 |
org/opentrafficsim/editor/NodeActions.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'editor' hides a field. | 72 |
org/opentrafficsim/editor/OtsEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | FileLength | File length is 2,058 lines (max allowed is 2,000). | 1 | |
| sizes | MethodLength | Method OtsEditor length is 152 lines (max allowed is 150). | 306 | |
| coding | HiddenField | 'scenario' hides a field. | 499 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 516 | |
| coding | HiddenField | 'properties' hides a field. | 665 | |
| coding | HiddenField | 'coupledNode' hides a field. | 803 | |
| coding | HiddenField | 'backNode' hides a field. | 803 | |
| coding | HiddenField | 'backAttribute' hides a field. | 803 | |
| coding | HiddenField | 'clipboard' hides a field. | 1959 | |
| coding | HiddenField | 'cut' hides a field. | 1959 |
org/opentrafficsim/editor/OtsRunner.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 46 |
org/opentrafficsim/editor/Schema.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'elements' hides a field. | 597 |
org/opentrafficsim/editor/Undo.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'ignoreChanges' must be private and have accessor methods. | 54 | |
| sizes | MethodLength | Method notify length is 170 lines (max allowed is 150). | 231 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 461 | |
| design | VisibilityModifier | Variable 'type' must be private and have accessor methods. | 499 | |
| design | VisibilityModifier | Variable 'subActions' must be private and have accessor methods. | 502 | |
| design | VisibilityModifier | Variable 'node' must be private and have accessor methods. | 505 | |
| design | VisibilityModifier | Variable 'parent' must be private and have accessor methods. | 508 | |
| design | VisibilityModifier | Variable 'attribute' must be private and have accessor methods. | 511 | |
| design | VisibilityModifier | Variable 'postActionShowNode' must be private and have accessor methods. | 514 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 524 |
org/opentrafficsim/editor/XiIncludeNode.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 297 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 541 | |
| coding | HiddenField | 'name' hides a field. | 771 | |
| coding | HiddenField | 'name' hides a field. | 787 |
org/opentrafficsim/editor/XsdPaths.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 13 |
org/opentrafficsim/editor/XsdTreeNode.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | FileLength | File length is 2,752 lines (max allowed is 2,000). | 1 | |
| coding | HiddenField | 'children' hides a field. | 354 | |
| coding | HiddenField | 'parent' hides a field. | 1468 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 1641 | |
| coding | HiddenField | 'stringFunction' hides a field. | 2141 | |
| coding | HiddenField | 'value' hides a field. | 2380 | |
| coding | HiddenField | 'maxOccurs' hides a field. | 2521 |
org/opentrafficsim/editor/decoration/AbstractNodeDecoratorAttribute.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'predicate' must be private and have accessor methods. | 28 | |
| design | VisibilityModifier | Variable 'attributes' must be private and have accessor methods. | 31 |
org/opentrafficsim/editor/decoration/AutomaticLinkId.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'lastNode' must be private and have accessor methods. | 27 | |
| design | VisibilityModifier | Variable 'lastId' must be private and have accessor methods. | 30 |
org/opentrafficsim/editor/decoration/DefaultDecorator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 191 |
org/opentrafficsim/editor/decoration/string/AbstractStringFunction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'overwrite' must be private and have accessor methods. | 28 |
org/opentrafficsim/editor/decoration/string/XiIncludeStringFunction.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| imports | UnusedImports | Unused import - org.opentrafficsim.editor.XsdTreeNodeRoot. | 8 |
org/opentrafficsim/editor/decoration/validation/CoupledValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 28 |
org/opentrafficsim/editor/decoration/validation/KeyValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'nodes' hides a field. | 160 |
org/opentrafficsim/editor/decoration/validation/NoDuplicateChildrenValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 123 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 123 |
org/opentrafficsim/editor/decoration/validation/ParentValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'contextPath' hides a field. | 63 | |
| coding | HiddenField | 'idAttribute' hides a field. | 74 | |
| coding | HiddenField | 'parentAttribute' hides a field. | 85 |
org/opentrafficsim/editor/decoration/validation/RoadLayoutElementValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 226 | |
| coding | HiddenField | 'attribute' hides a field. | 375 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 397 | |
| modifier | RedundantModifier | Redundant 'static' modifier. | 426 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 465 | |
| coding | HiddenField | 'attribute' hides a field. | 494 |
org/opentrafficsim/editor/decoration/validation/XPathValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'keyNode' must be private and have accessor methods. | 25 | |
| design | VisibilityModifier | Variable 'keyPath' must be private and have accessor methods. | 28 | |
| design | VisibilityModifier | Variable 'fields' must be private and have accessor methods. | 31 | |
| coding | HiddenField | 'fields' hides a field. | 44 | |
| modifier | ModifierOrder | 'public' modifier out of order with the JLS suggestions. | 165 | |
| modifier | ModifierOrder | 'public' modifier out of order with the JLS suggestions. | 172 |
org/opentrafficsim/editor/decoration/validation/XsdAllValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 93 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 93 |
org/opentrafficsim/editor/extensions/OdEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 60 |
org/opentrafficsim/editor/extensions/RoadLayoutEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 64 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 84 |
org/opentrafficsim/editor/extensions/RouteEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 59 |
org/opentrafficsim/editor/extensions/TrafCodEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 45 |
org/opentrafficsim/editor/extensions/map/ChangeListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'node' hides a field. | 102 | |
| coding | HiddenField | 'node' hides a field. | 126 | |
| coding | HiddenField | 'node' hides a field. | 173 |
org/opentrafficsim/editor/extensions/map/EditorMap.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | FinalClass | Class EditorMap should be declared as final. | 86 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 422 | |
| sizes | MethodLength | Method notify length is 188 lines (max allowed is 150). | 523 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 778 |
org/opentrafficsim/editor/extensions/map/MapData.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'node' hides a field. | 109 |
org/opentrafficsim/editor/extensions/map/MapLinkData.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'shape' hides a field. | 213 | |
| blocks | EmptyBlock | Must have at least one statement. | 422 | |
| blocks | EmptyBlock | Must have at least one statement. | 426 | |
| coding | HiddenField | 'laneData' hides a field. | 676 | |
| design | VisibilityModifier | Variable 'coordinates' must be private and have accessor methods. | 1018 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 1095 | |
| coding | HiddenField | 'from' hides a field. | 1301 | |
| coding | HiddenField | 'to' hides a field. | 1301 | |
| coding | HiddenField | 'length' hides a field. | 1308 | |
| coding | HiddenField | 'shape' hides a field. | 1324 | |
| coding | HiddenField | 'weighted' hides a field. | 1325 |
org/opentrafficsim/editor/extensions/map/MapStripeData.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | ParameterNumber | More than 7 parameters (found 9). | 73 |
org/opentrafficsim/editor/extensions/map/RoadLayoutListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method calculateData length is 162 lines (max allowed is 150). | 81 |
org/opentrafficsim/editor/listeners/ChangesListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'scenario' hides a field. | 95 |
org/opentrafficsim/editor/listeners/XsdTreeMouseListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'editor' hides a field. | 65 | |
| sizes | MethodLength | Method addDefaultActions length is 259 lines (max allowed is 150). | 247 |
org/opentrafficsim/editor/listeners/XsdTreeSelectionListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'editor' hides a field. | 108 |
