Résultats PMD
Le document suivant contient les résultats de PMD 5.3.5.
Fichiers
fr/paris/lutece/plugins/shorturl/service/ShortUrlRedirect.java
| Violation |
Ligne |
| Avoid unused private fields such as 'PARAMETER_KEY'. |
51 |
| Avoid unused private fields such as 'MESSAGE_URL_NOT_EXISTS'. |
52 |
| Use explicit scoping instead of the default package private level |
54 |
| Prefer StringBuffer over += for concatenating strings |
62 |
| String.indexOf(char) is faster than String.indexOf(String). |
70 |
fr/paris/lutece/plugins/shorturl/web/ShortUrlApp.java
| Violation |
Ligne |
| The class 'ShortUrlApp' has a Cyclomatic Complexity of 4 (Highest = 10). |
63–178 |
| The method 'getPage' has a Cyclomatic Complexity of 10. |
88–130 |
| Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' |
121 |
| Avoid unused method parameters such as 'timestamp'. |
133 |
| Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' |
138 |
| Avoid unused method parameters such as 'strAbbreviation'. |
143 |
| Avoid unused method parameters such as 'strPageName'. |
147 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
148 |
| Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
155 |
| Avoid using for statements without curly braces |
158–159 |
| Avoid using for statements without curly braces |
160–161 |
| Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
164 |
| Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. |
166 |
| Avoid using for statements without curly braces |
172–173 |
