Résultats PMD
Le document suivant contient les résultats de PMD 5.5.1.
Fichiers
fr/paris/lutece/plugins/chatbotrecorder/business/ChatBotApp.java
Violation |
Ligne |
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 |
Ligne |
Position literals first in String comparisons |
257 |
fr/paris/lutece/plugins/chatbotrecorder/business/ConversationHome.java
Violation |
Ligne |
Avoid using Literals in Conditional Statements |
88 |
fr/paris/lutece/plugins/chatbotrecorder/web/AbstractChatBotRecorderManagerJspBean.java
Violation |
Ligne |
Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. |
62 |
fr/paris/lutece/plugins/chatbotrecorder/web/ConversationJspBean.java
Violation |
Ligne |
Assigning an Object to null is a code smell. Consider refactoring. |
102 |
fr/paris/lutece/plugins/chatbotrecorder/web/ReplayConversationJspBean.java
Violation |
Ligne |
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 |
Ligne |
Assigning an Object to null is a code smell. Consider refactoring. |
106 |
fr/paris/lutece/plugins/chatbotrecorder/web/ScenarioJspBean.java
Violation |
Ligne |
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 |