PMD Results
The following document contains the results of PMD 5.6.1.
Files
com/damnhandy/uri/template/DefaultVarExploder.java
| Violation |
Line |
| Overridable method 'setSource' called during object construction |
94 |
| These nested if statements could be combined |
183–186 |
| Use opposite operator instead of the logic complement operator. |
183 |
| Avoid using if statements without curly braces |
219–220 |
| Deeply nested if..then statements are hard to read |
255–260 |
com/damnhandy/uri/template/Expression.java
| Violation |
Line |
| Overridable method 'toString' called during object construction |
283 |
| Private field 'operator' could be made final; it is only initialized in the declaration or constructor. |
511 |
| Private field 'varSpecs' could be made final; it is only initialized in the declaration or constructor. |
516 |
com/damnhandy/uri/template/Literal.java
| Violation |
Line |
| Overridable method 'getValue' called during object construction |
34 |
com/damnhandy/uri/template/UriTemplate.java
| Violation |
Line |
| Possible God class (WMC=145, ATFD=88, TCC=0.02217741935483871) |
16–1159 |
| Avoid if (x != y) ..; else ..; |
588–598 |
| These nested if statements could be combined |
623–644 |
| Avoid unnecessary if..then..else statements when returning booleans |
739–742 |
| Avoid unnecessary if..then..else statements when returning booleans |
766–777 |
| Avoid unnecessary if..then..else statements when returning booleans |
851–854 |
| Avoid reassigning parameters such as 'variable' |
936 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
1037 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
1068 |
| Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() |
1081 |
com/damnhandy/uri/template/UriTemplateBuilderException.java
| Violation |
Line |
| Document empty constructor |
24–27 |
com/damnhandy/uri/template/impl/UriTemplateParser.java
| Violation |
Line |
| Private field 'components' could be made final; it is only initialized in the declaration or constructor. |
45 |
com/damnhandy/uri/template/impl/VarSpec.java
| Violation |
Line |
| Avoid if (x != y) ..; else ..; |
172–195 |