PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/search/solr/nutch/business/NutchSearchEngine.java
| Violation |
Line |
| Singleton is not thread safe |
83–86 |
| Avoid instantiating Integer objects. Call Integer.valueOf() instead. |
100 |
| Avoid reassigning parameters such as 'strQuery' |
120 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid reassigning parameters such as 's' |
171 |
| Avoid variables with short names like s |
171 |
| Avoid reassigning parameters such as 's' |
171 |
fr/paris/lutece/plugins/search/solr/nutch/business/NutchSearchItem.java
| Violation |
Line |
| It is a good practice to call super() in a constructor |
58–64 |
fr/paris/lutece/plugins/search/solr/nutch/business/NutchServerService.java
| Violation |
Line |
| Private field '_solrServer' could be made final; it is only initialized in the declaration or constructor. |
54 |
| Singleton is not thread safe |
70–73 |
fr/paris/lutece/plugins/search/solr/nutch/util/NutchUtil.java
| Violation |
Line |
| Avoid using Literals in Conditional Statements |
109 |
fr/paris/lutece/plugins/search/solr/nutch/web/NutchSearchApp.java
| Violation |
Line |
| Avoid instantiating Integer objects. Call Integer.valueOf() instead. |
132 |
| Prefer StringBuffer over += for concatenating strings |
135 |
| Prefer StringBuffer over += for concatenating strings |
135 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
159 |
