Powerful open source chat software
PMD Results
The following document contains the results of PMD
Files
org/embedchat/protocol/codec/AbstractMessageDecoder.java
| Violation | Line |
|---|---|
| Perhaps 'readHeader' could be replaced by a local variable. | 15 |
| It is a good practice to call super() in a constructor | 17 |
| A method should have only one exit point, and that should be the last statement in the method | 24 |
| A method should have only one exit point, and that should be the last statement in the method | 31 |
| A method should have only one exit point, and that should be the last statement in the method | 47 |
org/embedchat/protocol/codec/AbstractMessageEncoder.java
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 14 |
org/embedchat/protocol/codec/ChatProtocolCodecFactory.java
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 20 |
org/embedchat/protocol/codec/client/LoginMessageDecoder.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 22 |
| A method should have only one exit point, and that should be the last statement in the method | 33 |
org/embedchat/protocol/codec/client/RoomsListRequestMessageDecoder.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 22 |
| A method should have only one exit point, and that should be the last statement in the method | 31 |
org/embedchat/protocol/codec/common/TextMessageDecoder.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 22 |
| A method should have only one exit point, and that should be the last statement in the method | 32 |
org/embedchat/protocol/codec/server/ErrorMessageDecoder.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 23 |
| A method should have only one exit point, and that should be the last statement in the method | 33 |
org/embedchat/protocol/codec/server/RoomsListMessageDecoder.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 23 |
| A method should have only one exit point, and that should be the last statement in the method | 35 |
org/embedchat/protocol/codec/server/UsersListMessageDecoder.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 23 |
| A method should have only one exit point, and that should be the last statement in the method | 35 |
org/embedchat/protocol/Constants.java
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 40 |
org/embedchat/protocol/message/AbstractMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 5 |
org/embedchat/protocol/message/client/LoginMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 18 |
| A method should have only one exit point, and that should be the last statement in the method | 68 |
org/embedchat/protocol/message/client/RoomsListRequestMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 8 |
| A method should have only one exit point, and that should be the last statement in the method | 36 |
org/embedchat/protocol/message/common/TextMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 7 |
| A method should have only one exit point, and that should be the last statement in the method | 43 |
org/embedchat/protocol/message/server/ErrorMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 18 |
| A method should have only one exit point, and that should be the last statement in the method | 70 |
org/embedchat/protocol/message/server/RoomsListMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 22 |
| A method should have only one exit point, and that should be the last statement in the method | 56 |
| It is a good practice to call super() in a constructor | 71 |
| A method should have only one exit point, and that should be the last statement in the method | 101 |
org/embedchat/protocol/message/server/UsersListMessage.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 25 |
| A method should have only one exit point, and that should be the last statement in the method | 59 |
| A method should have only one exit point, and that should be the last statement in the method | 83 |
| It is a good practice to call super() in a constructor | 95 |
- EmbedChat Protocol
- Project Documentation
