Checkstyle Results
The following document contains the results of Checkstyle 12.1.1 with /config/dsol-checks.xml ruleset.
Summary
| Files | |||
|---|---|---|---|
| 17 | 0 | 73 | 0 |
Files
Rules
| Category | Rule | Violations | Severity |
|---|---|---|---|
| blocks | NeedBraces | 26 | |
| coding | HiddenField
|
27 | |
| design | VisibilityModifier | 9 | |
| javadoc | JavadocStyle | 3 | |
| modifier | RedundantModifier | 6 | |
| naming | ConstantName | 2 |
Details
nl/tudelft/simulation/dsol/formalisms/flow/AbstractFlowModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'countMap' must be private and have accessor methods. | 31 | |
| design | VisibilityModifier | Variable 'blockMap' must be private and have accessor methods. | 34 | |
| design | VisibilityModifier | Variable 'blockNamingType' must be private and have accessor methods. | 37 |
nl/tudelft/simulation/dsol/formalisms/flow/Block.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'blockNumberCounter' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 36 |
nl/tudelft/simulation/dsol/formalisms/flow/CapacityRequest.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'amount' must be private and have accessor methods. | 24 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 83 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 112 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 114 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 116 |
nl/tudelft/simulation/dsol/formalisms/flow/CapacityRequestor.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'public' modifier. | 23 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 39 |
nl/tudelft/simulation/dsol/formalisms/flow/Create.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'entitySupplier' hides a field. | 123 | |
| coding | HiddenField | 'startTimeDist' hides a field. | 140 | |
| coding | HiddenField | 'startTime' hides a field. | 161 | |
| coding | HiddenField | 'intervalDist' hides a field. | 180 | |
| coding | HiddenField | 'batchSizeDist' hides a field. | 214 | |
| coding | HiddenField | 'maxNumberCreationEvents' hides a field. | 226 | |
| coding | HiddenField | 'maxNumberGeneratedEntities' hides a field. | 237 | |
| coding | HiddenField | 'endTime' hides a field. | 250 |
nl/tudelft/simulation/dsol/formalisms/flow/Delay.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'delayDistribution' hides a field. | 62 | |
| coding | HiddenField | 'delayFunction' hides a field. | 73 |
nl/tudelft/simulation/dsol/formalisms/flow/Duplicate.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'duplicateDestination' hides a field. | 62 | |
| coding | HiddenField | 'numberCopiesDist' hides a field. | 73 | |
| coding | HiddenField | 'duplicateReleaseFunction' hides a field. | 98 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 116 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 118 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 148 |
nl/tudelft/simulation/dsol/formalisms/flow/Entity.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| naming | ConstantName | Name 'cloner' must match pattern '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'. | 42 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 79 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 92 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 104 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 119 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 151 |
nl/tudelft/simulation/dsol/formalisms/flow/FlowBlock.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'receiveFunction' hides a field. | 88 | |
| coding | HiddenField | 'releaseFunction' hides a field. | 100 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 114 | |
| coding | HiddenField | 'destination' hides a field. | 125 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 168 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 183 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 198 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 230 | |
| javadoc | JavadocStyle | First sentence should end with a period. | 273 |
nl/tudelft/simulation/dsol/formalisms/flow/FlowModel.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| modifier | RedundantModifier | Redundant 'final' modifier. | 33 | |
| modifier | RedundantModifier | Redundant 'public' modifier. | 36 |
nl/tudelft/simulation/dsol/formalisms/flow/Queue.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| blocks | NeedBraces | 'if' construct must use '{}'s. | 202 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 219 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 237 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 239 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 257 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 259 |
nl/tudelft/simulation/dsol/formalisms/flow/Release.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'resource' must be private and have accessor methods. | 70 | |
| coding | HiddenField | 'fixedCapacityRelease' hides a field. | 97 | |
| coding | HiddenField | 'capacityReleaseFunction' hides a field. | 113 | |
| design | VisibilityModifier | Variable 'resource' must be private and have accessor methods. | 168 | |
| coding | HiddenField | 'fixedCapacityRelease' hides a field. | 195 | |
| coding | HiddenField | 'capacityReleaseFunction' hides a field. | 211 |
nl/tudelft/simulation/dsol/formalisms/flow/Resource.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| coding | HiddenField | 'releaseType' hides a field. | 69 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 190 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 266 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 268 | |
| blocks | NeedBraces | 'if' construct must use '{}'s. | 428 | |
| blocks | NeedBraces | 'else' construct must use '{}'s. | 430 | |
| modifier | RedundantModifier | Redundant 'static' modifier. | 464 |
nl/tudelft/simulation/dsol/formalisms/flow/Seize.java
| Severity | Category | Rule | Message | Line |
|---|---|---|---|---|
| design | VisibilityModifier | Variable 'storage' must be private and have accessor methods. | 40 | |
| design | VisibilityModifier | Variable 'resource' must be private and have accessor methods. | 136 | |
| coding | HiddenField | 'fixedCapacityClaim' hides a field. | 163 | |
| coding | HiddenField | 'capacityClaimFunction' hides a field. | 179 | |
| coding | HiddenField | 'resource' hides a field. | 233 | |
| design | VisibilityModifier | Variable 'resource' must be private and have accessor methods. | 261 | |
| coding | HiddenField | 'fixedCapacityClaim' hides a field. | 288 | |
| coding | HiddenField | 'capacityClaimFunction' hides a field. | 304 | |
| coding | HiddenField | 'resource' hides a field. | 358 | |
| modifier | RedundantModifier | Redundant 'static' modifier. | 385 |
