PMD Results

The following document contains the results of PMD 7.17.0.

Violations By Priority

Priority 2

inez.parser - io/github/magwas/inez/parse/ThrowingErrorListener.java

Rule	Violation	Line
SystemPrintln	Usage of System.out/err	32

Priority 3

inez.parser - io/github/magwas/inez/parse/ParseTextService.java

Rule	Violation	Line
CyclomaticComplexity	The method 'compileBridiFromTree(BridiContext)' has a cyclomatic complexity of 10.	37
UseConcurrentHashMap	If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation	40
AvoidLiteralsInIfCondition	Avoid using literals such as "{[" in if statements	47
AppendCharacterWithChar	Avoid appending characters as strings in StringBuffer.append.	47
AvoidLiteralsInIfCondition	Avoid using literals such as "]}" in if statements	48
ConsecutiveAppendsShouldReuse	StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable.	48
AppendCharacterWithChar	Avoid appending characters as strings in StringBuffer.append.	48
AssignmentInOperand	Avoid assignment to childIndex in operand	57
AssignmentInOperand	Avoid assignment to childIndex in operand	61
AssignmentInOperand	Avoid assignment to childIndex in operand	66

inez.parser - io/github/magwas/inez/parse/ThrowingErrorListener.java

Rule	Violation	Line
ImmutableField	Field 'input' may be declared final	13

inez.parser - io/github/magwas/inez/parser/BridiBaseListener.java

Rule	Violation	Line
CommentRequired	Class comments are unwanted	14
ClassesShouldBeKnown	Classes should be either Data, Service,Controller,Repository	14
TooManyMethods	This class has too many methods, consider refactoring it.	14–88
CommentRequired	Comments on @Override methods are unwanted	20
CommentRequired	Comments on @Override methods are unwanted	26
CommentRequired	Comments on @Override methods are unwanted	32
CommentRequired	Comments on @Override methods are unwanted	38
CommentRequired	Comments on @Override methods are unwanted	44
CommentRequired	Comments on @Override methods are unwanted	50
CommentRequired	Comments on @Override methods are unwanted	56
CommentRequired	Comments on @Override methods are unwanted	62
CommentRequired	Comments on @Override methods are unwanted	69
CommentRequired	Comments on @Override methods are unwanted	75
CommentRequired	Comments on @Override methods are unwanted	81
CommentRequired	Comments on @Override methods are unwanted	87

inez.parser - io/github/magwas/inez/parser/BridiListener.java

Rule	Violation	Line
CommentRequired	Class comments are unwanted	9
ClassesShouldBeKnown	Classes should be either Data, Service,Controller,Repository	9
CommentRequired	Public method and constructor comments are unwanted	14
CommentRequired	Public method and constructor comments are unwanted	19
CommentRequired	Public method and constructor comments are unwanted	24
CommentRequired	Public method and constructor comments are unwanted	29
CommentRequired	Public method and constructor comments are unwanted	34
CommentRequired	Public method and constructor comments are unwanted	39
CommentRequired	Public method and constructor comments are unwanted	44
CommentRequired	Public method and constructor comments are unwanted	49

Priority 4

inez.parser - io/github/magwas/inez/parser/BridiLexer.java

Rule	Violation	Line
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.Token'	5
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.TokenStream'	6
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.misc.*'	10

inez.parser - io/github/magwas/inez/parser/BridiParser.java

Rule	Violation	Line
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.misc.*'	6
UnnecessaryImport	Unused import 'java.util.Iterator'	9
UnnecessaryImport	Unused import 'java.util.ArrayList'	10

Files

inez.parser - io/github/magwas/inez/parse/ParseTextService.java

Rule	Violation	Priority	Line
CyclomaticComplexity	The method 'compileBridiFromTree(BridiContext)' has a cyclomatic complexity of 10.	3	37
UseConcurrentHashMap	If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation	3	40
AvoidLiteralsInIfCondition	Avoid using literals such as "{[" in if statements	3	47
AppendCharacterWithChar	Avoid appending characters as strings in StringBuffer.append.	3	47
AvoidLiteralsInIfCondition	Avoid using literals such as "]}" in if statements	3	48
ConsecutiveAppendsShouldReuse	StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable.	3	48
AppendCharacterWithChar	Avoid appending characters as strings in StringBuffer.append.	3	48
AssignmentInOperand	Avoid assignment to childIndex in operand	3	57
AssignmentInOperand	Avoid assignment to childIndex in operand	3	61
AssignmentInOperand	Avoid assignment to childIndex in operand	3	66

inez.parser - io/github/magwas/inez/parse/ThrowingErrorListener.java

Rule	Violation	Priority	Line
ImmutableField	Field 'input' may be declared final	3	13
SystemPrintln	Usage of System.out/err	2	32

inez.parser - io/github/magwas/inez/parser/BridiBaseListener.java

Rule	Violation	Priority	Line
CommentRequired	Class comments are unwanted	3	14
ClassesShouldBeKnown	Classes should be either Data, Service,Controller,Repository	3	14
TooManyMethods	This class has too many methods, consider refactoring it.	3	14–88
CommentRequired	Comments on @Override methods are unwanted	3	20
CommentRequired	Comments on @Override methods are unwanted	3	26
CommentRequired	Comments on @Override methods are unwanted	3	32
CommentRequired	Comments on @Override methods are unwanted	3	38
CommentRequired	Comments on @Override methods are unwanted	3	44
CommentRequired	Comments on @Override methods are unwanted	3	50
CommentRequired	Comments on @Override methods are unwanted	3	56
CommentRequired	Comments on @Override methods are unwanted	3	62
CommentRequired	Comments on @Override methods are unwanted	3	69
CommentRequired	Comments on @Override methods are unwanted	3	75
CommentRequired	Comments on @Override methods are unwanted	3	81
CommentRequired	Comments on @Override methods are unwanted	3	87

inez.parser - io/github/magwas/inez/parser/BridiLexer.java

Rule	Violation	Priority	Line
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.Token'	4	5
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.TokenStream'	4	6
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.misc.*'	4	10

inez.parser - io/github/magwas/inez/parser/BridiListener.java

Rule	Violation	Priority	Line
CommentRequired	Class comments are unwanted	3	9
ClassesShouldBeKnown	Classes should be either Data, Service,Controller,Repository	3	9
CommentRequired	Public method and constructor comments are unwanted	3	14
CommentRequired	Public method and constructor comments are unwanted	3	19
CommentRequired	Public method and constructor comments are unwanted	3	24
CommentRequired	Public method and constructor comments are unwanted	3	29
CommentRequired	Public method and constructor comments are unwanted	3	34
CommentRequired	Public method and constructor comments are unwanted	3	39
CommentRequired	Public method and constructor comments are unwanted	3	44
CommentRequired	Public method and constructor comments are unwanted	3	49

inez.parser - io/github/magwas/inez/parser/BridiParser.java

Rule	Violation	Priority	Line
UnnecessaryImport	Unused import 'org.antlr.v4.runtime.misc.*'	4	6
UnnecessaryImport	Unused import 'java.util.Iterator'	4	9
UnnecessaryImport	Unused import 'java.util.ArrayList'	4	10

Suppressed Violations

Filename	Rule message	Suppression type
io/github/magwas/inez/parser/BridiLexer.java	Classes should be either Data, Service,Controller,Repository	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Use one line for each declaration, it enhances code readability.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Parameter 'input' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Do not use non-final non-private static fields	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Do not use non-final non-private static fields	@suppresswarnings
io/github/magwas/inez/parser/BridiLexer.java	Field comments are unwanted	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Classes should be either Data, Service,Controller,Repository	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Classes should be either Data, Service,Controller,Repository	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Classes should be either Data, Service,Controller,Repository	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Classes should be either Data, Service,Controller,Repository	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Classes should be either Data, Service,Controller,Repository	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Multiple classes/records	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Switch statements or expressions should be exhaustive, add a default case (or missing enum branches)	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Use one line for each declaration, it enhances code readability.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Use one line for each declaration, it enhances code readability.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The value assigned to variable '_la' is never used (overwritten on line 134)	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The value assigned to variable '_la' is never used (overwritten on line 271)	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Avoid if (x != y) ..; else ..;	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_la' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_alt' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_localctx' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_localctx' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_la' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_localctx' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The local variable name '_localctx' doesn't match '[a-z][a-zA-Z0-9]*'	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'parent' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'invokingState' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'input' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'parent' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'invokingState' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'parent' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'invokingState' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'i' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'listener' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'parent' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Parameter 'invokingState' is not assigned and could be declared final	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'REF' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'TEXT' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'org.antlr.v4.runtime.atn': 'ATN' is already in scope because it is imported in this file	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'POST' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'NEWLINE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'PRE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'PRE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'POST' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'PRE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'TEXT' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'LITERALPRE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'TEXT' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'LITERALPRE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'TEXT' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'POST' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'POST' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'LITERALPOST' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'PRE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'TEXT' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'NEWLINE' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Unnecessary qualifier 'BridiParser': 'LITERALPOST' is already in scope because it is declared in an enclosing type	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Duplicate parentheses around `(_la & ~0x3f) == 0 && ((1L << _la) & 134L) != 0`.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Useless parentheses around `_la`.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Useless parentheses around `(_la & ~0x3f) == 0 && ((1L << _la) & 134L) != 0`.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Useless parentheses around `(_la & ~0x3f) == 0 && ((1L << _la) & 138L) != 0`.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Useless parentheses around `_la`.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Useless parentheses around `_la`.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	A method or constructor should not explicitly declare unchecked exception RecognitionException in its 'throws' clause	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	A method or constructor should not explicitly declare unchecked exception RecognitionException in its 'throws' clause	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	A method or constructor should not explicitly declare unchecked exception RecognitionException in its 'throws' clause	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	A method or constructor should not explicitly declare unchecked exception RecognitionException in its 'throws' clause	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	This class has too many methods, consider refactoring it.	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The class 'BridiParser' has a total cyclomatic complexity of 32 (highest 8).	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The class 'BridiContext' has a total cyclomatic complexity of 22 (highest 2).	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	Field comments are unwanted	@suppresswarnings
io/github/magwas/inez/parser/BridiParser.java	The String literal "CheckReturnValue" appears 5 times in this file; the first occurrence is on line 12	@suppresswarnings