The following document contains the results of PMD 3.7.
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 32 - 76 |
| Violation | Line |
|---|---|
| Use explicit scoping instead of the default package private level | 60 |
| It is a good practice to call super() in a constructor | 64 - 78 |
| Avoid assigments in operands | 104 - 106 |
| Avoid assigments in operands | 109 - 111 |
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 33 |
| Violation | Line |
|---|---|
| Avoid modifiers which are implied by the context | 5 |
| Avoid modifiers which are implied by the context | 6 |
| Avoid modifiers which are implied by the context | 7 |
| Avoid modifiers which are implied by the context | 8 |
| Avoid modifiers which are implied by the context | 10 |
| Avoid modifiers which are implied by the context | 11 |
| Avoid modifiers which are implied by the context | 13 |
| Avoid modifiers which are implied by the context | 14 |
| Avoid modifiers which are implied by the context | 15 |
| Avoid modifiers which are implied by the context | 16 |
| Violation | Line |
|---|---|
| Perhaps 'appName' could be replaced by a local variable. | 42 |
| Assigning an Object to null is a code smell. Consider refactoring. | 57 |
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 69 |