The following document contains the results of PMD 3.7.
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 8 - 52 |
| Use explicit scoping instead of the default package private level | 27 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 7 - 39 |
| Use explicit scoping instead of the default package private level | 10 |
| Use explicit scoping instead of the default package private level | 13 |
| Use explicit scoping instead of the default package private level | 16 |
| Use explicit scoping instead of the default package private level | 19 |
| Use explicit scoping instead of the default package private level | 22 |
| Use explicit scoping instead of the default package private level | 25 |
| Use explicit scoping instead of the default package private level | 29 |
| Violation | Line |
|---|---|
| Use explicit scoping instead of the default package private level | 15 |
| Use explicit scoping instead of the default package private level | 16 |
| Use explicit scoping instead of the default package private level | 17 |
| Use explicit scoping instead of the default package private level | 19 |
| Use explicit scoping instead of the default package private level | 20 |
| A method should have only one exit point, and that should be the last statement in the method | 92 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 3 - 10 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 3 - 10 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 9 - 102 |
| Violation | Line |
|---|---|
| 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 | 26 - 255 |
| Avoid assigments in operands | 122 - 136 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 9 - 128 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 12 - 19 |
| Violation | Line |
|---|---|
| It is a good practice to call super() in a constructor | 17 - 25 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 14 - 115 |
| A method should have only one exit point, and that should be the last statement in the method | 29 |
| A method should have only one exit point, and that should be the last statement in the method | 74 |
| 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 | 12 - 54 |
| Violation | Line |
|---|---|
| Assigning an Object to null is a code smell. Consider refactoring. | 26 |
| Assigning an Object to null is a code smell. Consider refactoring. | 52 |
| Assigning an Object to null is a code smell. Consider refactoring. | 64 |
| Violation | Line |
|---|---|
| Avoid unnecessary constructors - the compiler will generate these for you | 30 - 32 |
| Assigning an Object to null is a code smell. Consider refactoring. | 58 |
| Violation | Line |
|---|---|
| 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 | 11 |
| Avoid modifiers which are implied by the context | 12 |
| Avoid modifiers which are implied by the context | 13 |
| Violation | Line |
|---|---|
| Each class should declare at least one constructor | 24 - 209 |
| Assigning an Object to null is a code smell. Consider refactoring. | 105 |
| Violation | Line |
|---|---|
| Perhaps 'scrollH' could be replaced by a local variable. | 41 |
| Perhaps 'scrollV' could be replaced by a local variable. | 42 |
| These nested if statements could be combined | 245 - 250 |
| Violation | Line |
|---|---|
| A method should have only one exit point, and that should be the last statement in the method | 70 |
| Violation | Line |
|---|---|
| Perhaps 'mapPanel' could be replaced by a local variable. | 48 |
| It is a good practice to call super() in a constructor | 111 - 114 |
| Avoid empty if statements | 203 |
| Avoid empty if statements | 299 |
