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'. | 58 |
UnusedLocalVariable | Avoid unused local variables such as 'params'. | 244 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 267 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 288 |
CollapsibleIfStatements | This if statement could be combined with its parent | 385–388 |
nl/tudelft/simulation/dsol/jetty/sse/OtsWebServer.java
Rule | Violation | Line |
---|---|---|
UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 65 |
UnusedLocalVariable | Avoid unused local variables such as 'params'. | 277 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 300 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 321 |
CollapsibleIfStatements | This if statement could be combined with its parent | 418–421 |
nl/tudelft/simulation/dsol/jetty/sse/TestDemoServer.java
Rule | Violation | Line |
---|---|---|
EmptyControlStatement | Empty if statement | 402–417 |
EmptyControlStatement | Empty if statement | 406–417 |
EmptyControlStatement | Empty if statement | 410–417 |
EmptyControlStatement | Empty if statement | 414–417 |
nl/tudelft/simulation/dsol/web/animation/d2/HtmlAnimationPanel.java
Rule | Violation | Line |
---|---|---|
UnusedPrivateField | Avoid unused private fields such as 'gtuColorer'. | 94 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 163 |
CollapsibleIfStatements | This if statement could be combined with its parent | 187–190 |
EmptyCatchBlock | Avoid empty catch blocks | 309–312 |
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 | 37 |
org/opentrafficsim/web/test/CircularRoadModel.java
Rule | Violation | Line |
---|---|---|
UnnecessaryImport | Unused import 'java.util.LinkedHashSet' | 4 |
UnnecessaryImport | Unused import 'java.util.Set' | 7 |
org/opentrafficsim/web/test/TJunctionModel.java
Rule | Violation | Line |
---|---|---|
UnnecessaryImport | Unused import 'java.rmi.RemoteException' | 4 |
UnnecessaryImport | Unused import 'javax.naming.NamingException' | 6 |
UnnecessaryImport | Unused import 'org.opentrafficsim.core.network.NetworkException' | 15 |
UnnecessaryImport | Unused import 'org.opentrafficsim.draw.road.TrafficLightAnimation' | 16 |
Files
nl/tudelft/simulation/dsol/jetty/sse/OtsWebModel.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 3 | 58 |
UnusedLocalVariable | Avoid unused local variables such as 'params'. | 3 | 244 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 267 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 288 |
CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 385–388 |
nl/tudelft/simulation/dsol/jetty/sse/OtsWebServer.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedPrivateField | Avoid unused private fields such as 'dirtyControls'. | 3 | 65 |
UnusedLocalVariable | Avoid unused local variables such as 'params'. | 3 | 277 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 300 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 321 |
CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 418–421 |
nl/tudelft/simulation/dsol/jetty/sse/TestDemoServer.java
Rule | Violation | Priority | Line |
---|---|---|---|
EmptyControlStatement | Empty if statement | 3 | 402–417 |
EmptyControlStatement | Empty if statement | 3 | 406–417 |
EmptyControlStatement | Empty if statement | 3 | 410–417 |
EmptyControlStatement | Empty if statement | 3 | 414–417 |
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 | 37 |
nl/tudelft/simulation/dsol/web/animation/d2/HtmlAnimationPanel.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedPrivateField | Avoid unused private fields such as 'gtuColorer'. | 3 | 94 |
SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 163 |
CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 187–190 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 309–312 |
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 |
---|---|---|---|
UnnecessaryImport | Unused import 'java.util.LinkedHashSet' | 4 | 4 |
UnnecessaryImport | Unused import 'java.util.Set' | 4 | 7 |
org/opentrafficsim/web/test/TJunctionModel.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryImport | Unused import 'java.rmi.RemoteException' | 4 | 4 |
UnnecessaryImport | Unused import 'javax.naming.NamingException' | 4 | 6 |
UnnecessaryImport | Unused import 'org.opentrafficsim.core.network.NetworkException' | 4 | 15 |
UnnecessaryImport | Unused import 'org.opentrafficsim.draw.road.TrafficLightAnimation' | 4 | 16 |