PMD Results

The following document contains the results of PMD 4.2.5.

Files

it/amattioli/encapsulate/browsers/CalendarBrowser.java

Violation Line
This class has too many methods, consider refactoring it. 22 - 115

it/amattioli/encapsulate/browsers/CalendarBrowserImpl.java

Violation Line
This class has too many methods, consider refactoring it. 21 - 157

it/amattioli/encapsulate/dates/ConventionalTimeInterval.java

Violation Line
This class has too many methods, consider refactoring it. 25 - 160

it/amattioli/encapsulate/dates/DateChecks.java

Violation Line
Too many static imports may lead to messy code 1 - 52

it/amattioli/encapsulate/dates/Day.java

Violation Line
This class has too many methods, consider refactoring it. 20 - 175

it/amattioli/encapsulate/dates/DurationFormat.java

Violation Line
The class 'DurationFormat' has a Cyclomatic Complexity of 9 (Highest = 16). 10 - 106
The method format() has an NPath complexity of 3125 14 - 52
The method 'format' has a Cyclomatic Complexity of 16. 14 - 52
Avoid using if statements without curly braces 18
Avoid using if statements without curly braces 21
Avoid using if statements without curly braces 25
Avoid using if statements without curly braces 28
Avoid using if statements without curly braces 32
Avoid using if statements without curly braces 35
Avoid using if statements without curly braces 39
Avoid using if statements without curly braces 42
Avoid using if statements without curly braces 46
Avoid using if statements without curly braces 49

it/amattioli/encapsulate/dates/PhysicalDuration.java

Violation Line
This class has too many methods, consider refactoring it. 10 - 218

it/amattioli/encapsulate/dates/hibernate/DayHibernateType.java

Violation Line
This class has too many methods, consider refactoring it. 16 - 107

it/amattioli/encapsulate/money/Discount.java

Violation Line
Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) 6
Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) 17
Ensure you override both equals() and hashCode() 38

it/amattioli/encapsulate/money/Euro.java

Violation Line
This class has too many methods, consider refactoring it. 12 - 112
Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) 15

it/amattioli/encapsulate/money/Money.java

Violation Line
This class has too many methods, consider refactoring it. 21 - 427
Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) 26
Avoid using for statements without curly braces 370 - 371

it/amattioli/encapsulate/money/SimpleAccount.java

Violation Line
Avoid empty catch blocks 22 - 24
Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) 81

it/amattioli/encapsulate/money/hibernate/EuroHibernateType.java

Violation Line
This class has too many methods, consider refactoring it. 16 - 105
Avoid using if statements without curly braces 38
Avoid using if statements without curly braces 39

it/amattioli/encapsulate/range/AbstractRange.java

Violation Line
The class 'AbstractRange' has a Cyclomatic Complexity of 3 (Highest = 16). 3 - 92
The method 'intersect' has a Cyclomatic Complexity of 16. 68 - 90

it/amattioli/encapsulate/range/ContinousRange.java

Violation Line
The class 'ContinousRange' has a Cyclomatic Complexity of 7 (Highest = 10). 6 - 67
The method 'gap' has a Cyclomatic Complexity of 10. 31 - 49
The method minus() has an NPath complexity of 729 52 - 65

it/amattioli/encapsulate/range/DiscreteRange.java

Violation Line
The class 'DiscreteRange' has a Cyclomatic Complexity of 6 (Highest = 11). 5 - 73
The method gap() has an NPath complexity of 244 30 - 51
The method 'gap' has a Cyclomatic Complexity of 11. 30 - 51
The method minus() has an NPath complexity of 729 58 - 71

it/amattioli/encapsulate/range/Range.java

Violation Line
This class has too many methods, consider refactoring it. 15 - 134