From 1e2eddafc7e861bc1e9e573ca49acb017f6ee2aa Mon Sep 17 00:00:00 2001 From: Achim Date: Mon, 20 Jun 2016 18:15:20 +0200 Subject: [PATCH 1/3] ArmySchubser - Version 1.0.0 Rosen sind rot, Veilchen sind blau, der Schubser is jetzt feddisch, ich weiss es genau. --- ArmySchubser.java | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/ArmySchubser.java b/ArmySchubser.java index 307b18e..b05d1a3 100644 --- a/ArmySchubser.java +++ b/ArmySchubser.java @@ -6,7 +6,7 @@ import javax.swing.JOptionPane; * Schubst Einheiten umher. * * @author MaxiJohl, GruenerWal - * @version 0.3.0 + * @version 1.0.0 */ public class ArmySchubser extends Map_World @@ -33,15 +33,19 @@ public class ArmySchubser extends Map_World String toMoveString = JOptionPane.showInputDialog(null, "Wieviele Einheiten willst du verschieben?"); Integer entitiesToMove = Integer.valueOf(toMoveString); - if ( (sourceProvince.getEntityCount() - entitiesToMove) > 0) + if ( (sourceProvince.getEntityCount() - entitiesToMove) > 0 && entitiesToMove != null) { sourceProvince.removeFromEntities(entitiesToMove); destinationProvince.addToEntities(entitiesToMove); + sourceProvince = null; + destinationProvince = null; } - if ( (sourceProvince.getEntityCount() - entitiesToMove) <= 0 ) + if ( (sourceProvince.getEntityCount() - entitiesToMove) <= 0 && entitiesToMove != null) { JOptionPane.showMessageDialog(null,"Du hast nicht genügend Einheiten, um die gewünschte Anzahl von " + sourceProvince.getDisplayName() + " nach " + destinationProvince.getDisplayName() + " zu verschieben, Köhler."); + sourceProvince = null; + destinationProvince = null; } } @@ -91,5 +95,4 @@ public class ArmySchubser extends Map_World } } } -} - +} \ No newline at end of file From 31a04b704a3c40d27649fd539a9771f189512391 Mon Sep 17 00:00:00 2001 From: Achim Date: Mon, 20 Jun 2016 20:06:06 +0200 Subject: [PATCH 2/3] ArmySchubser - Version 1.0.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Kleiner Fix in der act-Methode, damit diese später leichter in die GeneralMap-Klasse kopiert werden kann. (Durch eine eigene Abfrage der Anzahl der Provinzen wird zukünftige Schreibarbeit eingespart.) - Less Code. More Army. More Schubser. - Removed Herobrine --- ArmySchubser.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ArmySchubser.java b/ArmySchubser.java index b05d1a3..6ead1b1 100644 --- a/ArmySchubser.java +++ b/ArmySchubser.java @@ -6,7 +6,7 @@ import javax.swing.JOptionPane; * Schubst Einheiten umher. * * @author MaxiJohl, GruenerWal - * @version 1.0.0 + * @version 1.0.1 */ public class ArmySchubser extends Map_World @@ -85,7 +85,7 @@ public class ArmySchubser extends Map_World { Province clickedProvince = null; - for ( int i = 1; i <= provinceCount; i++) + for ( int i = 1; i <= (provinces.length - 1); i++) { if (provinces[i].hasClicked() == true) { From 963241cb1ef0dcc7e976fcd3a223d9d06d7278bd Mon Sep 17 00:00:00 2001 From: Achim Date: Mon, 20 Jun 2016 21:04:42 +0200 Subject: [PATCH 3/3] ArmySchubser - Version 1.1 ER LEBT! --- ArmySchubser.java | 43 ++++++++++++++++++++----------------------- 1 file changed, 20 insertions(+), 23 deletions(-) diff --git a/ArmySchubser.java b/ArmySchubser.java index 6ead1b1..9bd5bc9 100644 --- a/ArmySchubser.java +++ b/ArmySchubser.java @@ -6,7 +6,7 @@ import javax.swing.JOptionPane; * Schubst Einheiten umher. * * @author MaxiJohl, GruenerWal - * @version 1.0.1 + * @version 1.1.0 */ public class ArmySchubser extends Map_World @@ -28,24 +28,24 @@ public class ArmySchubser extends Map_World * Nimmt zwei Provinzen entgegen, und fragt, wieviele Einheiten vom ersten zum zweiten Eintrag verschoben werden sollen. * Überprüft, ob eine Verschiebung möglich ist und führt sie bei Erfolg aus. */ - public static void moveEntities(Province sourceProvince, Province destinationProvince) + public void moveEntities(Province sourceProvince, Province destinationProvince) { String toMoveString = JOptionPane.showInputDialog(null, "Wieviele Einheiten willst du verschieben?"); Integer entitiesToMove = Integer.valueOf(toMoveString); - if ( (sourceProvince.getEntityCount() - entitiesToMove) > 0 && entitiesToMove != null) + if ( entitiesToMove != null ) { - sourceProvince.removeFromEntities(entitiesToMove); - destinationProvince.addToEntities(entitiesToMove); - sourceProvince = null; - destinationProvince = null; - } + if ( (sourceProvince.getEntityCount() - entitiesToMove) > 0) + { + sourceProvince.removeFromEntities(entitiesToMove); + destinationProvince.addToEntities(entitiesToMove); + JOptionPane.showMessageDialog(null,"Einheiten erfolgreich verschoben, Kommandant " + getPlayerName() + "."); + } - if ( (sourceProvince.getEntityCount() - entitiesToMove) <= 0 && entitiesToMove != null) - { - JOptionPane.showMessageDialog(null,"Du hast nicht genügend Einheiten, um die gewünschte Anzahl von " + sourceProvince.getDisplayName() + " nach " + destinationProvince.getDisplayName() + " zu verschieben, Köhler."); - sourceProvince = null; - destinationProvince = null; + else if ( (sourceProvince.getEntityCount() - entitiesToMove) <= 0) + { + JOptionPane.showMessageDialog(null,"Du hast nicht genügend Einheiten, um die gewünschte Anzahl von " + sourceProvince.getDisplayName() + " nach " + destinationProvince.getDisplayName() + " zu verschieben, Köhler."); + } } } @@ -54,25 +54,22 @@ public class ArmySchubser extends Map_World * Ist er das, so wird überprüft, ob eine neue, an savedProvince angrenzende Provinz angeklickt wurde. * Ist dies der Fall, werden beide Provinzen an moveEntities übergeben. */ - public static void useProvincesToMove(Province givenProvince) + public void useProvincesToMove(Province givenProvince) { if (savedProvince == null) { savedProvince = givenProvince; } - if (savedProvince != null && givenProvince != savedProvince) + else if ((savedProvince != null) && (givenProvince != savedProvince) && (savedProvince.getOwner() == givenProvince.getOwner()) && (savedProvince.getOwner() == currentPlayer) ) { if (givenProvince.isProvinceNear(savedProvince.getID()) == true) { moveEntities(savedProvince,givenProvince); } - - else - { - savedProvince = null; - } - } + + savedProvince = null; + } } /** @@ -83,8 +80,8 @@ public class ArmySchubser extends Map_World */ public void act() { - Province clickedProvince = null; - + Province clickedProvince; + for ( int i = 1; i <= (provinces.length - 1); i++) { if (provinces[i].hasClicked() == true)