Powerful open source chat software
PMD Results
The following document contains the results of PMD
Files
org/embedchat/desktop/client/Client.java
| Violation | Line |
|---|---|
| Perhaps 'handler' could be replaced by a local variable. | 26 |
| It is a good practice to call super() in a constructor | 29 |
| Assigning an Object to null is a code smell. Consider refactoring. | 69 |
| Assigning an Object to null is a code smell. Consider refactoring. | 104 |
org/embedchat/desktop/client/ClientSessionHandler.java
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 29 |
org/embedchat/desktop/Config.java
| Violation | Line |
|---|---|
| Perhaps 'resourcesFolder' could be replaced by a local variable. | 10 |
| It is a good practice to call super() in a constructor | 14 |
org/embedchat/desktop/gui/ChatGUI.java
| Violation | Line |
|---|---|
| Perhaps 'messagesBuff' could be replaced by a local variable. | 45 |
| Perhaps 'chatArea' could be replaced by a local variable. | 54 |
| Perhaps 'sendText' could be replaced by a local variable. | 58 |
| Perhaps 'smilesHandler' could be replaced by a local variable. | 60 |
| Perhaps 'bbCodeProcessor' could be replaced by a local variable. | 61 |
| It is a good practice to call super() in a constructor | 75 |
| These nested if statements could be combined | 323 |
| These nested if statements could be combined | 345 |
| These nested if statements could be combined | 370 |
org/embedchat/desktop/gui/MyHTMLEditorKit.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 6 |
| Each class should declare at least one constructor | 17 |
| A method should have only one exit point, and that should be the last statement in the method | 24 |
org/embedchat/desktop/gui/MyImageView.java
| Violation | Line |
|---|---|
| Assigning an Object to null is a code smell. Consider refactoring. | 77 |
| A method should have only one exit point, and that should be the last statement in the method | 146 |
| A method should have only one exit point, and that should be the last statement in the method | 203 |
| A method should have only one exit point, and that should be the last statement in the method | 232 |
| A method should have only one exit point, and that should be the last statement in the method | 235 |
| A method should have only one exit point, and that should be the last statement in the method | 252 |
| A method should have only one exit point, and that should be the last statement in the method | 257 |
| A method should have only one exit point, and that should be the last statement in the method | 277 |
| Assigning an Object to null is a code smell. Consider refactoring. | 291 |
| Assigning an Object to null is a code smell. Consider refactoring. | 385 |
| A method should have only one exit point, and that should be the last statement in the method | 429 |
| A method should have only one exit point, and that should be the last statement in the method | 431 |
| A method should have only one exit point, and that should be the last statement in the method | 458 |
| Assigning an Object to null is a code smell. Consider refactoring. | 462 |
| A method should have only one exit point, and that should be the last statement in the method | 464 |
| These nested if statements could be combined | 470 |
| These nested if statements could be combined | 474 |
| A method should have only one exit point, and that should be the last statement in the method | 487 |
| A method should have only one exit point, and that should be the last statement in the method | 508 |
| These nested if statements could be combined | 515 |
| A method should have only one exit point, and that should be the last statement in the method | 547 |
| A method should have only one exit point, and that should be the last statement in the method | 570 |
| A method should have only one exit point, and that should be the last statement in the method | 600 |
| A method should have only one exit point, and that should be the last statement in the method | 623 |
| Assigning an Object to null is a code smell. Consider refactoring. | 678 |
| Assigning an Object to null is a code smell. Consider refactoring. | 727 |
| Avoid empty if statements | 735 |
| A method should have only one exit point, and that should be the last statement in the method | 762 |
| Avoid assigments in operands | 768 |
| A method should have only one exit point, and that should be the last statement in the method | 777 |
| Use explicit scoping instead of the default package private level | 826 |
org/embedchat/desktop/gui/OptionsDialog.java
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 78 |
org/embedchat/desktop/gui/RoomListComboModel.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 9 |
org/embedchat/desktop/gui/SmilesHandler.java
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 44 |
| Avoid assigments in operands | 74 |
| Avoid assigments in operands | 86 |
| A method should have only one exit point, and that should be the last statement in the method | 119 |
| Assigning an Object to null is a code smell. Consider refactoring. | 160 |
| A method should have only one exit point, and that should be the last statement in the method | 175 |
| A method should have only one exit point, and that should be the last statement in the method | 181 |
org/embedchat/desktop/gui/SwingUtils.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 7 |
org/embedchat/desktop/Main.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 9 |
org/embedchat/desktop/Observer.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 3 |
org/embedchat/desktop/StateModel.java
| Violation | Line |
|---|---|
| Avoid unnecessary constructors - the compiler will generate these for you | 49 |
| It is a good practice to call super() in a constructor | 49 |
org/javabb/bbcode/BBCodeProcessor.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 33 |
| Avoid empty if statements | 262 |
| It is a good practice to call super() in a constructor | 329 |
| It is a good practice to call super() in a constructor | 338 |
org/javabb/bbcode/CodeTag.java
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 23 |
org/javabb/bbcode/NestedTag.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 27 |
| Avoid modifiers which are implied by the context | 32 |
org/javabb/bbcode/RegexTag.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 27 |
| Avoid modifiers which are implied by the context | 32 |
| Avoid modifiers which are implied by the context | 37 |
org/javabb/bbcode/SimpleRegexTag.java
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 33 |
org/javabb/bbcode/Tag.java
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 24 |
- EmbedChat desktop client
- Project Documentation
