Refactor CardDeck and CardDeckTest

src/main/java/solitaer/CardDeck.java

@@ -7,20 +7,20 @@ public class CardDeck {
 	private ArrayList<Card> deck = new ArrayList<Card>();
 
 	public CardDeck() {
-		for (int j = 0; j < 4; j++) {
-			for (int i = 1; i <= 13; i++) {
-				switch (j) {
+		for (int symboleZähler = 0; symboleZähler < 4; symboleZähler++) {
+			for (int nrZähler = 1; nrZähler <= 13; nrZähler++) {
+				switch (symboleZähler) {
 				case 0:
-					deck.add(new Card(i, Symbol.Herz, false));
+					deck.add(new Card(nrZähler, Symbol.Herz, false));
 					break;					
 				case 1:
-					deck.add(new Card(i, Symbol.Karo, false));
+					deck.add(new Card(nrZähler, Symbol.Karo, false));
 					break;
 				case 2:
-					deck.add(new Card(i, Symbol.Kreuz, false));
+					deck.add(new Card(nrZähler, Symbol.Kreuz, false));
 					break;
 				case 3:
-					deck.add(new Card(i, Symbol.Pik, false));
+					deck.add(new Card(nrZähler, Symbol.Pik, false));
 					break;
 				}
 

src/test/java/solitaer/CardDeckTest.java

@@ -24,12 +24,13 @@ class CardDeckTest {
 	
 	@ParameterizedTest
 	@MethodSource("testSymbole")
+	//Jedes Symbol muss einmal vorhand sein
 	void testCardDeckSymbole(String msg, Symbol _sym) {
 		
 		boolean result = false;
 		
-		for(int i = 0; i<deck.getDeck().size(); i++) {
-			if(deck.getDeck().get(i).getSymbol() == _sym) {
+		for(int allCardZähler = 0; allCardZähler<deck.getDeck().size(); allCardZähler++) {
+			if(deck.getDeck().get(allCardZähler).getSymbol() == _sym) {
 				result = true;
 				break;
 			}
@@ -51,6 +52,7 @@ class CardDeckTest {
 	}
 	
 	@Test
+	//Jedes Symbol muss 13 mal vorrhanden sein
 	void testCardDeckNumber() {
 		
 		boolean result  = true;
@@ -66,8 +68,8 @@ class CardDeckTest {
 
 	private boolean CheckAllCards(Symbol _sym) {
 		boolean result = true;
-		for(int i =1; i<=13; i++) {
-			if(deck.getDeck().contains(new Card(i, _sym, false))== false) {
+		for(int nrZähler =1; nrZähler<=13; nrZähler++) {
+			if(deck.getDeck().contains(new Card(nrZähler, _sym, false))== false) {
 				result = false;
 				break;
 			}