PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/indicator/service/IndicatorUpdaterService.java
| Violation |
Line |
| Use block level rather than method level synchronization |
58–67 |
| StringBuffer constructor is initialized with size 16, but has at least 38 characters appended. |
88 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
105 |
fr/paris/lutece/plugins/indicator/web/IndicatorHistoryJspBean.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
113 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
157 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
168 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
173 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
184 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
227 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
271 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
281 |
fr/paris/lutece/plugins/indicator/web/IndicatorJspBean.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
112 |
fr/paris/lutece/plugins/indicator/web/ManageIndicatorJspBean.java
| Violation |
Line |
| Abstract classes should be named AbstractXXX |
51–97 |
| Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
62 |
| Do not add empty strings |
91 |
