PMD Results

The following document contains the results of PMD 5.6.1.

Files

Date.java

Violation Line
Avoid short class names like Date 2–40
All classes and interfaces must belong to a named package 2–40
Use explicit scoping instead of the default package private level 3
Use explicit scoping instead of the default package private level 4
Use explicit scoping instead of the default package private level 5
Avoid unnecessary constructors - the compiler will generate these for you 7–9
Parameter 'day' is not assigned and could be declared final 15
Parameter 'month' is not assigned and could be declared final 23
Parameter 'year' is not assigned and could be declared final 31

DateController.java

Violation Line
All classes and interfaces must belong to a named package 2–28
Avoid unnecessary constructors - the compiler will generate these for you 4–6
Avoid variables with short names like t 8
Parameter 't' is not assigned and could be declared final 8
Avoid unnecessary if..then..else statements when returning booleans 15–19
A method should have only one exit point, and that should be the last statement in the method 16
A method should have only one exit point, and that should be the last statement in the method 18
A method should have only one exit point, and that should be the last statement in the method 21

NumerosValidosException.java

Violation Line
All classes and interfaces must belong to a named package 2–6
Parameter 'errorMessage' is not assigned and could be declared final 3

Principal.java

Violation Line
All classes and interfaces must belong to a named package 4–37
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 4–37
The field name indicates a constant but its modifiers do not 5
Variables should start with a lowercase character, 'TECLADO' starts with uppercase character. 5
Use explicit scoping instead of the default package private level 5
Parameter 'args' is not assigned and could be declared final 8
Local variable 'list' could be declared final 9
Local variable 'entrada' could be declared final 10
Potential violation of Law of Demeter (method chain calls) 13
Local variable 'd' could be declared final 16
Avoid variables with short names like d 16
Found 'DD'-anomaly for variable 'value' (lines '27'-'29'). 27–29