PMD Results
The following document contains the results of PMD 5.5.1.
Files
fr/paris/lutece/plugins/chatbotrecorder/business/ChatBotApp.java
| Violation |
Line |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
47–109 |
fr/paris/lutece/plugins/chatbotrecorder/business/ConversationDAO.java
| Violation |
Line |
| Position literals first in String comparisons |
257 |
fr/paris/lutece/plugins/chatbotrecorder/business/ConversationHome.java
| Violation |
Line |
| Avoid using Literals in Conditional Statements |
88 |
fr/paris/lutece/plugins/chatbotrecorder/web/AbstractChatBotRecorderManagerJspBean.java
| Violation |
Line |
| Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
62 |
fr/paris/lutece/plugins/chatbotrecorder/web/ConversationJspBean.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
102 |
fr/paris/lutece/plugins/chatbotrecorder/web/ReplayConversationJspBean.java
| Violation |
Line |
| The class 'ReplayConversationJspBean' has a Cyclomatic Complexity of 3 (Highest = 13). |
58–319 |
| The class 'ReplayConversationJspBean' has a Modified Cyclomatic Complexity of 3 (Highest = 10). |
58–319 |
| The class 'ReplayConversationJspBean' has a Standard Cyclomatic Complexity of 3 (Highest = 10). |
58–319 |
| Assigning an Object to null is a code smell. Consider refactoring. |
113 |
| The method 'compareConversations' has a Standard Cyclomatic Complexity of 10. |
265–318 |
| The method 'compareConversations' has a Cyclomatic Complexity of 13. |
265–318 |
| The method compareConversations() has an NPath complexity of 292 |
265–318 |
| The method 'compareConversations' has a Modified Cyclomatic Complexity of 10. |
265–318 |
| Avoid variables with short names like j |
287 |
fr/paris/lutece/plugins/chatbotrecorder/web/ReplayJspBean.java
| Violation |
Line |
| Assigning an Object to null is a code smell. Consider refactoring. |
106 |
fr/paris/lutece/plugins/chatbotrecorder/web/ScenarioJspBean.java
| Violation |
Line |
| A high number of imports can indicate a high degree of coupling within an object. |
34–524 |
| The class 'ScenarioJspBean' has a Cyclomatic Complexity of 4 (Highest = 13). |
71–524 |
| Assigning an Object to null is a code smell. Consider refactoring. |
132 |
| Position literals first in String comparisons |
190 |
| Avoid printStackTrace(); use a logger call instead. |
200 |
| Avoid using Literals in Conditional Statements |
208 |
| Avoid using Literals in Conditional Statements |
245 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
245 |
| Avoid using Literals in Conditional Statements |
449 |
| The method 'compareConversations' has a Cyclomatic Complexity of 13. |
472–523 |
