From 632d5304a38dd8598a2563b7788aa8a52880ff2d Mon Sep 17 00:00:00 2001 From: Jérémy Zurcher Date: Fri, 14 Nov 2014 12:20:01 +0100 Subject: States: code reoder --- .../tankontank/game/states/StateAnimation.java | 12 ++--- .../asynk/tankontank/game/states/StateAttack.java | 60 +++++++++++----------- .../asynk/tankontank/game/states/StateBreak.java | 26 +++++----- .../ch/asynk/tankontank/game/states/StateMove.java | 48 ++++++++--------- .../asynk/tankontank/game/states/StatePromote.java | 12 ++--- .../tankontank/game/states/StateReinforcement.java | 24 ++++----- .../asynk/tankontank/game/states/StateRotate.java | 50 +++++++++--------- .../asynk/tankontank/game/states/StateSelect.java | 32 ++++++------ 8 files changed, 132 insertions(+), 132 deletions(-) (limited to 'core/src/ch') diff --git a/core/src/ch/asynk/tankontank/game/states/StateAnimation.java b/core/src/ch/asynk/tankontank/game/states/StateAnimation.java index 8690e9c0..86bf61ca 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateAnimation.java +++ b/core/src/ch/asynk/tankontank/game/states/StateAnimation.java @@ -14,24 +14,24 @@ public class StateAnimation extends StateCommon } @Override - public void touchDown() + public StateType abort() { + return StateType.ABORT; } @Override - public void touchUp() + public StateType done() { + return StateType.DONE; } @Override - public StateType abort() + public void touchDown() { - return StateType.ABORT; } @Override - public StateType done() + public void touchUp() { - return StateType.DONE; } } diff --git a/core/src/ch/asynk/tankontank/game/states/StateAttack.java b/core/src/ch/asynk/tankontank/game/states/StateAttack.java index 37df9f77..97efd877 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateAttack.java +++ b/core/src/ch/asynk/tankontank/game/states/StateAttack.java @@ -43,36 +43,6 @@ public class StateAttack extends StateCommon map.unselectHex(to); } - @Override - public void touchDown() - { - } - - @Override - public void touchUp() - { - Unit unit = upHex.getUnit(); - - // activeUnit is the target - if ((activeUnit == null) && map.possibleTargets.contains(unit)) { - ctrl.hud.notify("Attack " + unit); - map.hidePossibleTargets(); - to = upHex; - activeUnit = unit; - activeUnit.showTarget(); - map.collectAttackAssists(selectedUnit, activeUnit, ctrl.player.unitIterator()); - map.showAttackAssists(); - ctrl.hud.actionButtons.show(Buttons.ATTACK.b | Buttons.DONE.b | ((ctrl.cfg.canCancel) ? Buttons.ABORT.b : 0)); - } - - if ((activeUnit != null) && map.attackAssists.contains(unit)) { - if(map.toggleAttackAssist(unit)) - ctrl.hud.notify(unit + " will fire"); - else - ctrl.hud.notify(unit + " wont fire"); - } - } - @Override public StateType abort() { @@ -104,4 +74,34 @@ public class StateAttack extends StateCommon return nextState; } + + @Override + public void touchDown() + { + } + + @Override + public void touchUp() + { + Unit unit = upHex.getUnit(); + + // activeUnit is the target + if ((activeUnit == null) && map.possibleTargets.contains(unit)) { + ctrl.hud.notify("Attack " + unit); + map.hidePossibleTargets(); + to = upHex; + activeUnit = unit; + activeUnit.showTarget(); + map.collectAttackAssists(selectedUnit, activeUnit, ctrl.player.unitIterator()); + map.showAttackAssists(); + ctrl.hud.actionButtons.show(Buttons.ATTACK.b | Buttons.DONE.b | ((ctrl.cfg.canCancel) ? Buttons.ABORT.b : 0)); + } + + if ((activeUnit != null) && map.attackAssists.contains(unit)) { + if(map.toggleAttackAssist(unit)) + ctrl.hud.notify(unit + " will fire"); + else + ctrl.hud.notify(unit + " wont fire"); + } + } } diff --git a/core/src/ch/asynk/tankontank/game/states/StateBreak.java b/core/src/ch/asynk/tankontank/game/states/StateBreak.java index debc9c10..a64a494f 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateBreak.java +++ b/core/src/ch/asynk/tankontank/game/states/StateBreak.java @@ -30,6 +30,19 @@ public class StateBreak extends StateCommon if (activeUnit != null) map.hideMove(activeUnit.getHex()); } + @Override + public StateType abort() + { + return StateType.ABORT; + } + + @Override + public StateType done() + { + doRotation(o); + return StateType.DONE; + } + @Override public void touchDown() { @@ -61,19 +74,6 @@ public class StateBreak extends StateCommon } } - @Override - public StateType abort() - { - return StateType.ABORT; - } - - @Override - public StateType done() - { - doRotation(o); - return StateType.DONE; - } - private void doRotation(Orientation o) { if (done || (activeUnit == null)) return; diff --git a/core/src/ch/asynk/tankontank/game/states/StateMove.java b/core/src/ch/asynk/tankontank/game/states/StateMove.java index 02f814ea..fbc71ea8 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateMove.java +++ b/core/src/ch/asynk/tankontank/game/states/StateMove.java @@ -52,6 +52,30 @@ public class StateMove extends StateCommon } } + @Override + public StateType abort() + { + hideAssists(); + ctrl.setAnimationCount(map.activatedPawns.size()); + map.revertMoves(); + if (activeUnit.move.entryMove) { + map.leaveBoard(activeUnit); + ctrl.player.revertUnitEntry(activeUnit); + } + return StateType.ABORT; + } + + @Override + public StateType done() + { + hideAssists(); + // be sure that the hq is activated + if (selectedUnit.canMove() && (map.activatedPawns.size() > 0)) { + selectedUnit.move(); + } + return StateType.DONE; + } + @Override public void touchDown() { @@ -78,30 +102,6 @@ public class StateMove extends StateCommon } } - @Override - public StateType abort() - { - hideAssists(); - ctrl.setAnimationCount(map.activatedPawns.size()); - map.revertMoves(); - if (activeUnit.move.entryMove) { - map.leaveBoard(activeUnit); - ctrl.player.revertUnitEntry(activeUnit); - } - return StateType.ABORT; - } - - @Override - public StateType done() - { - hideAssists(); - // be sure that the hq is activated - if (selectedUnit.canMove() && (map.activatedPawns.size() > 0)) { - selectedUnit.move(); - } - return StateType.DONE; - } - private void hideAssists() { map.hideMoveablePawns(); diff --git a/core/src/ch/asynk/tankontank/game/states/StatePromote.java b/core/src/ch/asynk/tankontank/game/states/StatePromote.java index 49bcad59..70cb8759 100644 --- a/core/src/ch/asynk/tankontank/game/states/StatePromote.java +++ b/core/src/ch/asynk/tankontank/game/states/StatePromote.java @@ -22,24 +22,24 @@ public class StatePromote extends StateCommon } @Override - public void touchDown() + public StateType abort() { + return StateType.ABORT; } @Override - public void touchUp() + public StateType done() { + return StateType.DONE; } @Override - public StateType abort() + public void touchDown() { - return StateType.ABORT; } @Override - public StateType done() + public void touchUp() { - return StateType.DONE; } } diff --git a/core/src/ch/asynk/tankontank/game/states/StateReinforcement.java b/core/src/ch/asynk/tankontank/game/states/StateReinforcement.java index 5a611f90..2c423c65 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateReinforcement.java +++ b/core/src/ch/asynk/tankontank/game/states/StateReinforcement.java @@ -30,6 +30,18 @@ public class StateReinforcement extends StateCommon ctrl.hud.playerInfo.unitDock.hide(); } + @Override + public StateType abort() + { + return StateType.ABORT; + } + + @Override + public StateType done() + { + return StateType.DONE; + } + @Override public void touchDown() { @@ -47,18 +59,6 @@ public class StateReinforcement extends StateCommon ctrl.setState(StateType.SELECT); } - @Override - public StateType abort() - { - return StateType.ABORT; - } - - @Override - public StateType done() - { - return StateType.DONE; - } - private void changeUnit(Unit unit) { activeUnit = unit; diff --git a/core/src/ch/asynk/tankontank/game/states/StateRotate.java b/core/src/ch/asynk/tankontank/game/states/StateRotate.java index 7390b0d2..2066a8a7 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateRotate.java +++ b/core/src/ch/asynk/tankontank/game/states/StateRotate.java @@ -42,31 +42,6 @@ public class StateRotate extends StateCommon map.hideOrientation(to); } - @Override - public void touchDown() - { - } - - @Override - public void touchUp() - { - if (rotationSet) return; - - o = Orientation.fromAdj(to.getCol(), to.getRow(), downHex.getCol(), downHex.getRow()); - - if (o == Orientation.KEEP) return; - if (!activeUnit.move.entryMove && rotateOnly && (o == activeUnit.getOrientation())) return; - rotationSet = true; - - - if (ctrl.cfg.mustValidate) { - map.hideDirections(to); - map.showOrientation(to, o); - ctrl.hud.actionButtons.show(Buttons.ROTATE.b | Buttons.DONE.b | ((ctrl.cfg.canCancel) ? Buttons.ABORT.b : 0)); - } else - doRotation(o); - } - @Override public StateType abort() { @@ -93,6 +68,31 @@ public class StateRotate extends StateCommon return StateType.DONE; } + @Override + public void touchDown() + { + } + + @Override + public void touchUp() + { + if (rotationSet) return; + + o = Orientation.fromAdj(to.getCol(), to.getRow(), downHex.getCol(), downHex.getRow()); + + if (o == Orientation.KEEP) return; + if (!activeUnit.move.entryMove && rotateOnly && (o == activeUnit.getOrientation())) return; + rotationSet = true; + + + if (ctrl.cfg.mustValidate) { + map.hideDirections(to); + map.showOrientation(to, o); + ctrl.hud.actionButtons.show(Buttons.ROTATE.b | Buttons.DONE.b | ((ctrl.cfg.canCancel) ? Buttons.ABORT.b : 0)); + } else + doRotation(o); + } + private void hideAssists() { map.hideMoveablePawns(); diff --git a/core/src/ch/asynk/tankontank/game/states/StateSelect.java b/core/src/ch/asynk/tankontank/game/states/StateSelect.java index e8b95394..82733fba 100644 --- a/core/src/ch/asynk/tankontank/game/states/StateSelect.java +++ b/core/src/ch/asynk/tankontank/game/states/StateSelect.java @@ -28,6 +28,22 @@ public class StateSelect extends StateCommon hidePossibilities(); } + @Override + public StateType abort() + { + if (selectedHex != null) + map.unselectHex(selectedHex); + hidePossibilities(); + map.clearAll(); + return StateType.ABORT; + } + + @Override + public StateType done() + { + return StateType.DONE; + } + @Override public void touchDown() { @@ -79,20 +95,4 @@ public class StateSelect extends StateCommon } if (selectedUnit != null) ctrl.hud.notify(selectedUnit.toString()); } - - @Override - public StateType abort() - { - if (selectedHex != null) - map.unselectHex(selectedHex); - hidePossibilities(); - map.clearAll(); - return StateType.ABORT; - } - - @Override - public StateType done() - { - return StateType.DONE; - } } -- cgit v1.2.3