PMD Results
The following document contains the results of PMD 7.7.0.
Violations By Priority
Priority 3
org/opentrafficsim/swing/graphs/SwingContourPlot.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 78 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 80 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 86 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 89 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 121 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 124 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 130 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 138 |
org/opentrafficsim/swing/graphs/SwingPlot.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 141–144 |
org/opentrafficsim/swing/graphs/SwingTrajectoryPlot.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 136 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 325 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 327 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 329 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 331 |
org/opentrafficsim/swing/gui/AnimationToggles.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 235 |
org/opentrafficsim/swing/gui/Appearance.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getName': the method is already in a final class | 82 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBackground': the method is already in a final class | 91 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getForeground': the method is already in a final class | 100 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBackdrop': the method is already in a final class | 109 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getFont': the method is already in a final class | 118 |
org/opentrafficsim/swing/gui/AppearanceApplication.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 129–132 |
org/opentrafficsim/swing/gui/OtsGtuColorPanel.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 88 |
org/opentrafficsim/swing/gui/OtsSimulationControlPanel.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'otsAnimationPanel'. | 129 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 277 |
| EmptyCatchBlock | Avoid empty catch blocks | 354–357 |
| UnnecessaryReturn | Unnecessary return statement | 673 |
org/opentrafficsim/swing/gui/OtsSimulationPanel.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 480 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 483 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 566 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1121–1124 |
org/opentrafficsim/swing/gui/PropertiesStore.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 109–112 |
| EmptyCatchBlock | Avoid empty catch blocks | 154–157 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 317 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 347 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 348 |
org/opentrafficsim/swing/script/AbstractSimulationScript.java
| Rule | Violation | Line |
|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 227 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 236 |
Priority 4
org/opentrafficsim/swing/graphs/SwingFundamentalDiagram.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 86 |
| UselessParentheses | Useless parentheses. | 202 |
org/opentrafficsim/swing/gui/AppearanceApplication.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AppearanceApplication': 'setDefaultFont' is already in scope | 119 |
org/opentrafficsim/swing/gui/OtsSimulationApplication.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Frame': 'MAXIMIZED_BOTH' is already in scope because it is inherited by an enclosing type | 36 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'WindowConstants': 'EXIT_ON_CLOSE' is already in scope because it is inherited by an enclosing type | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'WindowConstants': 'DISPOSE_ON_CLOSE' is already in scope because it is inherited by an enclosing type | 54 |
org/opentrafficsim/swing/gui/OtsSimulationControlPanel.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Thread': 'sleep' is already in scope | 352 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'RIGHT' is already in scope because it is inherited by an enclosing type | 765 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ClockLabel': 'UPDATEINTERVAL' is already in scope because it is declared in an enclosing type | 768 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'RIGHT' is already in scope because it is inherited by an enclosing type | 974 |
org/opentrafficsim/swing/gui/OtsSimulationPanel.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 296 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 629 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 693 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 740 |
Files
org/opentrafficsim/swing/graphs/SwingContourPlot.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 78 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 80 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 86 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 89 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 121 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 124 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 130 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 138 |
org/opentrafficsim/swing/graphs/SwingFundamentalDiagram.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 86 |
| UselessParentheses | Useless parentheses. | 4 | 202 |
org/opentrafficsim/swing/graphs/SwingPlot.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 141–144 |
org/opentrafficsim/swing/graphs/SwingTrajectoryPlot.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 136 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 325 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 327 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 329 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 331 |
org/opentrafficsim/swing/gui/AnimationToggles.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 235 |
org/opentrafficsim/swing/gui/Appearance.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getName': the method is already in a final class | 3 | 82 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBackground': the method is already in a final class | 3 | 91 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getForeground': the method is already in a final class | 3 | 100 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getBackdrop': the method is already in a final class | 3 | 109 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getFont': the method is already in a final class | 3 | 118 |
org/opentrafficsim/swing/gui/AppearanceApplication.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AppearanceApplication': 'setDefaultFont' is already in scope | 4 | 119 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 129–132 |
org/opentrafficsim/swing/gui/OtsGtuColorPanel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 88 |
org/opentrafficsim/swing/gui/OtsSimulationApplication.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Frame': 'MAXIMIZED_BOTH' is already in scope because it is inherited by an enclosing type | 4 | 36 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'WindowConstants': 'EXIT_ON_CLOSE' is already in scope because it is inherited by an enclosing type | 4 | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'WindowConstants': 'DISPOSE_ON_CLOSE' is already in scope because it is inherited by an enclosing type | 4 | 54 |
org/opentrafficsim/swing/gui/OtsSimulationControlPanel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'otsAnimationPanel'. | 3 | 129 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 277 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Thread': 'sleep' is already in scope | 4 | 352 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 354–357 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 673 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'RIGHT' is already in scope because it is inherited by an enclosing type | 4 | 765 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ClockLabel': 'UPDATEINTERVAL' is already in scope because it is declared in an enclosing type | 4 | 768 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'SwingConstants': 'RIGHT' is already in scope because it is inherited by an enclosing type | 4 | 974 |
org/opentrafficsim/swing/gui/OtsSimulationPanel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 4 | 296 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 480 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 483 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 566 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 4 | 629 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 4 | 693 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Component': 'LEFT_ALIGNMENT' is already in scope because it is inherited by an enclosing type | 4 | 740 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1121–1124 |
org/opentrafficsim/swing/gui/PropertiesStore.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 109–112 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 154–157 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 317 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 347 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 348 |
org/opentrafficsim/swing/script/AbstractSimulationScript.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 227 |
| UselessQualifiedThis | Useless qualified this usage in the same class. | 3 | 236 |