PMD Results
The following document contains the results of PMD 7.3.0.
Violations By Priority
Priority 3
nl/tudelft/simulation/dsol/jetty/sse/OtsWebModel.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 57 |
| UnusedLocalVariable | Avoid unused local variables such as 'params'. | 242 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 265 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 286 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 383–386 |
nl/tudelft/simulation/dsol/jetty/sse/OtsWebServer.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 64 |
| UnusedLocalVariable | Avoid unused local variables such as 'params'. | 275 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 298 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 319 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 416–419 |
nl/tudelft/simulation/dsol/jetty/sse/TestDemoServer.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 400–415 |
| EmptyControlStatement | Empty if statement | 404–415 |
| EmptyControlStatement | Empty if statement | 408–415 |
| EmptyControlStatement | Empty if statement | 412–415 |
nl/tudelft/simulation/dsol/web/animation/d2/HtmlAnimationPanel.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 171 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 195–198 |
| EmptyCatchBlock | Avoid empty catch blocks | 316–319 |
org/opentrafficsim/web/animation/WebAnimationToggles.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'showAnimationClass': the method is already in a final class | 108 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'hideAnimationClass': the method is already in a final class | 118 |
Priority 4
nl/tudelft/simulation/dsol/web/animation/HtmlDevice.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'GraphicsDevice': 'TYPE_RASTER_SCREEN' is already in scope because it is inherited by an enclosing type | 36 |
org/opentrafficsim/web/test/CircularRoadModel.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 280 |
Files
nl/tudelft/simulation/dsol/jetty/sse/OtsWebModel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 3 | 57 |
| UnusedLocalVariable | Avoid unused local variables such as 'params'. | 3 | 242 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 265 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 286 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 383–386 |
nl/tudelft/simulation/dsol/jetty/sse/OtsWebServer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 3 | 64 |
| UnusedLocalVariable | Avoid unused local variables such as 'params'. | 3 | 275 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 298 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 319 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 416–419 |
nl/tudelft/simulation/dsol/jetty/sse/TestDemoServer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 400–415 |
| EmptyControlStatement | Empty if statement | 3 | 404–415 |
| EmptyControlStatement | Empty if statement | 3 | 408–415 |
| EmptyControlStatement | Empty if statement | 3 | 412–415 |
nl/tudelft/simulation/dsol/web/animation/HtmlDevice.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'GraphicsDevice': 'TYPE_RASTER_SCREEN' is already in scope because it is inherited by an enclosing type | 4 | 36 |
nl/tudelft/simulation/dsol/web/animation/d2/HtmlAnimationPanel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 171 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 195–198 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 316–319 |
org/opentrafficsim/web/animation/WebAnimationToggles.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'showAnimationClass': the method is already in a final class | 3 | 108 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'hideAnimationClass': the method is already in a final class | 3 | 118 |
org/opentrafficsim/web/test/CircularRoadModel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 280 |