PMD Results
The following document contains the results of PMD 5.5.1.
Files
fr/paris/lutece/plugins/chatbot/business/ChatData.java
| Violation |
Line |
| Private field '_listPosts' could be made final; it is only initialized in the declaration or constructor. |
45 |
fr/paris/lutece/plugins/chatbot/business/Post.java
| Violation |
Line |
| Avoid short class names like Post |
41–166 |
| Document empty constructor |
58–60 |
fr/paris/lutece/plugins/chatbot/service/BotService.java
| Violation |
Line |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
49 |
fr/paris/lutece/plugins/chatbot/service/ChatService.java
| Violation |
Line |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
56 |
fr/paris/lutece/plugins/chatbot/service/bot/AbstractChatBot.java
| Violation |
Line |
| An empty method in an abstract class should be abstract instead |
287–290 |
| Document empty method |
288–290 |
fr/paris/lutece/plugins/chatbot/service/bot/MockChatBot.java
| Violation |
Line |
| Avoid unused imports such as 'fr.paris.lutece.plugins.chatbot.business.Post' |
37 |
| Avoid unused imports such as 'fr.paris.lutece.plugins.chatbot.business.Post' |
37 |
fr/paris/lutece/plugins/chatbot/web/ChatBotApp.java
| Violation |
Line |
| The String literal "true" appears 5 times in this file; the first occurrence is on line 155 |
155 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
185 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
211 |
| Avoid unnecessary if..then..else statements when returning booleans |
312–315 |
fr/paris/lutece/plugins/chatbot/web/ChatPageInclude.java
| Violation |
Line |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
63 |
