Checkstyle Results

The following document contains the results of Checkstyle 10.15.0 with /config/dsol-checks.xml ruleset.

Summary

Files  Info  Warnings  Errors
21 1 46 0

Files

File  I  W  E
nl/tudelft/simulation/naming/context/ContextFactory.java 0 1 0
nl/tudelft/simulation/naming/context/ContextInterface.java 0 21 0
nl/tudelft/simulation/naming/context/FileContext.java 0 1 0
nl/tudelft/simulation/naming/context/JvmContext.java 0 10 0
nl/tudelft/simulation/naming/context/RemoteContext.java 1 5 0
nl/tudelft/simulation/naming/context/event/InitialEventContext.java 0 5 0
nl/tudelft/simulation/naming/context/util/ContextUtil.java 0 3 0

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
  • max: "128"
 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