Showing 156 of 156 total issues
Avoid too many return statements within this method. Open
return new ConcurrentHashMapShufflingAdder(cv);Method printStackTraceIfUniqueDebugPoint has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
private void printStackTraceIfUniqueDebugPoint() {
if (config.shouldPrintStackTrace
&& this.isDebuggingUniquePoint()) {
StackTraceElement[] traces = Thread.currentThread().getStackTrace();
StringBuilder stackstring = new StringBuilder();- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method addExcludedGroups has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
private Xpp3Dom addExcludedGroups(Xpp3Dom configNode) {
for (Xpp3Dom config : configNode.getChildren()) {
if ("excludedGroups".equals(config.getName())) {
Logger.getGlobal().log(Level.INFO, "Adding excluded groups to existing ones");
String current = config.getValue();- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method debug has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
public String debug() throws MojoExecutionException {
//The test must have failed if it's being debugged, ergo there should exist a failing configuration
assert (!this.failingConfigurations.isEmpty());
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Method execute has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring. Open
@Override
public void execute() throws MojoExecutionException, MojoFailureException {
super.execute();
Logger.getGlobal().log(Level.INFO, "The original argline is: " + this.originalArgLine);
MojoExecutionException allExceptions = null;- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
TODO found Open
// TODO(gyori): Fix to check that the id was not used before in the- Exclude checks
TODO found Open
// TODO: pick a class with annotations and test:- Exclude checks
TODO found Open
// TODO(gyori): Make this parameterized and run with all- Exclude checks
TODO found Open
// TODO(gyori): Make this parameterized and run with all- Exclude checks
TODO found Open
// TODO: Temporary solution to prevent JVM crash. It's not safe to invoke isBooted()- Exclude checks
TODO found Open
// TODO(gyori): refactor this crap.- Exclude checks
TODO found Open
// TODO(gyori): fix and make this more robust. It does not check duplicates, substrings, etc.- Exclude checks
TODO found Open
// TODO: pick a class with annotations and test:- Exclude checks
TODO found Open
// TODO(gyori): fix and make this more robust. It does not check duplicates, substrings, etc.- Exclude checks
TODO found Open
// TODO(gyori): Make this parameterized and run with all- Exclude checks
TODO found Open
// TODO(gyori): Make this parameterized and run with all- Exclude checks