The following document contains the results of PMD 4.2.2.
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 38 |
Avoid modifiers which are implied by the context | 49 |
Avoid modifiers which are implied by the context | 58 |
Avoid modifiers which are implied by the context | 64 |
Violation | Line |
---|---|
Avoid using final local variables, turn them into fields | 68 |
Avoid using final local variables, turn them into fields | 87 |
Violation | Line |
---|---|
Avoid unnecessary temporaries when converting primitives to Strings | 49 |
Violation | Line |
---|---|
Avoid using final local variables, turn them into fields | 41 |
Found 'DD'-anomaly for variable 'delivered' (lines '76'-'97'). | 76 - 97 |
Avoid using final local variables, turn them into fields | 87 |
Avoid using final local variables, turn them into fields | 88 |
Avoid using final local variables, turn them into fields | 94 |
Found 'DU'-anomaly for variable 'delivered' (lines '97'-'122'). | 97 - 122 |
Avoid using final local variables, turn them into fields | 101 |
Avoid using final local variables, turn them into fields | 114 |
Violation | Line |
---|---|
Avoid using final local variables, turn them into fields | 40 |
Avoid using final local variables, turn them into fields | 62 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 39 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 39 |
Violation | Line |
---|---|
Avoid using final local variables, turn them into fields | 43 |
Violation | Line |
---|---|
Avoid using final local variables, turn them into fields | 54 |
Found 'DD'-anomaly for variable 'reporting_UA_name' (lines '87'-'90'). | 87 - 90 |
Found 'DD'-anomaly for variable 'reporting_UA_name' (lines '90'-'95'). | 90 - 95 |
Found 'DD'-anomaly for variable 'original_recipient' (lines '102'-'105'). | 102 - 105 |
Violation | Line |
---|---|
Avoid modifiers which are implied by the context | 65 |
Violation | Line |
---|---|
Found 'DD'-anomaly for variable 'actions' (lines '121'-'122'). | 121 - 122 |
Avoid assignments in operands | 122 |
Found 'DU'-anomaly for variable 'actions' (lines '122'-'128'). | 122 - 128 |
A method should have only one exit point, and that should be the last statement in the method | 125 |
Avoid using final local variables, turn them into fields | 161 |
Avoid using final local variables, turn them into fields | 291 |
Found 'DD'-anomaly for variable 'recipient' (lines '317'-'320'). | 317 - 320 |
Assigning an Object to null is a code smell. Consider refactoring. | 342 |
Found 'DD'-anomaly for variable 'messageID' (lines '370'-'373'). | 370 - 373 |
Found 'DD'-anomaly for variable 'messageID' (lines '373'-'377'). | 373 - 377 |
Avoid using final local variables, turn them into fields | 396 |
Avoid using final local variables, turn them into fields | 397 |
Avoid using final local variables, turn them into fields | 398 |
Found 'DD'-anomaly for variable 'results' (lines '398'-'401'). | 398 - 401 |
Found 'DU'-anomaly for variable 'results' (lines '398'-'407'). | 398 - 407 |
Avoid using final local variables, turn them into fields | 400 |
Found 'DD'-anomaly for variable 'results' (lines '401'-'401'). | 401 |
Found 'DU'-anomaly for variable 'results' (lines '401'-'407'). | 401 - 407 |
Violation | Line |
---|---|
It is a good practice to call super() in a constructor | 88 |
Avoid using final local variables, turn them into fields | 195 |
Avoid using final local variables, turn them into fields | 196 |
Found 'DD'-anomaly for variable 'message' (lines '227'-'229'). | 227 - 229 |
Found 'DU'-anomaly for variable 'deliveredTo' (lines '243'-'298'). | 243 - 298 |
These nested if statements could be combined | 265 - 275 |
Use explicit scoping instead of the default package private level | 336 - 366 |
Avoid using final local variables, turn them into fields | 341 |
Use explicit scoping instead of the default package private level | 368 - 371 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 34 - 37 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 34 - 37 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 31 - 34 |
Violation | Line |
---|---|
Avoid unnecessary constructors - the compiler will generate these for you | 32 - 35 |