PMD Results

The following document contains the results of PMD 4.2.5.

Files

cz/cuni/amis/pogamut/defcon/agent/DefConAgent.java

ViolationLine
Avoid unused private fields such as 'initedLatch'.37

cz/cuni/amis/pogamut/defcon/agent/impl/DefConAgentLogicController.java

ViolationLine
Avoid unused private methods such as 'firstGameLogicWorker()'.171

cz/cuni/amis/pogamut/defcon/agent/module/sensor/GameInfo.java

ViolationLine
Avoid unused private methods such as 'populatePoints()'.332
Unnecessary final modifier in final class891 - 893
Unnecessary final modifier in final class895 - 897
Unnecessary final modifier in final class899 - 901
Unnecessary final modifier in final class903 - 905
An empty statement (semicolon) not part of a loop1012

cz/cuni/amis/pogamut/defcon/ai/AbstractAI.java

ViolationLine
Avoid unnecessary return statements30

cz/cuni/amis/pogamut/defcon/communication/command/impl/DefConAct.java

ViolationLine
Avoid unused private fields such as 'log'.33
Avoid unused private fields such as 'eventBus'.35

cz/cuni/amis/pogamut/defcon/communication/messages/commands/PlaceFleet.java

ViolationLine
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/PlaceStructure.java

ViolationLine
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/RequestAlliance.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/RequestCeaseFire.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/RequestGameSpeed.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/RequestShareRadar.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/SendChatMessage.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/SendVote.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/SetActionTarget.java

ViolationLine
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/SetLandingTarget.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/SetMovementTarget.java

ViolationLine
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/SetState.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/commands/WhiteboardClear.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package16
Avoid unused constructor parameters such as 'original'.44 - 45

cz/cuni/amis/pogamut/defcon/communication/messages/commands/WhiteboardDraw.java

ViolationLine
No need to import a type that lives in the same package16

cz/cuni/amis/pogamut/defcon/communication/messages/infos/AirBase.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Battleship.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Bomber.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Carrier.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/CeasedFire.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/City.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/DebugIsReplayingGameChanged.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/DefConChanged.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/DefConEvent.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.base.communication.translator.event.IWorldChangeEvent'3

cz/cuni/amis/pogamut/defcon/communication/messages/infos/DefConObjectUpdate.java

ViolationLine
Avoid unused imports such as 'javabot.PogamutJBotSupport'19

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Destroyed.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/EventBatchBegin.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/EventBatchEnd.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Explosion.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Fighter.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Fleet.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/GameRunningChanged.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/GameSpeedChanged.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Gunshot.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Hit.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/InvalidObject.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/NewVote.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Nuke.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/NukeLaunchSilo.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/NukeLaunchSub.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/PingCarrier.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/PingDetection.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/PingSub.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/QueueItem.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Radar.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Saucer.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/SharedRadar.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Silo.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Sub.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/TeamRetractedVoted.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/TeamVoted.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/Tornado.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/UnceasedFire.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/UnsharedRadar.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/VictoryTimerActiveChanged.java

ViolationLine
Avoid unused imports such as 'cz.cuni.amis.pogamut.defcon.base3d.worldview.object.DefConLocation'14
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/VoteFinishedNo.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/messages/infos/VoteFinishedYes.java

ViolationLine
No need to import a type that lives in the same package17

cz/cuni/amis/pogamut/defcon/communication/translator/DefConMessageProducer.java

ViolationLine
Avoid unused private fields such as 'controller'.54

cz/cuni/amis/pogamut/defcon/communication/worldview/NativeMapSource.java

ViolationLine
Avoid unused private fields such as 'log'.32

cz/cuni/amis/pogamut/defcon/communication/worldview/modules/grid/IGrid.java

ViolationLine
Avoid unused imports such as 'java.util.Map'3

cz/cuni/amis/pogamut/defcon/communication/worldview/modules/grid/basic/BasicGrid.java

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

cz/cuni/amis/pogamut/defcon/communication/worldview/modules/managers/buildings/BuildingPlacementProvider.java

ViolationLine
Avoid unused private fields such as 'mapInfo'.27

cz/cuni/amis/pogamut/defcon/communication/worldview/polygons/GameMapInfoPolygons.java

ViolationLine
Avoid unused method parameters such as 'flag'.617 - 639
Avoid unused method parameters such as 'flag'.641 - 708
Ensure you override both equals() and hashCode()811
Avoid unused private methods such as 'simplifyPoly(LinkedList)'.1094 - 1095
Avoid unused private methods such as 'mathModulus(float,float)'.1273
Avoid unused private methods such as 'checkAndJoinBordersOnSeam(List,List)'.1277 - 1278
Avoid unused method parameters such as 'borderA'.1277 - 1283
Avoid unused method parameters such as 'borderB'.1277 - 1283

cz/cuni/amis/pogamut/defcon/consts/Event.java

ViolationLine
Avoid duplicate imports such as 'cz.cuni.amis.pogamut.defcon.communication.messages.infos.DefConEvent'5

cz/cuni/amis/pogamut/defcon/consts/UnitType.java

ViolationLine
Avoid empty catch blocks255 - 256

cz/cuni/amis/pogamut/defcon/utils/PolygonUtils.java

ViolationLine
Avoid unused local variables such as 'polygons'.146
This usage of the Collection.toArray() method will throw a ClassCastException.246
Avoid unused local variables such as 'm'.296
Avoid unused private methods such as 'checkDirection2(Location,Location,Location)'.331

cz/cuni/amis/pogamut/defcon/utils/closestpoints/ClosestPointsLookUp.java

ViolationLine
These nested if statements could be combined443 - 450
These nested if statements could be combined446 - 449

cz/cuni/amis/pogamut/defcon/utils/closestpoints/ClosestPointsManager.java

ViolationLine
Avoid unused imports such as 'java.util.Map.Entry'4
Avoid unused private fields such as 'log'.24

cz/cuni/amis/pogamut/defcon/utils/quadtree/BuildingPlacementQuadTreeLabellingMethod.java

ViolationLine
Avoid unused private fields such as 'log'.17

cz/cuni/amis/pogamut/defcon/utils/quadtree/QuadTreeBFSIterator.java

ViolationLine
Avoid unused private fields such as 'tree'.17
Avoid unused private fields such as 'node'.18
Avoid unused private fields such as 'finished'.20

cz/cuni/amis/pogamut/defcon/utils/quadtree/QuadTreePostorderIterator.java

ViolationLine
Avoid unused private fields such as 'tree'.16

cz/cuni/amis/pogamut/defcon/utils/quadtree/RectangularFillTester.java

ViolationLine
These nested if statements could be combined125 - 127
These nested if statements could be combined134 - 136

cz/cuni/amis/pogamut/defcon/utils/quadtree/UnitPlacementQuadTreeLabellingMethod.java

ViolationLine
Avoid unused private fields such as 'log'.17

javabot/ActExecutor.java

ViolationLine
Avoid unused private fields such as 'log'.28
Avoid unused private fields such as 'eventBus'.30