PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/paybox/PayboxUtil.java
| Violation |
Line |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
115 |
| Avoid using implementation types like 'LinkedHashMap'; use the interface instead |
190 |
| Avoid using implementation types like 'LinkedHashMap'; use the interface instead |
190 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
203 |
| StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. |
204 |
| System.arraycopy is more efficient |
222–225 |
| When doing a String.toLowerCase()/toUpperCase() call, use a Locale |
328 |
| Avoid using implementation types like 'LinkedHashMap'; use the interface instead |
364 |
| Avoid using implementation types like 'LinkedHashMap'; use the interface instead |
364 |
fr/paris/lutece/plugins/paybox/dao/entity/PayboxLogEntity.java
| Violation |
Line |
| Avoid variables with short names like id |
98 |
fr/paris/lutece/plugins/paybox/dao/impl/PayboxLogDAOImpl.java
| Violation |
Line |
| Avoid variables with short names like id |
84 |
| Avoid using final local variables, turn them into fields |
175 |
fr/paris/lutece/plugins/paybox/dto/PayboxLogDTO.java
| Violation |
Line |
| Avoid variables with short names like id |
98 |
fr/paris/lutece/plugins/paybox/service/PayboxService.java
| Violation |
Line |
| A method/constructor shouldnt explicitly throw java.lang.Exception |
56 |
| A method/constructor shouldnt explicitly throw java.lang.Exception |
56 |
| A method/constructor shouldnt explicitly throw java.lang.Exception |
67 |
| A method/constructor shouldnt explicitly throw java.lang.Exception |
67 |
fr/paris/lutece/plugins/paybox/service/impl/PayboxLogServiceImpl.java
| Violation |
Line |
| Avoid variables with short names like b |
102 |
fr/paris/lutece/plugins/paybox/service/role/PayboxReadResourceIdService.java
| Violation |
Line |
| It is a good practice to call super() in a constructor |
56–59 |
| Avoid variables with short names like rt |
66 |
| Avoid variables with short names like p |
71 |
fr/paris/lutece/plugins/paybox/web/PayboxAdminJspBean.java
| Violation |
Line |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
87 |
fr/paris/lutece/plugins/paybox/web/PayboxJspBean.java
| Violation |
Line |
| Avoid using Literals in Conditional Statements |
78 |
| debug logging that involves string concatenation should be guarded with isDebugEnabled() checks |
85 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
102 |
fr/paris/lutece/plugins/paybox/web/PayboxServlet.java
| Violation |
Line |
| Avoid using Literals in Conditional Statements |
122 |
| debug logging that involves string concatenation should be guarded with isDebugEnabled() checks |
129 |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
135 |
