PMD Results
The following document contains the results of PMD 5.2.1.
Files
fr/paris/lutece/plugins/crmclient/business/CRMItem.java
Violation |
Line |
Abstract classes should be named AbstractXXX |
49–127 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
55 |
Document empty constructor |
61–63 |
Avoid unnecessary constructors - the compiler will generate these for you |
61–63 |
Avoid appending characters as strings in StringBuffer.append. |
121 |
fr/paris/lutece/plugins/crmclient/service/ICRMClientService.java
Violation |
Line |
Rather than using a lot of String arguments, consider using a container object for those values. |
62 |
Rather than using a lot of String arguments, consider using a container object for those values. |
72 |
Rather than using a lot of String arguments, consider using a container object for those values. |
102 |
Rather than using a lot of String arguments, consider using a container object for those values. |
114–115 |
Rather than using a lot of String arguments, consider using a container object for those values. |
127–128 |
Rather than using a lot of String arguments, consider using a container object for those values. |
141–142 |
Rather than using a lot of String arguments, consider using a container object for those values. |
156–157 |
Rather than using a lot of String arguments, consider using a container object for those values. |
171–172 |
fr/paris/lutece/plugins/crmclient/service/queue/DatabaseQueue.java
Violation |
Line |
Use block level rather than method level synchronization |
57–62 |
Use block level rather than method level synchronization |
68–80 |
fr/paris/lutece/plugins/crmclient/util/CRMException.java
Violation |
Line |
Avoid variables with short names like e |
54 |