Csongor

src/main/java/program/main/Controller.java

@@ -1,6 +1,5 @@
 package program.main;
 
-
 import java.util.Random;
 import static program.main.Main.game;
 import static program.main.State.Working;
@@ -11,7 +10,7 @@ public class Controller {
      * attributumok
      */
 
-    private static int CoolDown;
+    private static int coolDown;
 
     private static int roundsLeft = 10;
     private static int plumberCoolDown;
@@ -22,12 +21,12 @@ public class Controller {
 
     private static Random rand;
     private static int index;
-    static Nomad CurrentNomad;
-    static Plumber CurrentPlumber;
+    static Nomad currentNomad;
+    static Plumber currentPlumber;
 
-    static boolean CurrentTurnIsNomad = false;
+    static boolean  currentTurnIsNomad = false;
 
-    static boolean GameOver = false;
+    static boolean gameOver = false;
 
 
     /**functions
@@ -37,7 +36,7 @@ public class Controller {
      * setRounds
      * Változtatja a roundLeft adattag értékét -1 el
      */
-    public static void setRounds_min1() {
+    public static void decrementRounds() {
         roundsLeft--;
     }
 
@@ -46,15 +45,20 @@ public static void setRounds_min1() {
      *
      * @param i Az új hátralévő körök száma.
      */
-    public static void setRounds(int i){roundsLeft=i;}
+    public static void setRounds(int i){
+        if (i < 0){
+            throw new IllegalArgumentException("Round count cannot be negative");
+        }
+        roundsLeft = i;
+    }
 
     /**
      * Visszaadja a jelenlegi vízszerelő játékost.
      *
      * @return A jelenlegi vízszerelő játékos.
      */
     public static Plumber getCurrentPlumber() {
-        return CurrentPlumber;
+        return currentPlumber;
     }
 
     /**
@@ -63,7 +67,7 @@ public static Plumber getCurrentPlumber() {
      * @return A jelenlegi nomád játékos.
      */
     public static Nomad getCurrentNomad() {
-        return CurrentNomad;
+        return currentNomad;
     }
 
     /**
@@ -86,7 +90,7 @@ public static int getRounds() {
      */
     public static int getCoolDown() {
         // System.out.println("SUCCESS - Controller.getPlumberCoolDown()");
-        return CoolDown;
+        return coolDown;
     }
 
     /**
@@ -97,7 +101,7 @@ public static int getCoolDown() {
      */
     public static void setCoolDown(int value) {
         //System.out.println("SUCCESS - Controller.setNomadCoolDown()");
-        CoolDown = value;
+        coolDown = value;
     }
 
     /**
@@ -108,25 +112,25 @@ public static void setCoolDown(int value) {
      */
     public static int getNomadCoolDown() {
         // System.out.println("SUCCESS - Controller.getNomadCoolDown()");
-        return CoolDown;
+        return coolDown;
     }
 
     /**
      * Visszaadja a jelenlegi nomád játékost.
      *
      * @return A jelenlegi nomád játékos.
      */
-    public Nomad CurrentNomadTurn() {
-        return CurrentNomad;
+    public Nomad currentNomadTurn() {
+        return currentNomad;
     }
 
     /**
      * Visszaadja a jelenlegi vízszerelő játékost.
      *
      * @return A jelenlegi vízszerelő játékos.
      */
-    public Plumber CurrentPlumberTurn() {
-        return CurrentPlumber;
+    public Plumber currentPlumberTurn() {
+        return currentPlumber;
     }
 
     /**
@@ -135,82 +139,59 @@ public Plumber CurrentPlumberTurn() {
      */
     public void turn() {
         if (getRounds() == 0){
-            GameOver = true;
+            gameOver = true;
             return;
         }
-        //System.out.println("lefut a turn");
-        //System.out.println(game.plumberButtonPanel.getendturn());
         Main.game.getGameWindow().getPlumberButtonPanel().setVisible(true);
         for (int i = 0; i < game.Plumbers().size(); i++) {
-            //System.out.println("plumber size: " + game.Plumbers().size());
             game.Nomads().get(i).setMoved(false);
-            //System.out.println("plumber");
-            CurrentPlumber = game.Plumbers().get(i);
+            currentPlumber = game.Plumbers().get(i);
             while (!game.plumberButtonPanel.getendturn()) {
-                CurrentPlumber = game.Plumbers().get(i);
-                CurrentPlumber.turn();
+                currentPlumber = game.Plumbers().get(i);
+                currentPlumber.turn();
             }
             Main.game.gamePanel.repaint();
             game.Plumbers().get(i).setMoved(false);
 
-            //System.out.println("plumber ended");
             Main.game.gamePanel.repaint();
             for (int j = 0; j < game.Pipes().size(); j++) {
                 game.Pipes().get(j).turn();
             }
-            //System.out.println("minden pipe turn");
             for (int k = 0; k < game.Fields().size(); k++) {
                 game.Fields().get(k).turn();
             }
-            //System.out.println("minden fiels turn");
 
             game.plumberButtonPanel.resetTurn(false);
-            //System.out.println("plumber turn reset");
-            //System.out.println(game.plumberButtonPanel.getendturn());
-            //System.out.println("szerelo : " + plumberPoints);
-            //System.out.println("nomad : " + nomadPoints);
 
         }
-        CurrentTurnIsNomad = true;
+         currentTurnIsNomad = true;
         Main.game.getGameWindow().getPlumberButtonPanel().setVisible(false);
         Main.game.getGameWindow().getNomadButtonPanel().setVisible(true);
 
 
         for (int i = 0; i < game.Nomads().size(); i++) {
             game.Nomads().get(i).setMoved(false);
-            CurrentNomad = game.Nomads().get(i);
+            currentNomad = game.Nomads().get(i);
             while (!game.nomadButtonPanel.getendturn()) {
-                CurrentNomad = game.Nomads().get(i);
-                CurrentPlumber.turn();
+                currentNomad = game.Nomads().get(i);
+                currentPlumber.turn();
             }
             Main.game.gamePanel.repaint();
             game.Nomads().get(i).setMoved(false);
 
-            //System.out.println("plumber ended");
             Main.game.gamePanel.repaint();
             for (int k = 0; k < game.Pipes().size(); k++) {
                 game.Pipes().get(k).turn();
             }
-            //System.out.println("minden pipe turn");
             for (int j = 0; j < game.Fields().size(); j++) {
                 game.Fields().get(j).turn();
             }
-            //System.out.println("minden fiels turn");
-
             game.nomadButtonPanel.resetTurn(false);
-            //System.out.println("nomad turn reset");
-            //System.out.println(game.nomadButtonPanel.getendturn());
-            //.out.println("szerelo : " + plumberPoints);
-            //System.out.println("nomad : " + nomadPoints);
         }
-        CurrentTurnIsNomad = false;
+         currentTurnIsNomad = false;
         Main.game.getGameWindow().getNomadButtonPanel().setVisible(false);
         Main.game.getGameWindow().getPlumberButtonPanel().setVisible(true);
-        setRounds_min1();
-
-
-
-
+        decrementRounds();
     }
 
     /**
@@ -276,7 +257,7 @@ public static void destroyPump() {
     }
 
     public static boolean getCurrentTurnIsNomad() {
-        return CurrentTurnIsNomad;
+        return  currentTurnIsNomad;
     }
 
     public static void setPlumberPoints(int plumberPoints) {

src/main/java/program/main/Field.java

@@ -1,12 +1,8 @@
 package program.main;
 
-
-import java.awt.image.BufferedImage;
 import java.io.Serializable;
 import java.util.ArrayList;
 
-
-
 /**
  * A pálya ezekből az elemekből áll. Ez egy absztrakt osztály.
  */
@@ -21,17 +17,14 @@ public abstract class Field implements Round, Serializable {
 
     public void SetName(String s) {
         ID = s;
-        //System.out.println("SUCCESS - Field.SetName()");
     }
 
     public String GetName() {
-        //System.out.println("SUCCESS - Field.GetName()");
         return ID;
     }
 
     @Override
     public String toString() {
-        //System.out.println("SUCCESS - Field.toString()");
         return (ID + " Field");
     }
 
@@ -127,9 +120,7 @@ public String toString() {
     /**
      * A víz áramlását valósítja meg.
      */
-    public void waterFlowing() {
-        //System.out.println("SUCCESS - Field.waterFlowing");
-    }
+    public abstract void waterFlowing();
 
     /**
      * Beállítja, hogy a ciszternában van-e pumpa a newPump adattag értékének változtatásával.
@@ -145,17 +136,16 @@ public void waterFlowing() {
      */
     public abstract boolean getNewPump();
 
-    public abstract ArrayList getConnectedPipes();
+    public abstract ArrayList<Pipe> getConnectedPipes();
 
-    public abstract ArrayList getConnectedPumps();
+    public abstract ArrayList<Pump> getConnectedPumps();
 
     /**
      * A field állapotának lekérésére szolgál
      *
      * @param state
      */
     public void setState(State state) {
-        //System.out.println("SUCCESS - Field.setState()");
         this.state = state;
     }
 
@@ -165,7 +155,6 @@ public void setState(State state) {
      * @return
      */
     public State getState() {
-        //System.out.println("SUCCESS - Field.getState()");
         return state;
     }