PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/comarquage/modules/solr/search/SolrComarquageLocalIndexer.java
| Violation |
Line |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
104 |
fr/paris/lutece/plugins/comarquage/modules/solr/search/SolrComarquagePublicIndexer.java
| Violation |
Line |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
104 |
fr/paris/lutece/plugins/comarquage/modules/solr/utils/parsers/CoMarquageSolrLocalParser.java
| Violation |
Line |
| Private field '_listSolrItems' could be made final; it is only initialized in the declaration or constructor. |
105 |
| Private field '_strType' could be made final; it is only initialized in the declaration or constructor. |
113 |
| Private field '_strSite' could be made final; it is only initialized in the declaration or constructor. |
114 |
| It is a good practice to call super() in a constructor |
122–163 |
| Assigning an Object to null is a code smell. Consider refactoring. |
252 |
| The user-supplied array 'ch' is stored directly. |
321 |
| The user-supplied array 'ch' is stored directly. |
321 |
| The user-supplied array 'ch' is stored directly. |
321 |
| The user-supplied array 'ch' is stored directly. |
321 |
| Avoid variables with short names like ch |
321 |
fr/paris/lutece/plugins/comarquage/modules/solr/utils/parsers/CoMarquageSolrPublicParser.java
| Violation |
Line |
| The class 'CoMarquageSolrPublicParser' has a Cyclomatic Complexity of 4 (Highest = 13). |
64–376 |
| Private field '_listSolrItems' could be made final; it is only initialized in the declaration or constructor. |
109 |
| Private field '_strType' could be made final; it is only initialized in the declaration or constructor. |
117 |
| Private field '_strSite' could be made final; it is only initialized in the declaration or constructor. |
119 |
| It is a good practice to call super() in a constructor |
127–181 |
| Assigning an Object to null is a code smell. Consider refactoring. |
281 |
| The user-supplied array 'ch' is stored directly. |
319 |
| The user-supplied array 'ch' is stored directly. |
319 |
| The user-supplied array 'ch' is stored directly. |
319 |
| Avoid variables with short names like ch |
319 |
| The user-supplied array 'ch' is stored directly. |
319 |
| The user-supplied array 'ch' is stored directly. |
319 |
| The user-supplied array 'ch' is stored directly. |
319 |
| The method 'characters' has a Cyclomatic Complexity of 13. |
319–365 |
