PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/dila/modules/solr/search/SolrDilaLocalIndexer.java
| Violation |
Line |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
103 |
fr/paris/lutece/plugins/dila/modules/solr/search/SolrDilaPublicIndexer.java
| Violation |
Line |
| Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
103 |
fr/paris/lutece/plugins/dila/modules/solr/utils/parsers/DilaSolrLocalParser.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 |
| Private field '_strProdUrl' could be made final; it is only initialized in the declaration or constructor. |
115 |
| Private field '_dilaLocalService' could be made final; it is only initialized in the declaration or constructor. |
121 |
| It is a good practice to call super() in a constructor |
127–161 |
| Assigning an Object to null is a code smell. Consider refactoring. |
239 |
| The user-supplied array 'ch' is stored directly. |
314 |
| The user-supplied array 'ch' is stored directly. |
314 |
| The user-supplied array 'ch' is stored directly. |
314 |
| The user-supplied array 'ch' is stored directly. |
314 |
| Avoid variables with short names like ch |
314 |
| The user-supplied array 'ch' is stored directly. |
314 |
fr/paris/lutece/plugins/dila/modules/solr/utils/parsers/DilaSolrPublicParser.java
| Violation |
Line |
| The class 'DilaSolrPublicParser' has a Cyclomatic Complexity of 4 (Highest = 13). |
69–404 |
| Private field '_listSolrItems' could be made final; it is only initialized in the declaration or constructor. |
110 |
| Private field '_strType' could be made final; it is only initialized in the declaration or constructor. |
118 |
| Private field '_strSite' could be made final; it is only initialized in the declaration or constructor. |
120 |
| Private field '_strProdUrl' could be made final; it is only initialized in the declaration or constructor. |
121 |
| It is a good practice to call super() in a constructor |
130–175 |
| Assigning an Object to null is a code smell. Consider refactoring. |
264 |
| Assigning an Object to null is a code smell. Consider refactoring. |
269 |
| The method 'characters' has a Cyclomatic Complexity of 13. |
344–393 |
| The user-supplied array 'ch' is stored directly. |
344 |
| Avoid variables with short names like ch |
344 |
| The user-supplied array 'ch' is stored directly. |
344 |
| The user-supplied array 'ch' is stored directly. |
344 |
| The user-supplied array 'ch' is stored directly. |
344 |
| The user-supplied array 'ch' is stored directly. |
344 |
| The user-supplied array 'ch' is stored directly. |
344 |
