Checkstyle Results
The following document contains the results of Checkstyle 10.17.0 with /config/dsol-checks.xml ruleset.
Summary
Files | Info | Warnings | Errors |
---|---|---|---|
21 | 1 | 46 | 0 |
Files
Rules
Category | Rule | Violations | Severity |
---|---|---|---|
blocks | NeedBraces | 11 | Warning |
design | DesignForExtension | 1 | Warning |
HideUtilityClassConstructor | 1 | Warning | |
VisibilityModifier | 8 | Warning | |
misc | TodoComment | 2 | Warning |
modifier | RedundantModifier | 22 | Warning |
naming | StaticVariableName | 1 | Warning |
sizes | LineLength
|
1 | Info |
Details
nl/tudelft/simulation/naming/context/ContextFactory.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 31 |
nl/tudelft/simulation/naming/context/ContextInterface.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | modifier | RedundantModifier | Redundant 'public' modifier. | 50 |
Warning | modifier | RedundantModifier | Redundant 'static' modifier. | 50 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 50 |
Warning | modifier | RedundantModifier | Redundant 'public' modifier. | 53 |
Warning | modifier | RedundantModifier | Redundant 'static' modifier. | 53 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 53 |
Warning | modifier | RedundantModifier | Redundant 'public' modifier. | 56 |
Warning | modifier | RedundantModifier | Redundant 'static' modifier. | 56 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 56 |
Warning | modifier | RedundantModifier | Redundant 'public' modifier. | 64 |
Warning | modifier | RedundantModifier | Redundant 'static' modifier. | 64 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 64 |
Warning | modifier | RedundantModifier | Redundant 'public' modifier. | 75 |
Warning | modifier | RedundantModifier | Redundant 'static' modifier. | 75 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 75 |
Warning | modifier | RedundantModifier | Redundant 'public' modifier. | 87 |
Warning | modifier | RedundantModifier | Redundant 'static' modifier. | 87 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 87 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 352 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 366 |
Warning | modifier | RedundantModifier | Redundant 'final' modifier. | 415 |
nl/tudelft/simulation/naming/context/FileContext.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | misc | TodoComment | Comment matches to-do format 'TODO:'. | 64 |
nl/tudelft/simulation/naming/context/JvmContext.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | design | VisibilityModifier | Variable 'parent' must be private and have accessor methods. | 39 |
Warning | design | VisibilityModifier | Variable 'elements' must be private and have accessor methods. | 48 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 465 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 628 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 630 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 632 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 637 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 640 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 644 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 647 |
nl/tudelft/simulation/naming/context/RemoteContext.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | design | VisibilityModifier | Variable 'embeddedContext' must be private and have accessor methods. | 47 |
Warning | design | VisibilityModifier | Variable 'remoteEventProducer' must be private and have accessor methods. | 50 |
Warning | misc | TodoComment | Comment matches to-do format 'TODO:'. | 272 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 322 |
Info | sizes | LineLength | Line is longer than 128 characters (found 143). | 359 |
Warning | design | DesignForExtension | Class 'RemoteContext' looks like designed for extension (can be subclassed), but the method 'getEventListenerMap' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'RemoteContext' final or making the method 'getEventListenerMap' static/final/abstract/empty, or adding allowed annotation for the method. | 400 |
nl/tudelft/simulation/naming/context/event/InitialEventContext.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | naming | StaticVariableName | Name 'INSTANCE' must match pattern '^[a-z][a-zA-Z0-9]*$'. | 89 |
Warning | design | VisibilityModifier | Variable 'properties' must be private and have accessor methods. | 92 |
Warning | design | VisibilityModifier | Variable 'defaultInitCtx' must be private and have accessor methods. | 98 |
Warning | design | VisibilityModifier | Variable 'gotDefault' must be private and have accessor methods. | 101 |
Warning | design | VisibilityModifier | Variable 'contextEventProducerImpl' must be private and have accessor methods. | 104 |
nl/tudelft/simulation/naming/context/util/ContextUtil.java
Severity | Category | Rule | Message | Line |
---|---|---|---|---|
Warning | design | HideUtilityClassConstructor | Utility classes should not have a public or default constructor. | 25 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 44 |
Warning | blocks | NeedBraces | 'if' construct must use '{}'s. | 72 |