diff --git a/fh.fd.ci.client/src/main/java/de/fd/fh/FigureFarmer.java b/fh.fd.ci.client/src/main/java/de/fd/fh/FigureFarmer.java index 8439753..af1aaad 100644 --- a/fh.fd.ci.client/src/main/java/de/fd/fh/FigureFarmer.java +++ b/fh.fd.ci.client/src/main/java/de/fd/fh/FigureFarmer.java @@ -2,9 +2,10 @@ package de.fd.fh; public class FigureFarmer extends Figure { - public int mColor; + enum Color { Black, White}; + public Color mColor; - public FigureFarmer(int colorCode) + public FigureFarmer(Color colorCode) { mColor = colorCode; } diff --git a/fh.fd.ci.client/src/main/java/de/fd/fh/Game.java b/fh.fd.ci.client/src/main/java/de/fd/fh/Game.java index 6219439..16baa7b 100644 --- a/fh.fd.ci.client/src/main/java/de/fd/fh/Game.java +++ b/fh.fd.ci.client/src/main/java/de/fd/fh/Game.java @@ -27,12 +27,12 @@ public class Game for (int j = 0; j < Figure.fieldLength; j++) // col { - mField[Game.whiteRowFarmer * Figure.fieldLength + j] = new FigureFarmer(0); + mField[Game.whiteRowFarmer * Figure.fieldLength + j] = new FigureFarmer(FigureFarmer.Color.White); } for (int j = 0; j < Figure.fieldLength; j++) // col { - mField[Game.blackRowFarmer * Figure.fieldLength + j] = new FigureFarmer(1); + mField[Game.blackRowFarmer * Figure.fieldLength + j] = new FigureFarmer(FigureFarmer.Color.Black); } } } diff --git a/fh.fd.ci.client/src/test/java/de/fd/fh/FigureFarmerTest.java b/fh.fd.ci.client/src/test/java/de/fd/fh/FigureFarmerTest.java index bfac82e..eafeb23 100644 --- a/fh.fd.ci.client/src/test/java/de/fd/fh/FigureFarmerTest.java +++ b/fh.fd.ci.client/src/test/java/de/fd/fh/FigureFarmerTest.java @@ -10,7 +10,7 @@ class FigureFarmerTest @Test void whiteFarmerValidDestination() { - Figure f = new FigureFarmer(0); + Figure f = new FigureFarmer(FigureFarmer.Color.White); assertTrue(f.moveAllowed(Figure.fieldLength * 6 + 0, Figure.fieldLength * 5 + 0, new Figure[Figure.fieldLength * Figure.fieldLength])); @@ -19,7 +19,7 @@ class FigureFarmerTest @Test void whiteFarmerInvalidDestinationDown() // statt nach "oben" wird nach "unten" gegangen { - Figure f = new FigureFarmer(0); + Figure f = new FigureFarmer(FigureFarmer.Color.White); assertFalse(f.moveAllowed(Figure.fieldLength * 6 + 0, Figure.fieldLength * 7 + 0, new Figure[Figure.fieldLength * Figure.fieldLength])); @@ -28,7 +28,7 @@ class FigureFarmerTest @Test void whiteFarmerInvalidDestinationUp() // mehr als 1 Schritt nach "oben" { - Figure f = new FigureFarmer(0); + Figure f = new FigureFarmer(FigureFarmer.Color.White); assertFalse(f.moveAllowed(Figure.fieldLength * 6 + 0, Figure.fieldLength * 4 + 0, new Figure[Figure.fieldLength * Figure.fieldLength])); @@ -37,7 +37,7 @@ class FigureFarmerTest @Test void checkMoveWithValidDestinationVar2() // Bauer (w) steht in der Mitte vom Feld und geht 1 nach oben { - Figure f = new FigureFarmer(0); + Figure f = new FigureFarmer(FigureFarmer.Color.White); assertTrue(f.moveAllowed(Figure.fieldLength * 4 + 4, Figure.fieldLength * 3 + 4, new Figure[Figure.fieldLength * Figure.fieldLength])); @@ -46,7 +46,7 @@ class FigureFarmerTest @Test void whiteFarmerInvalidDestinationDownVar2() // Bauer (w) steht in der Mitte vom Feld und geht 1 nach unten { - Figure f = new FigureFarmer(0); + Figure f = new FigureFarmer(FigureFarmer.Color.White); assertFalse(f.moveAllowed(Figure.fieldLength * 4 + 4, Figure.fieldLength * 5 + 4, new Figure[Figure.fieldLength * Figure.fieldLength])); diff --git a/fh.fd.ci.client/src/test/java/de/fd/fh/GameTest.java b/fh.fd.ci.client/src/test/java/de/fd/fh/GameTest.java index b260a15..52664f7 100644 --- a/fh.fd.ci.client/src/test/java/de/fd/fh/GameTest.java +++ b/fh.fd.ci.client/src/test/java/de/fd/fh/GameTest.java @@ -51,18 +51,18 @@ class GameTest // weiße Bauern for (int j = 0; j < Figure.fieldLength; j++) // col { - assertEquals(new FigureFarmer(0).getClass(), + assertEquals(new FigureFarmer(FigureFarmer.Color.White).getClass(), g.mField[Game.whiteRowFarmer * Figure.fieldLength + j].getClass()); - assertEquals(new FigureFarmer(0), + assertEquals(new FigureFarmer(FigureFarmer.Color.White), g.mField[Game.whiteRowFarmer * Figure.fieldLength + j]); } // schwarze Bauern for (int j = 0; j < Figure.fieldLength; j++) // col { - assertEquals(new FigureFarmer(1).getClass(), + assertEquals(new FigureFarmer(FigureFarmer.Color.Black).getClass(), g.mField[Game.blackRowFarmer * Figure.fieldLength + j].getClass()); - assertEquals(new FigureFarmer(1), + assertEquals(new FigureFarmer(FigureFarmer.Color.Black), g.mField[Game.blackRowFarmer * Figure.fieldLength + j]); } }