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/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/shady/engine/ShadeEngine.java

ViolationLine
Avoid unused local variables such as 'call'.51
Avoid unused private fields such as 'name'.135
Avoid unused private fields such as 'ctx'.140

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

ViolationLine
These nested if statements could be combined174 - 176
These nested if statements could be combined300 - 302

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

ViolationLine
These nested if statements could be combined64 - 66

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

ViolationLine
These nested if statements could be combined236 - 238

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

ViolationLine
These nested if statements could be combined170 - 172
These nested if statements could be combined259 - 261
Unnecessary final modifier in final class279 - 300

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

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

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 combined515 - 517
These nested if statements could be combined522 - 524
These nested if statements could be combined556 - 558
These nested if statements could be combined563 - 565

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

ViolationLine
Avoid unused imports such as 'java.util.logging.Level'5
Avoid unused imports such as 'java.util.logging.Logger'6
Avoid empty catch blocks107 - 108
Avoid empty catch blocks112 - 113

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.exceptions.InvalidFormatException'3
Avoid unused imports such as 'java.util.LinkedList'8

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

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

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.engine.VariableContext'3
Avoid unused imports such as 'java.util.LinkedList'12
Avoid unused imports such as 'java.util.logging.Level'13
Avoid unused imports such as 'java.util.logging.Logger'14
Avoid unused private methods such as 'getElementParameters(PoshElement)'.128

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.Competence'6

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.engine.FireResult.Type'11

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.engine.FireResult.Type'6
Avoid unused private fields such as 'runningOnceResult'.19

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.ActionPattern'5
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.Adopt'6
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.Competence'7
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.TriggeredAction'11

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

ViolationLine
Avoid unused imports such as 'java.util.Collections'7

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

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.ActionPattern'7
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.Competence'8
Avoid unused imports such as 'cz.cuni.amis.pogamut.sposh.elements.TriggeredAction'13

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

ViolationLine
Avoid unused imports such as 'java.util.Set'7

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

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

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

ViolationLine
Avoid unused imports such as 'java.util.logging.Level'12
Avoid unused imports such as 'java.util.logging.Logger'13

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 false386
Do not use if statements that are always true or always false396
Do not use if statements that are always true or always false412
Do not use if statements that are always true or always false417
Do not use if statements that are always true or always false421
Do not use if statements that are always true or always false436
Do not use if statements that are always true or always false441
Do not use if statements that are always true or always false445
Avoid unused local variables such as '_comment'.457
An empty statement (semicolon) not part of a loop466
Do not use if statements that are always true or always false474
An empty statement (semicolon) not part of a loop493
Do not use if statements that are always true or always false506
Do not use if statements that are always true or always false510
Do not use if statements that are always true or always false523
Do not use if statements that are always true or always false559
Do not use if statements that are always true or always false569
Do not use if statements that are always true or always false578
An empty statement (semicolon) not part of a loop606
An empty statement (semicolon) not part of a loop616
Do not use if statements that are always true or always false623
Avoid unused local variables such as '_element'.631
An empty statement (semicolon) not part of a loop646
Do not use if statements that are always true or always false652
Do not use if statements that are always true or always false664
Do not use if statements that are always true or always false668
Do not use if statements that are always true or always false683
Do not use if statements that are always true or always false687
An empty statement (semicolon) not part of a loop708
Do not use if statements that are always true or always false716
An empty statement (semicolon) not part of a loop736
Do not use if statements that are always true or always false746
Avoid unused local variables such as '_actList'.757
An empty statement (semicolon) not part of a loop765
Do not use if statements that are always true or always false795
Do not use if statements that are always true or always false806
Do not use if statements that are always true or always false810
Do not use if statements that are always true or always false814
Do not use if statements that are always true or always false818
Do not use if statements that are always true or always false848
Do not use if statements that are always true or always false859
Do not use if statements that are always true or always false863
Do not use if statements that are always true or always false867
Do not use if statements that are always true or always false871
Do not use if statements that are always true or always false875
Do not use if statements that are always true or always false879
Do not use if statements that are always true or always false898
Do not use if statements that are always true or always false911
Do not use if statements that are always true or always false917
An empty statement (semicolon) not part of a loop941
Do not use if statements that are always true or always false948
Do not use if statements that are always true or always false964
Do not use if statements that are always true or always false968
Do not use if statements that are always true or always false989
Do not use if statements that are always true or always false991
An empty statement (semicolon) not part of a loop1013
Do not use if statements that are always true or always false1015
Do not use if statements that are always true or always false1035
An empty statement (semicolon) not part of a loop1041
Do not use if statements that are always true or always false1056
Do not use if statements that are always true or always false1060
Do not use if statements that are always true or always false1088
Do not use if statements that are always true or always false1090
Do not use if statements that are always true or always false1100
Do not use if statements that are always true or always false1113
An empty statement (semicolon) not part of a loop1123
Do not use if statements that are always true or always false1130
Do not use if statements that are always true or always false1132
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
Do not use if statements that are always true or always false1221
Do not use if statements that are always true or always false1225

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 value887
These nested if statements could be combined909 - 910
Do not start a literal by 0 unless it's an octal value1084
Do not start a literal by 0 unless it's an octal value1257
Do not start a literal by 0 unless it's an octal value1265

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

ViolationLine
Overriding method merely calls super124 - 126