Checkstyle Results
The following document contains the results of Checkstyle 10.15.0 with /config/dsol-checks.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 86 | 5 | 95 | 0 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | EmptyBlock | 2 | |
| coding | HiddenField
|
40 | |
| MissingSwitchDefault | 1 | ||
| design | FinalClass | 2 | |
| HideUtilityClassConstructor | 2 | ||
| VisibilityModifier | 17 | ||
| imports | UnusedImports | 1 | |
| misc | TodoComment | 9 | |
| modifier | ModifierOrder | 6 | |
| RedundantModifier | 13 | ||
| sizes | FileLength | 2 | |
LineLength
|
1 | ||
| MethodLength | 4 |
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. | 323 |
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,075 lines (max allowed is 2,000). | 1 | |
| sizes | MethodLength | Method OtsEditor length is 153 lines (max allowed is 150). | 306 | |
| coding | HiddenField | 'scenario' hides a field. | 500 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 517 | |
| coding | HiddenField | 'properties' hides a field. | 653 | |
| coding | HiddenField | 'coupledNode' hides a field. | 791 | |
| coding | HiddenField | 'backNode' hides a field. | 791 | |
| coding | HiddenField | 'backAttribute' hides a field. | 791 | |
| coding | HiddenField | 'clipboard' hides a field. | 1974 | |
| coding | HiddenField | 'cut' hides a field. | 1974 |
org/opentrafficsim/editor/Schema.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'elements' hides a field. | 597 | |
| sizes | LineLength | Line is longer than 128 characters (found 133). | 753 |
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). | 228 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 458 | |
| 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. | 335 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 615 | |
| coding | HiddenField | 'name' hides a field. | 883 | |
| coding | HiddenField | 'name' hides a field. | 901 |
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,727 lines (max allowed is 2,000). | 1 | |
| coding | HiddenField | 'children' hides a field. | 353 | |
| coding | HiddenField | 'parent' hides a field. | 1459 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 1632 | |
| coding | HiddenField | 'stringFunction' hides a field. | 2132 | |
| coding | HiddenField | 'value' hides a field. | 2375 | |
| coding | HiddenField | 'maxOccurs' hides a field. | 2517 |
org/opentrafficsim/editor/XsdTreeNodeUtil.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 34 |
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. | 164 |
org/opentrafficsim/editor/decoration/validation/NoDuplicateChildrenValidator.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 126 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 126 |
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. | 229 | |
| coding | HiddenField | 'attribute' hides a field. | 380 | |
| modifier | RedundantModifier | Redundant 'final' modifier. | 402 | |
| modifier | RedundantModifier | Redundant 'static' modifier. | 432 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 471 | |
| coding | HiddenField | 'attribute' hides a field. | 501 |
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. | 94 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 94 |
org/opentrafficsim/editor/extensions/Adapters.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 38 | |
| modifier | ModifierOrder | 'static' modifier out of order with the JLS suggestions. | 42 |
org/opentrafficsim/editor/extensions/OdEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 61 |
org/opentrafficsim/editor/extensions/RoadLayoutEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 65 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 87 |
org/opentrafficsim/editor/extensions/RouteEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 60 |
org/opentrafficsim/editor/extensions/TrafCodEditor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 46 |
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. | 127 | |
| coding | HiddenField | 'node' hides a field. | 174 |
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:'. | 421 | |
| misc | TodoComment | Comment matches to-do format 'TODO:'. | 739 |
org/opentrafficsim/editor/extensions/map/MapCrossSectionData.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'centerLine' must be private and have accessor methods. | 32 |
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 |
|---|---|---|---|---|
| blocks | EmptyBlock | Must have at least one statement. | 385 | |
| blocks | EmptyBlock | Must have at least one statement. | 389 | |
| coding | HiddenField | 'from' hides a field. | 520 | |
| coding | HiddenField | 'to' hides a field. | 523 | |
| coding | HiddenField | 'laneData' hides a field. | 634 | |
| design | FinalClass | Class ShapeListener should be declared as final. | 743 | |
| design | VisibilityModifier | Variable 'coordinates' must be private and have accessor methods. | 776 | |
| coding | MissingSwitchDefault | switch without "default" clause. | 855 | |
| coding | HiddenField | 'from' hides a field. | 1061 | |
| coding | HiddenField | 'to' hides a field. | 1061 | |
| coding | HiddenField | 'length' hides a field. | 1068 | |
| coding | HiddenField | 'shape' hides a field. | 1084 | |
| coding | HiddenField | 'weighted' hides a field. | 1085 |
org/opentrafficsim/editor/extensions/map/RoadLayoutListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| sizes | MethodLength | Method calculateData length is 176 lines (max allowed is 150). | 82 |
org/opentrafficsim/editor/listeners/ChangesListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'scenario' hides a field. | 97 |
org/opentrafficsim/editor/listeners/XsdTreeMouseListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'editor' hides a field. | 66 | |
| sizes | MethodLength | Method addDefaultActions length is 270 lines (max allowed is 150). | 251 |
org/opentrafficsim/editor/listeners/XsdTreeSelectionListener.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'editor' hides a field. | 109 |
