PMD Results
The following document contains the results of PMD 7.17.0.
Violations By Priority
Priority 3
nl/tudelft/simulation/dsol/simulators/ErrorStrategy.java
| Rule |
Violation |
Line |
| UnnecessaryModifier |
Unnecessary modifier 'final' on method 'getDefaultLogLevel': the method is already in a final class |
75 |
nl/tudelft/simulation/dsol/simulators/Simulator.java
nl/tudelft/simulation/dsol/statistics/SimCounter.java
nl/tudelft/simulation/dsol/statistics/SimPersistent.java
nl/tudelft/simulation/dsol/statistics/SimTally.java
Priority 4
nl/tudelft/simulation/dsol/animation/Editable.java
nl/tudelft/simulation/dsol/experiment/Experiment.java
nl/tudelft/simulation/dsol/formalisms/eventscheduling/AbstractSimEvent.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
31 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
43 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MIN_PRIORITY' is already in scope because it is inherited by an enclosing type |
54 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MAX_PRIORITY' is already in scope because it is inherited by an enclosing type |
54 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MIN_PRIORITY' is already in scope because it is inherited by an enclosing type |
57 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MAX_PRIORITY' is already in scope because it is inherited by an enclosing type |
57 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AbstractSimEvent': 'constructorCounter' is already in scope because it is declared in an enclosing type |
60 |
nl/tudelft/simulation/dsol/formalisms/eventscheduling/LambdaSimEvent.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
32 |
nl/tudelft/simulation/dsol/formalisms/eventscheduling/SimEvent.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
52 |
nl/tudelft/simulation/dsol/simulators/DessSimulator.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
70 |
nl/tudelft/simulation/dsol/simulators/DevDessAnimator.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AnimatorInterface': 'UPDATE_ANIMATION_EVENT' is already in scope because it is inherited by an enclosing type |
54 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
97 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
121 |
nl/tudelft/simulation/dsol/simulators/DevDessSimulator.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
87 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
110 |
nl/tudelft/simulation/dsol/simulators/DevsAnimator.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AnimatorInterface': 'DEFAULT_ANIMATION_DELAY' is already in scope because it is inherited by an enclosing type |
33 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AnimatorInterface': 'UPDATE_ANIMATION_EVENT' is already in scope because it is inherited by an enclosing type |
51 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
79 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
100 |
nl/tudelft/simulation/dsol/simulators/DevsRealTimeAnimator.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
141 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
146 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
228 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
237 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
255 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
274 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
282 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
306 |
nl/tudelft/simulation/dsol/simulators/DevsSimulator.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
212 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
240 |
nl/tudelft/simulation/dsol/simulators/Simulator.java
nl/tudelft/simulation/dsol/statistics/SimPersistent.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LocalEventProducer': 'EventProducer::FIRST_POSITION' is already in scope because it is inherited by an enclosing type |
71 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LocalEventProducer': 'EventProducer::LAST_POSITION' is already in scope because it is inherited by an enclosing type |
74 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LocalEventProducer': 'EventProducer::LAST_POSITION' is already in scope because it is inherited by an enclosing type |
78 |
| UselessParentheses |
Useless parentheses around `(T) timedEvent.getTimeStamp()`. |
173 |
nl/tudelft/simulation/dsol/statistics/SimTally.java
nl/tudelft/simulation/jstats/distributions/DistBeta.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.pow(x, this.alpha1 - 1) * Math.pow(1 - x, ...`. |
66 |
nl/tudelft/simulation/jstats/distributions/DistErlang.java
| Rule |
Violation |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'DistErlang': 'GAMMATHRESHOLD' is already in scope because it is declared in an enclosing type |
62 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'DistErlang': 'GAMMATHRESHOLD' is already in scope because it is declared in an enclosing type |
68 |
nl/tudelft/simulation/jstats/distributions/DistExponential.java
nl/tudelft/simulation/jstats/distributions/DistGamma.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.pow(this.scale, -this.shape) * Math.pow(x,...`. |
136 |
nl/tudelft/simulation/jstats/distributions/DistGeometric.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.floor(Math.log(u) / this.lnp)`. |
50 |
nl/tudelft/simulation/jstats/distributions/DistNegBinomial.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.floor(Math.log(u) / this.lnp)`. |
58 |
nl/tudelft/simulation/jstats/distributions/DistNormalTrunc.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.sqrt(2 * Math.PI * Math.pow(this.sigma, 2))`. |
189 |
nl/tudelft/simulation/jstats/distributions/DistPoisson.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.exp(-this.lambda) * Math.pow(this.lambda, ...`. |
62 |
nl/tudelft/simulation/jstats/filters/MaxDiffFilter.java
| Rule |
Violation |
Line |
| UselessParentheses |
Useless parentheses around `Math.abs(value[1] - this.lastReceivedValue)`. |
52 |
nl/tudelft/simulation/jstats/math/ProbMath.java
nl/tudelft/simulation/jstats/ode/integrators/Gill.java
nl/tudelft/simulation/jstats/streams/DX120Generator.java
nl/tudelft/simulation/jstats/streams/MersenneTwister.java
nl/tudelft/simulation/jstats/streams/RandomNumberGenerator.java
Files
nl/tudelft/simulation/dsol/animation/Editable.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'Editable': 'editables' is already in scope because it is declared in an enclosing type |
4 |
42 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'Editable': 'editables' is already in scope because it is declared in an enclosing type |
4 |
80 |
nl/tudelft/simulation/dsol/experiment/Experiment.java
nl/tudelft/simulation/dsol/formalisms/eventscheduling/AbstractSimEvent.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
31 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
43 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MIN_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
54 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MAX_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
54 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MIN_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
57 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::MAX_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
57 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AbstractSimEvent': 'constructorCounter' is already in scope because it is declared in an enclosing type |
4 |
60 |
nl/tudelft/simulation/dsol/formalisms/eventscheduling/LambdaSimEvent.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
32 |
nl/tudelft/simulation/dsol/formalisms/eventscheduling/SimEvent.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimEvent': 'SimEventInterface::NORMAL_PRIORITY' is already in scope because it is inherited by an enclosing type |
4 |
52 |
nl/tudelft/simulation/dsol/simulators/DessSimulator.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
70 |
nl/tudelft/simulation/dsol/simulators/DevDessAnimator.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AnimatorInterface': 'UPDATE_ANIMATION_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
54 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
97 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
121 |
nl/tudelft/simulation/dsol/simulators/DevDessSimulator.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
87 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
110 |
nl/tudelft/simulation/dsol/simulators/DevsAnimator.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AnimatorInterface': 'DEFAULT_ANIMATION_DELAY' is already in scope because it is inherited by an enclosing type |
4 |
33 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'AnimatorInterface': 'UPDATE_ANIMATION_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
51 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
79 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
100 |
nl/tudelft/simulation/dsol/simulators/DevsRealTimeAnimator.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
141 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
146 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
228 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
237 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
255 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
274 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
282 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
306 |
nl/tudelft/simulation/dsol/simulators/DevsSimulator.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
212 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'SimulatorInterface': 'TIME_CHANGED_EVENT' is already in scope because it is inherited by an enclosing type |
4 |
240 |
nl/tudelft/simulation/dsol/simulators/ErrorStrategy.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryModifier |
Unnecessary modifier 'final' on method 'getDefaultLogLevel': the method is already in a final class |
3 |
75 |
nl/tudelft/simulation/dsol/simulators/Simulator.java
nl/tudelft/simulation/dsol/statistics/SimCounter.java
nl/tudelft/simulation/dsol/statistics/SimPersistent.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LocalEventProducer': 'EventProducer::FIRST_POSITION' is already in scope because it is inherited by an enclosing type |
4 |
71 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LocalEventProducer': 'EventProducer::LAST_POSITION' is already in scope because it is inherited by an enclosing type |
4 |
74 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'LocalEventProducer': 'EventProducer::LAST_POSITION' is already in scope because it is inherited by an enclosing type |
4 |
78 |
| UnnecessaryReturn |
Unnecessary return statement |
3 |
154 |
| UselessParentheses |
Useless parentheses around `(T) timedEvent.getTimeStamp()`. |
4 |
173 |
nl/tudelft/simulation/dsol/statistics/SimTally.java
nl/tudelft/simulation/jstats/distributions/DistBeta.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.pow(x, this.alpha1 - 1) * Math.pow(1 - x, ...`. |
4 |
66 |
nl/tudelft/simulation/jstats/distributions/DistErlang.java
| Rule |
Violation |
Priority |
Line |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'DistErlang': 'GAMMATHRESHOLD' is already in scope because it is declared in an enclosing type |
4 |
62 |
| UnnecessaryFullyQualifiedName |
Unnecessary qualifier 'DistErlang': 'GAMMATHRESHOLD' is already in scope because it is declared in an enclosing type |
4 |
68 |
nl/tudelft/simulation/jstats/distributions/DistExponential.java
nl/tudelft/simulation/jstats/distributions/DistGamma.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.pow(this.scale, -this.shape) * Math.pow(x,...`. |
4 |
136 |
nl/tudelft/simulation/jstats/distributions/DistGeometric.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.floor(Math.log(u) / this.lnp)`. |
4 |
50 |
nl/tudelft/simulation/jstats/distributions/DistNegBinomial.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.floor(Math.log(u) / this.lnp)`. |
4 |
58 |
nl/tudelft/simulation/jstats/distributions/DistNormalTrunc.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.sqrt(2 * Math.PI * Math.pow(this.sigma, 2))`. |
4 |
189 |
nl/tudelft/simulation/jstats/distributions/DistPoisson.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.exp(-this.lambda) * Math.pow(this.lambda, ...`. |
4 |
62 |
nl/tudelft/simulation/jstats/filters/MaxDiffFilter.java
| Rule |
Violation |
Priority |
Line |
| UselessParentheses |
Useless parentheses around `Math.abs(value[1] - this.lastReceivedValue)`. |
4 |
52 |
nl/tudelft/simulation/jstats/math/ProbMath.java
nl/tudelft/simulation/jstats/ode/integrators/Gill.java
nl/tudelft/simulation/jstats/streams/DX120Generator.java
nl/tudelft/simulation/jstats/streams/MersenneTwister.java
nl/tudelft/simulation/jstats/streams/RandomNumberGenerator.java