PMD Results
The following document contains the results of PMD 6.47.0.
Violations By Priority
Priority 3
ru/akman/maven/plugins/BaseToolMojo.java
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 17–979 |
| CyclomaticComplexity | The class 'BaseToolMojo' has a total cyclomatic complexity of 136 (highest 33). | 53–979 |
| TooManyFields | Too many fields | 53–979 |
| GodClass | Possible God Class (WMC=136, ATFD=44, TCC=4.545%) | 53–979 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 118 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 123 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 128 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 133 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 138 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 143 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 149 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 155 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 160 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 165 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 170 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 175 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 181 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 187 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 197 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 207 |
| NPathComplexity | The method 'getToolExecutablePath(String, File, String)' has an NPath complexity of 256, current threshold is 200 | 386–422 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 391 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 399 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 407 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 415 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 436 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 443 |
| CognitiveComplexity | The method 'findToolExecutable(String, List)' has a cognitive complexity of 16, current threshold is 15 | 456–481 |
| UnusedAssignment | The initializer for variable 'toolFile' is never used (overwritten on lines 464 and 472) | 459 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 502 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 522–525 |
| CyclomaticComplexity | The method 'getCorrespondingJavaVersion(String)' has a cyclomatic complexity of 11. | 560–593 |
| CognitiveComplexity | The method 'getCorrespondingJavaVersion(String)' has a cognitive complexity of 21, current threshold is 15 | 560–593 |
| CognitiveComplexity | The method 'logCommandLineExecution(Commandline, int, String, String)' has a cognitive complexity of 18, current threshold is 15 | 618–643 |
| ExcessiveMethodLength | Avoid really long methods. | 849–977 |
| NcssCount | The method 'init(String, File, String)' has a NCSS line count of 60. | 849–977 |
| CognitiveComplexity | The method 'init(String, File, String)' has a cognitive complexity of 38, current threshold is 15 | 849–977 |
| CyclomaticComplexity | The method 'init(String, File, String)' has a cyclomatic complexity of 33. | 849–977 |
| NPathComplexity | The method 'init(String, File, String)' has an NPath complexity of 786432, current threshold is 200 | 849–977 |
ru/akman/maven/plugins/CommandLineBuilder.java
| Rule | Violation | Line |
|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 33 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 38 |
ru/akman/maven/plugins/CommandLineOption.java
| Rule | Violation | Line |
|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 34 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 39 |
ru/akman/maven/plugins/jlink/Compress.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'Compress' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=4) | 24–72 |
ru/akman/maven/plugins/jlink/DependencySet.java
| Rule | Violation | Line |
|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 31 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 38 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 48 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 58 |
ru/akman/maven/plugins/jlink/JlinkMojo.java
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 17–1810 |
| TooManyFields | Too many fields | 102–1810 |
| GodClass | Possible God Class (WMC=220, ATFD=169, TCC=15.152%) | 102–1810 |
| ExcessiveClassLength | Avoid really long classes. | 102–1810 |
| CyclomaticComplexity | The class 'JlinkMojo' has a total cyclomatic complexity of 220 (highest 57). | 102–1810 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 102–1810 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 133 |
| LongVariable | Avoid excessively long variable names like projectDependencies | 138 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 138 |
| LongVariable | Avoid excessively long variable names like mainModuleDescriptor | 143 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 143 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 149 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 155 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 163 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 171 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 247 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 263 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 273 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 294 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 309 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 317 |
| LongVariable | Avoid excessively long variable names like resourceslastsorter | 325 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 325 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 333 |
| AvoidDuplicateLiterals | The String literal "false" appears 9 times in this file; the first occurrence is on line 341 | 341 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 343 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 353 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 370 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 380 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 390 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 400 |
| LongVariable | Avoid excessively long variable names like ignoresigninginformation | 412 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 412 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 429 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 466 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 486 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 503 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 520 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 530 |
| LongVariable | Avoid excessively long variable names like stripjavadebugattributes | 540 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 540 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 550 |
| LongVariable | Avoid excessively long variable names like stripnativecommands | 550 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 563 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 580 |
| LongVariable | Avoid excessively long variable names like excludejmodsection | 588 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 588 |
| LongVariable | Avoid excessively long variable names like generatejliclasses | 601 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 601 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 628 |
| ShortVariable | Avoid variables with short names like vm | 651 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 651 |
| CognitiveComplexity | The method 'getFileSets()' has a cognitive complexity of 16, current threshold is 15 | 865–892 |
| CognitiveComplexity | The method 'getDirSets()' has a cognitive complexity of 16, current threshold is 15 | 902–929 |
| CyclomaticComplexity | The method 'filterDependency(DependencySet, File, JavaModuleDescriptor)' has a cyclomatic complexity of 20. | 1004–1065 |
| CognitiveComplexity | The method 'filterDependency(DependencySet, File, JavaModuleDescriptor)' has a cognitive complexity of 45, current threshold is 15 | 1004–1065 |
| NPathComplexity | The method 'filterDependency(DependencySet, File, JavaModuleDescriptor)' has an NPath complexity of 210, current threshold is 200 | 1004–1065 |
| UnusedAssignment | The initializer for variable 'result' is never used (overwritten on lines 1090, 1092, 1096 and 1098) | 1086 |
| UnusedAssignment | The initializer for variable 'result' is never used (overwritten on lines 1128, 1130, 1134 and 1136) | 1124 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 1159 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 1180 |
| CyclomaticComplexity | The method 'processModules(CommandLineBuilder)' has a cyclomatic complexity of 15. | 1193–1243 |
| NPathComplexity | The method 'processModules(CommandLineBuilder)' has an NPath complexity of 1308, current threshold is 200 | 1193–1243 |
| CognitiveComplexity | The method 'processModules(CommandLineBuilder)' has a cognitive complexity of 26, current threshold is 15 | 1193–1243 |
| UnusedAssignment | The initializer for variable 'opt' is never used (overwritten on lines 1225 and 1238) | 1195 |
| NcssCount | The method 'processOptions(CommandLineBuilder)' has a NCSS line count of 135. | 1252–1500 |
| NPathComplexity | The method 'processOptions(CommandLineBuilder)' has an NPath complexity of 2147483647, current threshold is 200 | 1252–1500 |
| ExcessiveMethodLength | Avoid really long methods. | 1252–1500 |
| CognitiveComplexity | The method 'processOptions(CommandLineBuilder)' has a cognitive complexity of 79, current threshold is 15 | 1252–1500 |
| CyclomaticComplexity | The method 'processOptions(CommandLineBuilder)' has a cyclomatic complexity of 57. | 1252–1500 |
| UnusedAssignment | The initializer for variable 'opt' is never used (overwritten on line 1256) | 1254 |
| CognitiveComplexity | The method 'copyFiles(List, File)' has a cognitive complexity of 18, current threshold is 15 | 1510–1535 |
| CyclomaticComplexity | The method 'processLauncherScripts()' has a cyclomatic complexity of 21. | 1542–1634 |
| CognitiveComplexity | The method 'processLauncherScripts()' has a cognitive complexity of 30, current threshold is 15 | 1542–1634 |
| NPathComplexity | The method 'processLauncherScripts()' has an NPath complexity of 13312, current threshold is 200 | 1542–1634 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 1544 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 1549 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 1577 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1615 |
| NPathComplexity | The method 'execute()' has an NPath complexity of 13824, current threshold is 200 | 1681–1808 |
| CognitiveComplexity | The method 'execute()' has a cognitive complexity of 18, current threshold is 15 | 1681–1808 |
| ExcessiveMethodLength | Avoid really long methods. | 1681–1808 |
| CyclomaticComplexity | The method 'execute()' has a cyclomatic complexity of 25. | 1681–1808 |
| LocalVariableCouldBeFinal | Local variable 'classpathElements' could be declared final | 1728 |
| LongVariable | Avoid excessively long variable names like modulepathElements | 1729 |
| LocalVariableCouldBeFinal | Local variable 'modulepathElements' could be declared final | 1729 |
| LocalVariableCouldBeFinal | Local variable 'pathExceptions' could be declared final | 1730 |
| UnusedAssignment | The initializer for variable 'exitCode' is never used (overwritten on line 1779) | 1777 |
ru/akman/maven/plugins/jlink/Launcher.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'Launcher' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=14, WMC=14) | 24–187 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 34 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 39 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 44 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 54 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 59 |
ru/akman/maven/plugins/jlink/ModulePath.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'ModulePath' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=8, WMC=8) | 26–120 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 31 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 36 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 41 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 46 |
ru/akman/maven/plugins/jlink/ReleaseInfo.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'ReleaseInfo' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=6) | 25–96 |
ru/akman/maven/plugins/jlink/JlinkMojoTest.java
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 17–1141 |
| TooManyStaticImports | Too many static imports may lead to messy code | 17–1141 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 45–1141 |
| ExcessiveClassLength | Avoid really long classes. | 45–1141 |
| SingularField | Perhaps 'container' could be replaced by a local variable. | 60 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 60 |
| SingularField | Perhaps 'toolchainManager' could be replaced by a local variable. | 65 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 65 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 70 |
| SingularField | Perhaps 'session' could be replaced by a local variable. | 75 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 75 |
| SingularField | Perhaps 'execution' could be replaced by a local variable. | 80 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 80 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 85 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 92 |
| AvoidDuplicateLiterals | The String literal "modulepath" appears 20 times in this file; the first occurrence is on line 184 | 184 |
| AvoidDuplicateLiterals | The String literal "unchecked" appears 8 times in this file; the first occurrence is on line 520 | 520 |
| LongVariable | Avoid excessively long variable names like resourceslastsorter | 605 |
| AvoidDuplicateLiterals | The String literal "launcher" appears 9 times in this file; the first occurrence is on line 663 | 663 |
| LongVariable | Avoid excessively long variable names like ignoresigninginformation | 826 |
| AvoidDuplicateLiterals | The String literal "compress" appears 5 times in this file; the first occurrence is on line 846 | 846 |
| AvoidDuplicateLiterals | The String literal "**/*-info.class" appears 4 times in this file; the first occurrence is on line 892 | 892 |
| AvoidDuplicateLiterals | The String literal "regex:/java[a-z]+$" appears 4 times in this file; the first occurrence is on line 894 | 894 |
| AvoidDuplicateLiterals | The String literal "@filename" appears 4 times in this file; the first occurrence is on line 895 | 895 |
| LongVariable | Avoid excessively long variable names like stripjavadebugattributes | 979 |
| LongVariable | Avoid excessively long variable names like stripnativecommands | 993 |
| LongVariable | Avoid excessively long variable names like excludejmodsection | 1040 |
| LongVariable | Avoid excessively long variable names like generatejliclasses | 1055 |
| AvoidDuplicateLiterals | The String literal "releaseinfo" appears 5 times in this file; the first occurrence is on line 1 071 | 1071 |
| ShortVariable | Avoid variables with short names like vm | 1133 |
Files
ru/akman/maven/plugins/BaseToolMojo.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 17–979 |
| CyclomaticComplexity | The class 'BaseToolMojo' has a total cyclomatic complexity of 136 (highest 33). | 3 | 53–979 |
| TooManyFields | Too many fields | 3 | 53–979 |
| GodClass | Possible God Class (WMC=136, ATFD=44, TCC=4.545%) | 3 | 53–979 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 118 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 123 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 128 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 133 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 138 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 143 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 149 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 155 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 160 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 165 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 170 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 175 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 181 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 187 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 197 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 207 |
| NPathComplexity | The method 'getToolExecutablePath(String, File, String)' has an NPath complexity of 256, current threshold is 200 | 3 | 386–422 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 391 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 399 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 407 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 415 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 436 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 443 |
| CognitiveComplexity | The method 'findToolExecutable(String, List)' has a cognitive complexity of 16, current threshold is 15 | 3 | 456–481 |
| UnusedAssignment | The initializer for variable 'toolFile' is never used (overwritten on lines 464 and 472) | 3 | 459 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 502 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 522–525 |
| CyclomaticComplexity | The method 'getCorrespondingJavaVersion(String)' has a cyclomatic complexity of 11. | 3 | 560–593 |
| CognitiveComplexity | The method 'getCorrespondingJavaVersion(String)' has a cognitive complexity of 21, current threshold is 15 | 3 | 560–593 |
| CognitiveComplexity | The method 'logCommandLineExecution(Commandline, int, String, String)' has a cognitive complexity of 18, current threshold is 15 | 3 | 618–643 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 849–977 |
| NcssCount | The method 'init(String, File, String)' has a NCSS line count of 60. | 3 | 849–977 |
| CognitiveComplexity | The method 'init(String, File, String)' has a cognitive complexity of 38, current threshold is 15 | 3 | 849–977 |
| CyclomaticComplexity | The method 'init(String, File, String)' has a cyclomatic complexity of 33. | 3 | 849–977 |
| NPathComplexity | The method 'init(String, File, String)' has an NPath complexity of 786432, current threshold is 200 | 3 | 849–977 |
ru/akman/maven/plugins/CommandLineBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 33 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 38 |
ru/akman/maven/plugins/CommandLineOption.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 34 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 39 |
ru/akman/maven/plugins/jlink/Compress.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'Compress' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=4) | 3 | 24–72 |
ru/akman/maven/plugins/jlink/DependencySet.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 31 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 38 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 48 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 58 |
ru/akman/maven/plugins/jlink/JlinkMojo.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 17–1810 |
| TooManyFields | Too many fields | 3 | 102–1810 |
| GodClass | Possible God Class (WMC=220, ATFD=169, TCC=15.152%) | 3 | 102–1810 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 102–1810 |
| CyclomaticComplexity | The class 'JlinkMojo' has a total cyclomatic complexity of 220 (highest 57). | 3 | 102–1810 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 102–1810 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 133 |
| LongVariable | Avoid excessively long variable names like projectDependencies | 3 | 138 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 138 |
| LongVariable | Avoid excessively long variable names like mainModuleDescriptor | 3 | 143 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 143 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 149 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 155 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 163 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 171 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 247 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 263 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 273 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 294 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 309 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 317 |
| LongVariable | Avoid excessively long variable names like resourceslastsorter | 3 | 325 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 325 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 333 |
| AvoidDuplicateLiterals | The String literal "false" appears 9 times in this file; the first occurrence is on line 341 | 3 | 341 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 343 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 353 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 370 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 380 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 390 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 400 |
| LongVariable | Avoid excessively long variable names like ignoresigninginformation | 3 | 412 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 412 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 429 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 466 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 486 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 503 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 520 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 530 |
| LongVariable | Avoid excessively long variable names like stripjavadebugattributes | 3 | 540 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 540 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 550 |
| LongVariable | Avoid excessively long variable names like stripnativecommands | 3 | 550 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 563 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 580 |
| LongVariable | Avoid excessively long variable names like excludejmodsection | 3 | 588 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 588 |
| LongVariable | Avoid excessively long variable names like generatejliclasses | 3 | 601 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 601 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 628 |
| ShortVariable | Avoid variables with short names like vm | 3 | 651 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 651 |
| CognitiveComplexity | The method 'getFileSets()' has a cognitive complexity of 16, current threshold is 15 | 3 | 865–892 |
| CognitiveComplexity | The method 'getDirSets()' has a cognitive complexity of 16, current threshold is 15 | 3 | 902–929 |
| CyclomaticComplexity | The method 'filterDependency(DependencySet, File, JavaModuleDescriptor)' has a cyclomatic complexity of 20. | 3 | 1004–1065 |
| CognitiveComplexity | The method 'filterDependency(DependencySet, File, JavaModuleDescriptor)' has a cognitive complexity of 45, current threshold is 15 | 3 | 1004–1065 |
| NPathComplexity | The method 'filterDependency(DependencySet, File, JavaModuleDescriptor)' has an NPath complexity of 210, current threshold is 200 | 3 | 1004–1065 |
| UnusedAssignment | The initializer for variable 'result' is never used (overwritten on lines 1090, 1092, 1096 and 1098) | 3 | 1086 |
| UnusedAssignment | The initializer for variable 'result' is never used (overwritten on lines 1128, 1130, 1134 and 1136) | 3 | 1124 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 1159 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 1180 |
| CyclomaticComplexity | The method 'processModules(CommandLineBuilder)' has a cyclomatic complexity of 15. | 3 | 1193–1243 |
| NPathComplexity | The method 'processModules(CommandLineBuilder)' has an NPath complexity of 1308, current threshold is 200 | 3 | 1193–1243 |
| CognitiveComplexity | The method 'processModules(CommandLineBuilder)' has a cognitive complexity of 26, current threshold is 15 | 3 | 1193–1243 |
| UnusedAssignment | The initializer for variable 'opt' is never used (overwritten on lines 1225 and 1238) | 3 | 1195 |
| NcssCount | The method 'processOptions(CommandLineBuilder)' has a NCSS line count of 135. | 3 | 1252–1500 |
| NPathComplexity | The method 'processOptions(CommandLineBuilder)' has an NPath complexity of 2147483647, current threshold is 200 | 3 | 1252–1500 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 1252–1500 |
| CognitiveComplexity | The method 'processOptions(CommandLineBuilder)' has a cognitive complexity of 79, current threshold is 15 | 3 | 1252–1500 |
| CyclomaticComplexity | The method 'processOptions(CommandLineBuilder)' has a cyclomatic complexity of 57. | 3 | 1252–1500 |
| UnusedAssignment | The initializer for variable 'opt' is never used (overwritten on line 1256) | 3 | 1254 |
| CognitiveComplexity | The method 'copyFiles(List, File)' has a cognitive complexity of 18, current threshold is 15 | 3 | 1510–1535 |
| CyclomaticComplexity | The method 'processLauncherScripts()' has a cyclomatic complexity of 21. | 3 | 1542–1634 |
| CognitiveComplexity | The method 'processLauncherScripts()' has a cognitive complexity of 30, current threshold is 15 | 3 | 1542–1634 |
| NPathComplexity | The method 'processLauncherScripts()' has an NPath complexity of 13312, current threshold is 200 | 3 | 1542–1634 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 1544 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 1549 |
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 1577 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1615 |
| NPathComplexity | The method 'execute()' has an NPath complexity of 13824, current threshold is 200 | 3 | 1681–1808 |
| CognitiveComplexity | The method 'execute()' has a cognitive complexity of 18, current threshold is 15 | 3 | 1681–1808 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 1681–1808 |
| CyclomaticComplexity | The method 'execute()' has a cyclomatic complexity of 25. | 3 | 1681–1808 |
| LocalVariableCouldBeFinal | Local variable 'classpathElements' could be declared final | 3 | 1728 |
| LongVariable | Avoid excessively long variable names like modulepathElements | 3 | 1729 |
| LocalVariableCouldBeFinal | Local variable 'modulepathElements' could be declared final | 3 | 1729 |
| LocalVariableCouldBeFinal | Local variable 'pathExceptions' could be declared final | 3 | 1730 |
| UnusedAssignment | The initializer for variable 'exitCode' is never used (overwritten on line 1779) | 3 | 1777 |
ru/akman/maven/plugins/jlink/Launcher.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'Launcher' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=14, WMC=14) | 3 | 24–187 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 34 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 39 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 44 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 54 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 59 |
ru/akman/maven/plugins/jlink/ModulePath.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'ModulePath' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=8, WMC=8) | 3 | 26–120 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 31 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 36 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 41 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 46 |
ru/akman/maven/plugins/jlink/ReleaseInfo.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'ReleaseInfo' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=6) | 3 | 25–96 |
ru/akman/maven/plugins/jlink/JlinkMojoTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 17–1141 |
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 17–1141 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 45–1141 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 45–1141 |
| SingularField | Perhaps 'container' could be replaced by a local variable. | 3 | 60 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 60 |
| SingularField | Perhaps 'toolchainManager' could be replaced by a local variable. | 3 | 65 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 65 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 70 |
| SingularField | Perhaps 'session' could be replaced by a local variable. | 3 | 75 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 75 |
| SingularField | Perhaps 'execution' could be replaced by a local variable. | 3 | 80 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 80 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 85 |
| BeanMembersShouldSerialize | Found non-transient, non-static member. Please mark as transient or provide accessors. | 3 | 92 |
| AvoidDuplicateLiterals | The String literal "modulepath" appears 20 times in this file; the first occurrence is on line 184 | 3 | 184 |
| AvoidDuplicateLiterals | The String literal "unchecked" appears 8 times in this file; the first occurrence is on line 520 | 3 | 520 |
| LongVariable | Avoid excessively long variable names like resourceslastsorter | 3 | 605 |
| AvoidDuplicateLiterals | The String literal "launcher" appears 9 times in this file; the first occurrence is on line 663 | 3 | 663 |
| LongVariable | Avoid excessively long variable names like ignoresigninginformation | 3 | 826 |
| AvoidDuplicateLiterals | The String literal "compress" appears 5 times in this file; the first occurrence is on line 846 | 3 | 846 |
| AvoidDuplicateLiterals | The String literal "**/*-info.class" appears 4 times in this file; the first occurrence is on line 892 | 3 | 892 |
| AvoidDuplicateLiterals | The String literal "regex:/java[a-z]+$" appears 4 times in this file; the first occurrence is on line 894 | 3 | 894 |
| AvoidDuplicateLiterals | The String literal "@filename" appears 4 times in this file; the first occurrence is on line 895 | 3 | 895 |
| LongVariable | Avoid excessively long variable names like stripjavadebugattributes | 3 | 979 |
| LongVariable | Avoid excessively long variable names like stripnativecommands | 3 | 993 |
| LongVariable | Avoid excessively long variable names like excludejmodsection | 3 | 1040 |
| LongVariable | Avoid excessively long variable names like generatejliclasses | 3 | 1055 |
| AvoidDuplicateLiterals | The String literal "releaseinfo" appears 5 times in this file; the first occurrence is on line 1 071 | 3 | 1071 |
| ShortVariable | Avoid variables with short names like vm | 3 | 1133 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| ru/akman/maven/plugins/jlink/JlinkMojo.java | Avoid unused private methods such as 'getExcludedDependencies(DependencySet)'. | annotation |