Checkstyle Results

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

rss feed

Summary

Files
 Info
 Warnings
 Errors
248 11 33 0

Files

File
 I
 W
 E
org/opentrafficsim/core/animation/Drawable.java 1 0 0
org/opentrafficsim/core/dsol/OTSDEVSRTParallelMove.java 1 1 0
org/opentrafficsim/core/geometry/DirectedPoint.java 0 3 0
org/opentrafficsim/core/geometry/OTSBufferingJTS.java 1 0 0
org/opentrafficsim/core/geometry/OTSLine3D.java 2 0 0
org/opentrafficsim/core/geometry/OTSOffsetLinePK.java 0 1 0
org/opentrafficsim/core/gtu/AbstractGTU.java 0 8 0
org/opentrafficsim/core/gtu/GTUDumper.java 1 0 0
org/opentrafficsim/core/gtu/perception/DirectEgoPerception.java 0 14 0
org/opentrafficsim/core/gtu/plan/operational/OperationalPlanBuilder.java 1 1 0
org/opentrafficsim/core/network/Network.java 0 3 0
org/opentrafficsim/core/network/OTSNetwork.java 0 2 0
org/opentrafficsim/core/perception/collections/HistoricalHashMap.java 1 0 0
org/opentrafficsim/core/perception/collections/HistoricalHashSet.java 1 0 0
org/opentrafficsim/core/units/distributions/ContinuousDistFloatScalar.java 2 0 0

Rules

Category Rule Violations Severity
blocks EmptyBlock 1
 Warning
NeedBraces 11
 Warning
design DesignForExtension 5
 Warning
VisibilityModifier 3
 Warning
javadoc JavadocPackage 1
 Info
misc FinalParameters 1
 Warning
TodoComment 10
 Warning
naming StaticVariableName 2
 Warning
sizes LineLength
  • max: "128"
8
 Info
MethodLength 2
 Info

Details

org/opentrafficsim/core/animation/Drawable.java

Severity Category Rule Message Line
 Info
javadoc JavadocPackage Missing package-info.java file. 1

org/opentrafficsim/core/dsol/OTSDEVSRTParallelMove.java

Severity Category Rule Message Line
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 91
 Info
sizes MethodLength Method length is 336 lines (max allowed is 150). 93

org/opentrafficsim/core/geometry/DirectedPoint.java

Severity Category Rule Message Line
 Warning
design DesignForExtension Class 'DirectedPoint' looks like designed for extension (can be subclassed), but the method 'getRotX' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectedPoint' final or making the method 'getRotX' static/final/abstract/empty, or adding allowed annotation for the method. 58
 Warning
design DesignForExtension Class 'DirectedPoint' looks like designed for extension (can be subclassed), but the method 'getRotY' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectedPoint' final or making the method 'getRotY' static/final/abstract/empty, or adding allowed annotation for the method. 63
 Warning
design DesignForExtension Class 'DirectedPoint' looks like designed for extension (can be subclassed), but the method 'getRotZ' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectedPoint' final or making the method 'getRotZ' static/final/abstract/empty, or adding allowed annotation for the method. 68

org/opentrafficsim/core/geometry/OTSBufferingJTS.java

Severity Category Rule Message Line
 Info
sizes MethodLength Method length is 171 lines (max allowed is 150). 88

org/opentrafficsim/core/geometry/OTSLine3D.java

Severity Category Rule Message Line
 Info
sizes LineLength Line is longer than 128 characters (found 131). 377
 Info
sizes LineLength Line is longer than 128 characters (found 129). 378

org/opentrafficsim/core/geometry/OTSOffsetLinePK.java

Severity Category Rule Message Line
 Warning
blocks EmptyBlock Must have at least one statement. 258

org/opentrafficsim/core/gtu/AbstractGTU.java

Severity Category Rule Message Line
 Warning
design VisibilityModifier Variable 'operationalPlan' must be private and have accessor methods. 92
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 104
 Warning
naming StaticVariableName Name 'ALIGNED' must match pattern '^[a-z][a-zA-Z0-9]*$'. 105
 Warning
design VisibilityModifier Variable 'ALIGNED' must be private and have accessor methods. 105
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 108
 Warning
naming StaticVariableName Name 'ALIGN_COUNT' must match pattern '^[a-z][a-zA-Z0-9]*$'. 109
 Warning
design VisibilityModifier Variable 'ALIGN_COUNT' must be private and have accessor methods. 109
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 739

org/opentrafficsim/core/gtu/GTUDumper.java

Severity Category Rule Message Line
 Info
sizes LineLength Line is longer than 128 characters (found 134). 95

org/opentrafficsim/core/gtu/perception/DirectEgoPerception.java

Severity Category Rule Message Line
 Warning
design DesignForExtension Class 'DirectEgoPerception' looks like designed for extension (can be subclassed), but the method 'hashCode' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectEgoPerception' final or making the method 'hashCode' static/final/abstract/empty, or adding allowed annotation for the method. 176
 Warning
design DesignForExtension Class 'DirectEgoPerception' looks like designed for extension (can be subclassed), but the method 'equals' does not have javadoc that explains how to do that safely. If class is not designed for extension consider making the class 'DirectEgoPerception' final or making the method 'equals' static/final/abstract/empty, or adding allowed annotation for the method. 188
 Warning
misc FinalParameters Parameter obj should be final. 189
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 191
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 193
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 195
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 200
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 203
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 207
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 210
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 214
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 217
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 221
 Warning
blocks NeedBraces 'if' construct must use '{}'s. 224

org/opentrafficsim/core/gtu/plan/operational/OperationalPlanBuilder.java

Severity Category Rule Message Line
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 69
 Info
sizes LineLength Line is longer than 128 characters (found 129). 213

org/opentrafficsim/core/network/Network.java

Severity Category Rule Message Line
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 452
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 505
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 514

org/opentrafficsim/core/network/OTSNetwork.java

Severity Category Rule Message Line
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 688
 Warning
misc TodoComment Comment matches to-do format 'TODO:'. 690

org/opentrafficsim/core/perception/collections/HistoricalHashMap.java

Severity Category Rule Message Line
 Info
sizes LineLength Line is longer than 128 characters (found 132). 10

org/opentrafficsim/core/perception/collections/HistoricalHashSet.java

Severity Category Rule Message Line
 Info
sizes LineLength Line is longer than 128 characters (found 132). 10

org/opentrafficsim/core/units/distributions/ContinuousDistFloatScalar.java

Severity Category Rule Message Line
 Info
sizes LineLength Line is longer than 128 characters (found 132). 207
 Info
sizes LineLength Line is longer than 128 characters (found 134). 210