PMD Results
The following document contains the results of PMD 6.55.0.
Violations By Priority
Priority 3
org/basepom/mojo/dvc/HelpMojo.java
Priority 5
org/basepom/mojo/dvc/AbstractDependencyVersionsMojo.java
org/basepom/mojo/dvc/PluginLog.java
Rule |
Violation |
Line |
ProperLogger |
Logger should be defined private static final and have the correct class |
27 |
org/basepom/mojo/dvc/QualifiedName.java
Rule |
Violation |
Line |
TooManyMethods |
This class has too many methods, consider refactoring it. |
37–197 |
org/basepom/mojo/dvc/dependency/DependencyMapBuilder.java
Rule |
Violation |
Line |
CognitiveComplexity |
The method 'mapProject(MavenProject, DependencyFilter)' has a cognitive complexity of 16, current threshold is 15 |
93–163 |
CyclomaticComplexity |
The method 'mapProject(MavenProject, DependencyFilter)' has a cyclomatic complexity of 10. |
93–163 |
CognitiveComplexity |
The method 'loadDependencyTree(DependencyNode, DependencyFilter, ImmutableMap.Builder)' has a cognitive complexity of 21, current threshold is 15 |
165–196 |
org/basepom/mojo/dvc/dependency/DependencyTreeResolver.java
Rule |
Violation |
Line |
CognitiveComplexity |
The method 'computeResolutionMap(MavenProject, ScopeLimitingFilter)' has a cognitive complexity of 20, current threshold is 15 |
101–163 |
CyclomaticComplexity |
The method 'computeResolutionMap(MavenProject, ScopeLimitingFilter)' has a cyclomatic complexity of 11. |
101–163 |
AvoidCatchingThrowable |
A catch statement should never catch throwable since it includes errors. |
151–153 |
org/basepom/mojo/dvc/model/ResolverDefinition.java
org/basepom/mojo/dvc/mojo/DependencyVersionsCheckMojo.java
Rule |
Violation |
Line |
NcssCount |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a NCSS line count of 74. |
79–215 |
CognitiveComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a cognitive complexity of 48, current threshold is 15 |
79–215 |
CyclomaticComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a cyclomatic complexity of 25. |
79–215 |
NPathComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has an NPath complexity of 1145856, current threshold is 200 |
79–215 |
AvoidInstantiatingObjectsInLoops |
Avoid instantiating new objects inside loops |
136 |
org/basepom/mojo/dvc/mojo/DependencyVersionsListMojo.java
org/basepom/mojo/dvc/HelpMojo.java
Rule |
Violation |
Line |
TooManyMethods |
This class has too many methods, consider refactoring it. |
30–448 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' |
43 |
CyclomaticComplexity |
The method 'execute()' has a cyclomatic complexity of 11. |
97–160 |
NPathComplexity |
The method 'execute()' has an NPath complexity of 256, current threshold is 200 |
97–160 |
CognitiveComplexity |
The method 'writeGoal(StringBuilder, String, Element)' has a cognitive complexity of 15, current threshold is 15 |
219–257 |
CyclomaticComplexity |
The method 'writeGoal(StringBuilder, String, Element)' has a cyclomatic complexity of 11. |
219–257 |
CyclomaticComplexity |
The method 'writeParameter(StringBuilder, Node, Node)' has a cyclomatic complexity of 10. |
259–297 |
NPathComplexity |
The method 'writeParameter(StringBuilder, Node, Node)' has an NPath complexity of 216, current threshold is 200 |
259–297 |
Files
org/basepom/mojo/dvc/AbstractDependencyVersionsMojo.java
org/basepom/mojo/dvc/PluginLog.java
Rule |
Violation |
Priority |
Line |
ProperLogger |
Logger should be defined private static final and have the correct class |
5 |
27 |
org/basepom/mojo/dvc/QualifiedName.java
Rule |
Violation |
Priority |
Line |
TooManyMethods |
This class has too many methods, consider refactoring it. |
5 |
37–197 |
org/basepom/mojo/dvc/dependency/DependencyMapBuilder.java
Rule |
Violation |
Priority |
Line |
CognitiveComplexity |
The method 'mapProject(MavenProject, DependencyFilter)' has a cognitive complexity of 16, current threshold is 15 |
5 |
93–163 |
CyclomaticComplexity |
The method 'mapProject(MavenProject, DependencyFilter)' has a cyclomatic complexity of 10. |
5 |
93–163 |
CognitiveComplexity |
The method 'loadDependencyTree(DependencyNode, DependencyFilter, ImmutableMap.Builder)' has a cognitive complexity of 21, current threshold is 15 |
5 |
165–196 |
org/basepom/mojo/dvc/dependency/DependencyTreeResolver.java
Rule |
Violation |
Priority |
Line |
CognitiveComplexity |
The method 'computeResolutionMap(MavenProject, ScopeLimitingFilter)' has a cognitive complexity of 20, current threshold is 15 |
5 |
101–163 |
CyclomaticComplexity |
The method 'computeResolutionMap(MavenProject, ScopeLimitingFilter)' has a cyclomatic complexity of 11. |
5 |
101–163 |
AvoidCatchingThrowable |
A catch statement should never catch throwable since it includes errors. |
5 |
151–153 |
org/basepom/mojo/dvc/model/ResolverDefinition.java
org/basepom/mojo/dvc/mojo/DependencyVersionsCheckMojo.java
Rule |
Violation |
Priority |
Line |
NcssCount |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a NCSS line count of 74. |
5 |
79–215 |
CognitiveComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a cognitive complexity of 48, current threshold is 15 |
5 |
79–215 |
CyclomaticComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a cyclomatic complexity of 25. |
5 |
79–215 |
NPathComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has an NPath complexity of 1145856, current threshold is 200 |
5 |
79–215 |
AvoidInstantiatingObjectsInLoops |
Avoid instantiating new objects inside loops |
5 |
136 |
org/basepom/mojo/dvc/mojo/DependencyVersionsListMojo.java
Rule |
Violation |
Priority |
Line |
CognitiveComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a cognitive complexity of 32, current threshold is 15 |
5 |
56–155 |
CyclomaticComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has a cyclomatic complexity of 15. |
5 |
56–155 |
NPathComplexity |
The method 'doExecute(ImmutableSetMultimap, DependencyMap)' has an NPath complexity of 7936, current threshold is 200 |
5 |
56–155 |
AvoidInstantiatingObjectsInLoops |
Avoid instantiating new objects inside loops |
5 |
113 |
org/basepom/mojo/dvc/HelpMojo.java
Rule |
Violation |
Priority |
Line |
TooManyMethods |
This class has too many methods, consider refactoring it. |
5 |
30–448 |
UnnecessaryFullyQualifiedName |
Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' |
5 |
43 |
IdenticalCatchBranches |
'catch' branch identical to 'IOException' branch |
3 |
83–86 |
IdenticalCatchBranches |
'catch' branch identical to 'IOException' branch |
3 |
87–90 |
CyclomaticComplexity |
The method 'execute()' has a cyclomatic complexity of 11. |
5 |
97–160 |
NPathComplexity |
The method 'execute()' has an NPath complexity of 256, current threshold is 200 |
5 |
97–160 |
UseDiamondOperator |
Explicit type arguments can be replaced by Diamond Operator |
3 |
191 |
CognitiveComplexity |
The method 'writeGoal(StringBuilder, String, Element)' has a cognitive complexity of 15, current threshold is 15 |
5 |
219–257 |
CyclomaticComplexity |
The method 'writeGoal(StringBuilder, String, Element)' has a cyclomatic complexity of 11. |
5 |
219–257 |
CyclomaticComplexity |
The method 'writeParameter(StringBuilder, Node, Node)' has a cyclomatic complexity of 10. |
5 |
259–297 |
NPathComplexity |
The method 'writeParameter(StringBuilder, Node, Node)' has an NPath complexity of 216, current threshold is 200 |
5 |
259–297 |
UseDiamondOperator |
Explicit type arguments can be replaced by Diamond Operator |
3 |
348 |
LinguisticNaming |
Linguistics Antipattern - The transform method 'toLines' should not return void linguistically |
3 |
370–411 |
Suppressed Violations
Filename |
Rule message |
Suppression type |
Reason |
org/basepom/mojo/dvc/AbstractDependencyVersionsMojo.java |
A catch statement that catches an exception only to rethrow it should be avoided. |
annotation |
- |
org/basepom/mojo/dvc/CheckExclusionsFilter.java |
Avoid using a branching statement as the last in a loop. |
annotation |
- |
org/basepom/mojo/dvc/model/ResolverDefinition.java |
Consider using varargs for methods or constructors which take an array the last parameter. |
annotation |
- |