Refactoring: Verschlankung der Fassade durchgeführt.

.gitignore

@@ -1,3 +0,0 @@
-/bin/
-/.classpath
-/.project

MyKniffel/src/de/hs_mannheim/informatik/games/kniffel/domain/Gewinnkarte.java

@@ -3,47 +3,41 @@ package de.hs_mannheim.informatik.games.kniffel.domain;
 public class Gewinnkarte {
 	private Kategorie[] kategorien;
 	private final int KATEGORIE_ANZAHL = 6;
-	private int summe;
-
+	
 	public Gewinnkarte() {
 		kategorien = new Kategorie[KATEGORIE_ANZAHL];
-		summe = 0;
-
+		
 		for (int i = 1; i <= kategorien.length; i++) {
 			kategorien[i-1] = new Kategorie(i);
 		}
 	}
-
+	
 	public boolean punkteEintragen(int kategorie, Würfel[] würfel) {
 		if (!kategorien[kategorie].isOffen())
 			return false;
-
-		summe+= kategorien[kategorie].punkteEintragen(würfel);
-
+			
+		kategorien[kategorie].punkteEintragen(würfel);
+		
 		return true;
 	}
-
+	
 	public String toString() {
 		StringBuilder sb = new StringBuilder();
-
+		
 		int i = 1;
 		for (Kategorie k : kategorien) {
 			sb.append(String.format("%2d. %s: ", i++, k.getName()));
-
+			
 			if (k.isOffen())
 				sb.append("offen");
 			else
 				sb.append(k.getPunkte());
-
-			sb.append(System.lineSeparator());
+			
+			if (i <= kategorien.length)
+				sb.append(System.lineSeparator());
 		}
-
-		sb.append(String.format("Summe: %3d Punkte", summe));
+		
 		return sb.toString();
 	}
 
-	public int getKategorieAnzahl() {
-		return KATEGORIE_ANZAHL;
-	}
-
 }

MyKniffel/src/de/hs_mannheim/informatik/games/kniffel/domain/Spiel.java

@@ -1,16 +1,14 @@
 package de.hs_mannheim.informatik.games.kniffel.domain;
 
-import java.util.ArrayList;
+import java.util.List;
 
 public class Spiel {
 	private Spieler[] spieler;
 	private int spieleranzahl;
 	private int aktuellerSpieler;
 	private Würfelbecher becher;
-	private int rundenzähler;
-	private int spielerzähler;
 	
-	public Spiel(ArrayList<String> spielernamen) {
+	public Spiel(List<String> spielernamen) {
 		spieleranzahl = spielernamen.size();
 		spieler = new Spieler[spieleranzahl];
 		
@@ -19,15 +17,13 @@ public class Spiel {
 		}
 		
 		aktuellerSpieler = (int)(Math.random() * spieleranzahl);
-		rundenzähler = 1;
-		spielerzähler = 1;
 		
 		becher = new Würfelbecher();
 		becher.würfle();
 	}
 
-	public Gewinnkarte getGewinnkarteAktuellerSpieler() {
-		return spieler[aktuellerSpieler].getGewinnkarte();
+	public String getGewinnkartentextAktuellerSpieler() {
+		return spieler[aktuellerSpieler].getGewinnkarte().toString();
 	}
 
 	public String getNameAktuellerSpieler() {
@@ -46,16 +42,5 @@ public class Spiel {
 	public boolean punkteEintragen(int kategorie) {
 		return spieler[aktuellerSpieler].getGewinnkarte().punkteEintragen(kategorie - 1, becher.getWürfel());
 	}
-
-	public boolean nächsterSpieler() {
-		aktuellerSpieler = ++aktuellerSpieler % spieleranzahl;
-		
-		if (++spielerzähler > spieleranzahl) {
-			++rundenzähler;
-			spielerzähler = 1;
-		}
-		
-		return rundenzähler <= spieler[aktuellerSpieler].getGewinnkarte().getKategorieAnzahl();
-	}
 	
 }

MyKniffel/src/de/hs_mannheim/informatik/games/kniffel/domain/WürfelTest.java

@@ -1,43 +1,22 @@
 package de.hs_mannheim.informatik.games.kniffel.domain;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 
-import java.util.HashSet;
-import java.util.Set;
-
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
 class WürfelTest {
-	
+
 	@Test
-	void testWürfelEinsBisSechs() {
+	void testWürfel() {
 		Würfel w = new Würfel();
 		
 		for (int i = 0; i < 1000; i++) {
 			int augen = w.würfle();
 			assertTrue(augen >= 1 && augen <= 6);
-			assertEquals(augen, w.getAugen());
 		}
 	}
 	
 	@Test
-	void testAugenVerteilung() {
-		Würfel w = new Würfel();
-		
-		Set<Integer> zahlen = new HashSet<>();
-		
-		for (int i = 0; i < 1000; i++) {
-			zahlen.add(w.würfle());
-		}
-		
-		assertEquals(6, zahlen.size());
-	}
-	
-	
-	@Test
-	@Disabled
 	void test8erWürfel() {
 		Würfel w = new Würfel(8);
 		

MyKniffel/src/de/hs_mannheim/informatik/games/kniffel/facade/KniffelApi.java

@@ -1,35 +1,14 @@
 package de.hs_mannheim.informatik.games.kniffel.facade;
 
-import java.util.ArrayList;
+import java.util.List;
 
-import de.hs_mannheim.informatik.games.kniffel.domain.Gewinnkarte;
 import de.hs_mannheim.informatik.games.kniffel.domain.Spiel;
 
 public class KniffelApi {
-	private Spiel spiel;
-	
-	public void starteSpiel(ArrayList<String> spielernamen, boolean starWarsModus) {
+
+	public Spiel starteSpiel(List<String> spielernamen, boolean starWarsModus) {
 		// TODO: StarWars über Vererbung einbauen?
-		spiel = new Spiel(spielernamen);
+		return new Spiel(spielernamen);
 	}
 	
-	public Gewinnkarte getGewinnkarte() {
-		return spiel.getGewinnkarteAktuellerSpieler();	
-	}
-
-	public String getNameAktuellerSpieler() {
-		return spiel.getNameAktuellerSpieler();
-	}
-	
-	public int[] wurfDurchführen(String... würfel) {
-		return spiel.wurfDurchführen(würfel);
-	}
-
-	public boolean punkteEintragen(int kategorie) {
-		return spiel.punkteEintragen(kategorie);
-	}
-
-	public boolean nächsterSpieler() {
-		return spiel.nächsterSpieler();
-	}
 }

MyKniffel/src/de/hs_mannheim/informatik/games/kniffel/tui/SpielStart.java

@@ -3,11 +3,13 @@ package de.hs_mannheim.informatik.games.kniffel.tui;
 import java.util.ArrayList;
 import java.util.Scanner;
 
+import de.hs_mannheim.informatik.games.kniffel.domain.Spiel;
 import de.hs_mannheim.informatik.games.kniffel.facade.KniffelApi;
 
 public class SpielStart {
 	private Scanner kb;
 	private KniffelApi api;
+	private Spiel spiel;
 	
 	public SpielStart(Scanner kb, KniffelApi api) {
 		this.kb = kb;
@@ -41,9 +43,9 @@ public class SpielStart {
 		
 		String sw = kb.nextLine();
 		
-		api.starteSpiel(spielernamen, sw.toLowerCase().equals("j"));
+		spiel = api.starteSpiel(spielernamen, sw.toLowerCase().equals("j"));
 		
-		new SpielrundeSpielen(kb, api).verarbeiteEingabe();
+		new SpielrundeSpielen(kb, spiel).verarbeiteEingabe();
 	}
 
 }

MyKniffel/src/de/hs_mannheim/informatik/games/kniffel/tui/SpielrundeSpielen.java

@@ -2,75 +2,68 @@ package de.hs_mannheim.informatik.games.kniffel.tui;
 
 import java.util.Scanner;
 
-import de.hs_mannheim.informatik.games.kniffel.facade.KniffelApi;
+import de.hs_mannheim.informatik.games.kniffel.domain.Spiel;
 
 public class SpielrundeSpielen {
 	private Scanner kb;
-	private KniffelApi api;
+	private Spiel spiel;
 
-	public SpielrundeSpielen(Scanner kb, KniffelApi api) {
+	public SpielrundeSpielen(Scanner kb, Spiel spiel) {
 		this.kb = kb;
-		this.api = api;
+		this.spiel = spiel;
 	}
 
 	public void verarbeiteEingabe() {
-		do {
-			System.out.print(">>>>>>>>>");
-			gewinnkarteAusgeben();
+		gewinnkarteAusgeben();
 
-			String[] zuWürfeln = {"1", "2", "3", "4", "5"};
+		String[] zuWürfeln = new String[0];
 
-			for (int i = 1; i <= 3; i++) {
-				int[] würfelwerte = api.wurfDurchführen(zuWürfeln);
-				System.out.print(i + ". Wurf: ");
-				for (int w : würfelwerte) {
-					System.out.print(w + " ");
-				}
-				System.out.println();
-				System.out.println("         1 2 3 4 5");
-				System.out.println();
-
-				if (i == 3)
-					break;
-
-				System.out.println("Welche Würfel sollen noch einmal geworfen werden? ");
-				System.out.println("Nr. des/der zu werfenden Würfel(s) durch Leerzeichen getrennt eingeben.");
-				System.out.println("Eine leere Eingabe, beendet das Würfeln.");
-				System.out.print("Eingabe: ");
-
-				String eingabe = kb.nextLine();	
-				if (eingabe.isEmpty())
-					break;
-
-				zuWürfeln = eingabe.split(" ");
+		for (int i = 1; i <= 3; i++) {
+			int[] würfelwerte = this.spiel.wurfDurchführen(zuWürfeln);
+			System.out.print(i + ". Wurf: ");
+			for (int w : würfelwerte) {
+				System.out.print(w + " ");
 			}
+			System.out.println();
+			System.out.println("         1 2 3 4 5");
+			System.out.println();
 
-			gewinnkarteAusgeben();
+			if (i == 3)
+				break;
 
-			System.out.println("In welcher Kategorie sollen die Punkte eingetragen werden?");
-			do {
-				System.out.print("> ");
-				int kategorie= Integer.parseInt(kb.nextLine());
+			System.out.println("Welche Würfel sollen noch einmal geworfen werden? ");
+			System.out.println("Nr. des/der zu werfenden Würfel(s) durch Leerzeichen getrennt eingeben.");
+			System.out.println("Eine leere Eingabe, beendet das Würfeln.");
+			System.out.print("Eingabe: ");
 
-				if (!api.punkteEintragen(kategorie))
-					System.out.println("Kategorie bereits belegt, bitte andere wählen.");
-				else
-					break;
-			} while(false);
+			String eingabe = kb.nextLine();	
+			if (eingabe.isEmpty())
+				break;
 
-			gewinnkarteAusgeben();
+			zuWürfeln = eingabe.split(" ");
+		}
 
-		} while(api.nächsterSpieler());
+		gewinnkarteAusgeben();
 
-		System.out.println("Spielende!");
-		// TODO: Gewinnerprüfung
+		System.out.println("In welcher Kategorie sollen die Punkte eingetragen werden?");
+		do {
+			System.out.print("> ");
+			int kategorie= Integer.parseInt(kb.nextLine());
+
+			if (!this.spiel.punkteEintragen(kategorie))
+				System.out.println("Kategorie bereits belegt, bitte andere wählen.");
+			else
+				break;
+		} while(false);
+
+		gewinnkarteAusgeben();
 	}
 
 	private void gewinnkarteAusgeben() {
 		System.out.println();
 		System.out.println("##################################################");
-		System.out.println("Gewinnkarte von " + api.getNameAktuellerSpieler());
-		System.out.println(api.getGewinnkarte().toString());
+		System.out.println("Gewinnkarte von " + this.spiel.getNameAktuellerSpieler());
+		System.out.println(this.spiel.getGewinnkartentextAktuellerSpieler());
 		System.out.println("##################################################");
 		System.out.println();
 	}

src/de/hs_mannheim/informatik/games/kniffel/domain/WürfelTest.java

@@ -1,50 +0,0 @@
-package de.hs_mannheim.informatik.games.kniffel.domain;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertTrue;
-
-import java.util.HashSet;
-import java.util.Set;
-
-import org.junit.jupiter.api.Disabled;
-import org.junit.jupiter.api.Test;
-
-class WürfelTest {
-	
-	@Test
-	void testWürfelEinsBisSechs() {
-		Würfel w = new Würfel();
-		
-		for (int i = 0; i < 1000; i++) {
-			int augen = w.würfle();
-			assertTrue(augen >= 1 && augen <= 6);
-			assertEquals(augen, w.getAugen());
-		}
-	}
-	
-	@Test
-	void testAugenVerteilung() {
-		Würfel w = new Würfel();
-		
-		Set<Integer> zahlen = new HashSet<>();
-		
-		for (int i = 0; i < 1000; i++) {
-			zahlen.add(w.würfle());
-		}
-		
-		assertEquals(6, zahlen.size());
-	}
-	
-	
-	@Test
-	@Disabled
-	void test8erWürfel() {
-		Würfel w = new Würfel(8);
-		
-		for (int i = 0; i < 1000; i++) {
-			int augen = w.würfle();
-			assertTrue(augen >= 1 && augen <= 8);
-		}
-	}
-
-}