PMD Results

The following document contains the results of PMD 6.4.0.

Files

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/HtmlTool.java

Violation Priority Line This class has a bunch of public methods and attributes 4 161529 This class has too many methods, consider refactoring it. 3 691529 Avoid really long classes. 3 691529 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 253 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 485 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 638 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 756 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 827 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 855 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 904 Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 4 945 No need to call String.valueOf to append to a string. 4 1255

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/JsoupUtils.java

Violation Priority Line Consider simply returning the value vs storing it in local variable 'doc' 4 70 Avoid using if statements without curly braces 1 9091 Consider simply returning the value vs storing it in local variable 'parser' 4 99

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/SkinConfigTool.java

Violation Priority Line This class has a bunch of public methods and attributes 4 161059 An empty statement (semicolon) not part of a loop 3 676

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/URITool.java

Violation Priority Line The utility class name 'URITool' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper)' 3 34185

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/model/Component.java

Violation Priority Line This abstract class does not have any abstract methods 3 35130

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/model/MenuItem.java

Violation Priority Line An empty statement (semicolon) not part of a loop 3 61

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/snippet/ComponentResolver.java

Violation Priority Line Avoid using if statements without curly braces 1 139140

Reflow Velocity Tools - org/devacfr/maven/skins/reflow/snippet/SnippetContext.java

Violation Priority Line Consider simply returning the value vs storing it in local variable 'parser' 4 108 Consider simply returning the value vs storing it in local variable 'context' 4 235