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 |