ArmySchubser - Version 1.1

ER LEBT!

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)