The following document contains the results of PMD
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 65 |
| Assigning an Object to null is a code smell. Consider refactoring. | 130 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 52 |
| A method should have only one exit point, and that should be the last statement in the method | 129 |
| A method should have only one exit point, and that should be the last statement in the method | 133 |
| Avoid instantiating Boolean objects; you can usually invoke Boolean.TRUE instead. | 136 |
| A method should have only one exit point, and that should be the last statement in the method | 144 |
| A method should have only one exit point, and that should be the last statement in the method | 148 |
| A method should have only one exit point, and that should be the last statement in the method | 159 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 50 |
| A method should have only one exit point, and that should be the last statement in the method | 64 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 47 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 47 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 69 |
| It is a good practice to call super() in a constructor | 72 |
| A method should have only one exit point, and that should be the last statement in the method | 122 |
| A method should have only one exit point, and that should be the last statement in the method | 152 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 53 |
| It is a good practice to call super() in a constructor | 57 |
| Unnecessary final modifier in final class | 69 |
| Unnecessary final modifier in final class | 78 |
| Unnecessary final modifier in final class | 87 |
| Unnecessary final modifier in final class | 96 |
| Unnecessary final modifier in final class | 107 |
| Unnecessary final modifier in final class | 118 |
| Unnecessary final modifier in final class | 129 |
| Unnecessary final modifier in final class | 140 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 39 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 34 |
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 53 |
| Avoid modifiers which are implied by the context | 68 |
| Avoid modifiers which are implied by the context | 83 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 109 |
| Avoid unnecessary temporaries when converting primitives to Strings | 227 |
| Avoid empty catch blocks | 234 |
| Assigning an Object to null is a code smell. Consider refactoring. | 341 |
| Assigning an Object to null is a code smell. Consider refactoring. | 351 |
| A method should have only one exit point, and that should be the last statement in the method | 364 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 145 |
| Assigning an Object to null is a code smell. Consider refactoring. | 198 |
| A method should have only one exit point, and that should be the last statement in the method | 309 |
| A method should have only one exit point, and that should be the last statement in the method | 341 |
| Avoid empty catch blocks | 466 |
| Avoid empty catch blocks | 524 |
| Avoid empty catch blocks | 531 |
| A method should have only one exit point, and that should be the last statement in the method | 591 |
| Avoid assigments in operands | 605 |
| A method should have only one exit point, and that should be the last statement in the method | 708 |
| A method should have only one exit point, and that should be the last statement in the method | 721 |
| Avoid assigments in operands | 772 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 26 |
| Violation | Line |
|---|---|
| Avoid unnecessary constructors - the compiler will generate these for you | 79 |
| It is a good practice to call super() in a constructor | 79 |
| A method should have only one exit point, and that should be the last statement in the method | 89 |
| Assigning an Object to null is a code smell. Consider refactoring. | 151 |
| Assigning an Object to null is a code smell. Consider refactoring. | 157 |
| 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 | 85 |
| A method should have only one exit point, and that should be the last statement in the method | 87 |
| A method should have only one exit point, and that should be the last statement in the method | 120 |
| A method should have only one exit point, and that should be the last statement in the method | 124 |
| A method should have only one exit point, and that should be the last statement in the method | 126 |
| A method should have only one exit point, and that should be the last statement in the method | 128 |
| A method should have only one exit point, and that should be the last statement in the method | 150 |
| A method should have only one exit point, and that should be the last statement in the method | 170 |
| A method should have only one exit point, and that should be the last statement in the method | 172 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 52 |
| A method should have only one exit point, and that should be the last statement in the method | 103 |
| A method should have only one exit point, and that should be the last statement in the method | 107 |
| Avoid instantiating Boolean objects; you can usually invoke Boolean.TRUE instead. | 110 |
| A method should have only one exit point, and that should be the last statement in the method | 118 |
| A method should have only one exit point, and that should be the last statement in the method | 122 |
| A method should have only one exit point, and that should be the last statement in the method | 133 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 51 |
| A method should have only one exit point, and that should be the last statement in the method | 65 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 64 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 62 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 70 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 64 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 78 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 78 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 64 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 61 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 77 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 69 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 66 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 67 |
| It is a good practice to call super() in a constructor | 70 |
| A method should have only one exit point, and that should be the last statement in the method | 96 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 54 |
| Assigning an Object to null is a code smell. Consider refactoring. | 314 |
| Assigning an Object to null is a code smell. Consider refactoring. | 332 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 72 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 60 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 40 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 60 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 61 |
| Violation | Line |
|---|---|
| Perhaps '_javaClass' could be replaced by a local variable. | 53 |
| It is a good practice to call super() in a constructor | 56 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 90 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 40 |
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 46 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 67 |
| 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 | 77 |
| A method should have only one exit point, and that should be the last statement in the method | 95 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 43 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 46 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 77 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 71 |
| Avoid empty if statements | 76 |
| Unnecessary final modifier in final class | 95 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 58 |
| Double checked locking is not thread safe in Java. | 66 |
| Unnecessary final modifier in final class | 66 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 70 |
| It is a good practice to call super() in a constructor | 81 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 82 |
| Assigning an Object to null is a code smell. Consider refactoring. | 197 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 101 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 45 |
| A method should have only one exit point, and that should be the last statement in the method | 62 |
| A method should have only one exit point, and that should be the last statement in the method | 78 |
| A method should have only one exit point, and that should be the last statement in the method | 86 |