PMD Results

The following document contains the results of PMD 4.2.5.

Files

cz/cuni/amis/pogamut/shady/IArgument.java

ViolationLine
Ensure you override both equals() and hashCode()31

cz/cuni/amis/pogamut/shady/ShadeEngine.java

ViolationLine
Avoid unused local variables such as 'call'.46
Avoid unused private fields such as 'name'.130
Avoid unused private fields such as 'ctx'.135

cz/cuni/amis/pogamut/shady/ShadeNode.java

ViolationLine
Avoid unused private fields such as 'descr'.14

cz/cuni/amis/pogamut/shady/ShadyParser.java

ViolationLine
An empty statement (semicolon) not part of a loop25
Do not use if statements that are always true or always false34
An empty statement (semicolon) not part of a loop53
Do not use if statements that are always true or always false61
Do not use if statements that are always true or always false74
Do not use if statements that are always true or always false81
Do not use if statements that are always true or always false94
Do not use if statements that are always true or always false98
Do not use if statements that are always true or always false105
Do not use if statements that are always true or always false111
An empty statement (semicolon) not part of a loop123
Do not use if statements that are always true or always false131
An empty statement (semicolon) not part of a loop143
Do not use if statements that are always true or always false151
Do not use if statements that are always true or always false158
Do not use if statements that are always true or always false165
Do not use if statements that are always true or always false172
Do not use if statements that are always true or always false179
Do not use if statements that are always true or always false186
Do not use if statements that are always true or always false193
Do not use if statements that are always true or always false215
Do not use if statements that are always true or always false219
Do not use if statements that are always true or always false223
Do not use if statements that are always true or always false242
Do not use if statements that are always true or always false253
An empty statement (semicolon) not part of a loop267
Do not use if statements that are always true or always false276
Do not use if statements that are always true or always false286
Do not use if statements that are always true or always false290
Do not use if statements that are always true or always false294
Do not use if statements that are always true or always false298
An empty statement (semicolon) not part of a loop320
Do not use if statements that are always true or always false329
Avoid empty catch blocks602

cz/cuni/amis/pogamut/shady/ShadyParserTokenManager.java

ViolationLine
These nested if statements could be combined218 - 219
Do not start a literal by 0 unless it's an octal value429
Do not start a literal by 0 unless it's an octal value533
Do not start a literal by 0 unless it's an octal value697

cz/cuni/amis/pogamut/shady/TokenMgrError.java

ViolationLine
Overriding method merely calls super124 - 126

cz/cuni/amis/pogamut/sposh/elements/ActionPattern.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.MissingParameterException'7
These nested if statements could be combined174 - 176

cz/cuni/amis/pogamut/sposh/elements/Adopt.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.Arguments.Argument'3
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.MissingParameterException'7
These nested if statements could be combined78 - 80

cz/cuni/amis/pogamut/sposh/elements/Competence.java

ViolationLine
These nested if statements could be combined189 - 191

cz/cuni/amis/pogamut/sposh/elements/LapChain.java

ViolationLine
Avoid empty if statements239 - 241

cz/cuni/amis/pogamut/sposh/elements/LapPath.java

ViolationLine
Avoid unused imports such as 'java.util.ArrayList'5
Avoid unused imports such as 'java.util.Collection'7

cz/cuni/amis/pogamut/sposh/elements/PoshElementListener.java

ViolationLine
Avoid unused imports such as 'java.awt.Desktop'3

cz/cuni/amis/pogamut/sposh/elements/PoshPlan.java

ViolationLine
Avoid unused private methods such as 'getReachableActionNames()'.87
Avoid unused method parameters such as 'c'.107 - 109
Avoid unused method parameters such as 'ap'.111 - 113
These nested if statements could be combined539 - 541
These nested if statements could be combined546 - 548
These nested if statements could be combined580 - 582
These nested if statements could be combined587 - 589

cz/cuni/amis/pogamut/sposh/elements/Result.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.executor.ParamInfo'4
Avoid unused imports such as 'java.util.regex.Matcher'6
Avoid unused imports such as 'java.util.regex.Pattern'7
Avoid empty catch blocks111 - 112
Avoid empty catch blocks116 - 117

cz/cuni/amis/pogamut/sposh/elements/Sense.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.InvalidFormatException'6

cz/cuni/amis/pogamut/sposh/elements/Trigger.java

ViolationLine
Avoid unused local variables such as 'removed'.188

cz/cuni/amis/pogamut/sposh/engine/AbstractExecutor.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.FubarException'11
Avoid unused method parameters such as 'plan'.117 - 132

cz/cuni/amis/pogamut/sposh/engine/ActionExecutor.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.FubarException'5

cz/cuni/amis/pogamut/sposh/engine/CEExecutor.java

ViolationLine
Avoid unused private fields such as 'competence'.17
Avoid unused private fields such as 'choice'.18

cz/cuni/amis/pogamut/sposh/engine/INodeExecutor.java

ViolationLine
Avoid unused local variables such as 'resultType'.138

cz/cuni/amis/pogamut/sposh/engine/PoshEngine.java

ViolationLine
Unnecessary final modifier in final class89 - 92
Unnecessary final modifier in final class161 - 163
Unnecessary final modifier in final class170 - 172

cz/cuni/amis/pogamut/sposh/engine/SenseExecutor.java

ViolationLine
Avoid unused local variables such as 'comparison'.127

cz/cuni/amis/pogamut/sposh/engine/SenseListExecutor.java

ViolationLine
Avoid unused imports such as 'java.util.logging.Logger'14

cz/cuni/amis/pogamut/sposh/executor/ParamInfo.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.Result'4
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.FubarException'5
Avoid unused imports such as 'java.util.Arrays'6
Avoid unused imports such as 'java.util.HashSet'7
Avoid unused imports such as 'java.util.Set'8
Avoid unused imports such as 'java.util.logging.Level'9
Avoid unused imports such as 'java.util.logging.Logger'10

cz/cuni/amis/pogamut/sposh/executor/ParamsMethod.java

ViolationLine
Avoid unused imports such as 'java.lang.reflect.Field'8
Avoid unused imports such as 'java.util.logging.Level'14
Avoid unused imports such as 'java.util.logging.Logger'15

cz/cuni/amis/pogamut/sposh/executor/PrimitiveData.java

ViolationLine
No need to import a type that lives in the same package3
No need to import a type that lives in the same package4
No need to import a type that lives in the same package5
No need to import a type that lives in the same package6

cz/cuni/amis/pogamut/sposh/elements/PoshParser.java

ViolationLine
Do not use if statements that are always true or always false53
Avoid unused local variables such as 'dc'.64
Do not use if statements that are always true or always false122
Do not use if statements that are always true or always false133
Do not use if statements that are always true or always false144
Do not use if statements that are always true or always false194
Do not use if statements that are always true or always false204
Do not use if statements that are always true or always false214
An empty statement (semicolon) not part of a loop264
An empty statement (semicolon) not part of a loop274
Do not use if statements that are always true or always false319
Do not use if statements that are always true or always false342
Do not use if statements that are always true or always false361
Do not use if statements that are always true or always false373
Do not use if statements that are always true or always false385
Do not use if statements that are always true or always false395
Do not use if statements that are always true or always false411
Do not use if statements that are always true or always false416
Do not use if statements that are always true or always false420
Do not use if statements that are always true or always false435
Do not use if statements that are always true or always false440
Do not use if statements that are always true or always false444
Avoid unused local variables such as '_comment'.456
An empty statement (semicolon) not part of a loop465
Do not use if statements that are always true or always false473
An empty statement (semicolon) not part of a loop492
Do not use if statements that are always true or always false505
Do not use if statements that are always true or always false509
Do not use if statements that are always true or always false522
Do not use if statements that are always true or always false558
Do not use if statements that are always true or always false568
Do not use if statements that are always true or always false577
An empty statement (semicolon) not part of a loop605
An empty statement (semicolon) not part of a loop615
Do not use if statements that are always true or always false622
Avoid unused local variables such as '_element'.630
An empty statement (semicolon) not part of a loop645
Do not use if statements that are always true or always false651
Do not use if statements that are always true or always false663
Do not use if statements that are always true or always false667
Do not use if statements that are always true or always false682
Do not use if statements that are always true or always false686
An empty statement (semicolon) not part of a loop707
Do not use if statements that are always true or always false715
An empty statement (semicolon) not part of a loop735
Do not use if statements that are always true or always false745
Avoid unused local variables such as '_actList'.756
An empty statement (semicolon) not part of a loop764
Do not use if statements that are always true or always false794
Do not use if statements that are always true or always false805
Do not use if statements that are always true or always false809
Do not use if statements that are always true or always false813
Do not use if statements that are always true or always false817
Do not use if statements that are always true or always false847
Do not use if statements that are always true or always false858
Do not use if statements that are always true or always false862
Do not use if statements that are always true or always false866
Do not use if statements that are always true or always false870
Do not use if statements that are always true or always false874
Do not use if statements that are always true or always false878
Do not use if statements that are always true or always false897
Do not use if statements that are always true or always false910
Do not use if statements that are always true or always false916
An empty statement (semicolon) not part of a loop940
Do not use if statements that are always true or always false947
Do not use if statements that are always true or always false963
Do not use if statements that are always true or always false967
Do not use if statements that are always true or always false988
Do not use if statements that are always true or always false990
An empty statement (semicolon) not part of a loop1012
Do not use if statements that are always true or always false1014
Do not use if statements that are always true or always false1034
An empty statement (semicolon) not part of a loop1040
Do not use if statements that are always true or always false1055
Do not use if statements that are always true or always false1059
Do not use if statements that are always true or always false1086
Do not use if statements that are always true or always false1088
Do not use if statements that are always true or always false1098
Do not use if statements that are always true or always false1110
An empty statement (semicolon) not part of a loop1120
Do not use if statements that are always true or always false1127
Do not use if statements that are always true or always false1129
Do not use if statements that are always true or always false1193
Do not use if statements that are always true or always false1197
Do not use if statements that are always true or always false1201
Do not use if statements that are always true or always false1205
Do not use if statements that are always true or always false1209
Do not use if statements that are always true or always false1213
Do not use if statements that are always true or always false1217

cz/cuni/amis/pogamut/sposh/elements/PoshParserTokenManager.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.shady.ArgString'3
Avoid unused imports such as 'java.util.Collections'4
Avoid unused imports such as 'java.util.LinkedList'5
Avoid unused imports such as 'java.util.List'6
Do not start a literal by 0 unless it's an octal value884
These nested if statements could be combined900 - 901
Do not start a literal by 0 unless it's an octal value1062
Do not start a literal by 0 unless it's an octal value1235
Do not start a literal by 0 unless it's an octal value1243

cz/cuni/amis/pogamut/sposh/elements/TokenMgrError.java

ViolationLine
Overriding method merely calls super124 - 126