PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/exportuserpreferences/business/Key.java
| Violation |
Line |
| Avoid unused imports such as 'org.hibernate.validator.constraints' |
36 |
| Avoid unused imports such as 'javax.validation.constraints' |
38 |
| Avoid short class names like Key |
44–87 |
| A getX() method which returns a boolean should be named isX() |
74–77 |
fr/paris/lutece/plugins/exportuserpreferences/business/KeyDAO.java
| Violation |
Line |
| Avoid unused imports such as 'java.util.List' |
45 |
| Avoid unused imports such as 'java.util.List' |
45 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
208 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
216 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
217 |
| Avoid throwing raw exception types. |
223 |
fr/paris/lutece/plugins/exportuserpreferences/utils/CsvUtils.java
| Violation |
Line |
| Avoid unused imports such as 'org.apache.commons.beanutils.BeanUtils' |
42 |
| Avoid unused imports such as 'org.apache.commons.beanutils.BeanUtils' |
42 |
| Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' |
49 |
| Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' |
49 |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
59–206 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
78 |
| Avoid throwing raw exception types. |
86 |
| An empty statement (semicolon) not part of a loop |
95 |
| Avoid variables with short names like i |
101 |
| Avoid throwing raw exception types. |
117 |
| Avoid prefixing parameters by in, out or inOut. Uses Javadoc to document this behavior. |
134 |
| Avoid throwing raw exception types. |
147 |
| Avoid throwing raw exception types. |
201 |
fr/paris/lutece/plugins/exportuserpreferences/web/KeyJspBean.java
| Violation |
Line |
| Avoid unused private fields such as 'PROPERTY_DEFAULT_LIST_KEY_PER_PAGE'. |
91 |
| Assigning an Object to null is a code smell. Consider refactoring. |
119 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
242 |
| Avoid variables with short names like os |
269 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
284 |
fr/paris/lutece/plugins/exportuserpreferences/web/ManageExportuserpreferencesJspBean.java
| Violation |
Line |
| Abstract classes should be named AbstractXXX |
51–88 |
| Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
61 |
| Do not add empty strings |
82 |
