simulering av kortspel

I din main skapar du en kortlek (PairSet) som du skall använda. Sen utför du din patiens med den kortleken NBT_ITERATIONS gånger och eftersom kortleken håller reda på hur många kort du dragit så måste du väl skapa en ny kortleksinstans i varje iteration.

Variabeln nopatiens används inte. Då bör den tas bort

while (iter < NBR_ITERATIONS && avsluta) { ...}
Här har du två villkor och det andra sätter du false när du passerar NBR_ITERATIONS-1 vilket innebär att båda villkoren är samma sak så du kan ta bort avsluta helt. Dessutom är namnet på variabeln väldigt missvisande. Heter den avsluta så bör den ju va true när loopen ska avslutas. Men som sagt, ta bort den helt.

if () {

} else if () {}

Där behöver du inte ha något villkor i else if (). Är (a.second != k) falskt så är
a.second() == k alltid sant.

Var sätter du win för övrigt?

I PairSet så ska du lagra din kortlek. Du kanske kunde kalla klassen Deck i stället för att visa det.
När du sen populerar den i konstruktorn så räknar du lite konstigt. Jag skulle nog populera korten i två loopan, en för färg och en för valör. Jag skulle dessutom göra färg och valör till enum men det kan va överkurs, dina siffror funkar bra men det blir ite svårare att debugga.

I PairSet.more så gör du ett fel som man ser ofta i produktionskod. Du kontrollerar om ett villkor är sant och då returnerar du sant, annars falskt. Det rätta är att alltid returnera villkoret
public boolean more() {
return (n>0)
}

PairSet.pick() skulle jag nog lösa med att pairs låg i en ArrayList där du kan plocka ur element utan att skyffla element själv. Det blir enklare och säkrare.

package base;


public class Card {
	/*
	 * Representation av ett kort
	 * Lagra färg och värde som heltal som börjar på 1
	 */
	private int color; 
	private int value; 
	
	public Card (int color, int value) {
		this.color = color; 
		this.value = value; 
	}
	
	
	public int getColor() {
		return color; 
	}
	
	public int getValue() {
		return value; 
	}


	@Override
	public String toString() {
		return "Card [color=" + color + ", value=" + value + "]";
	}
	
}

Kortlek:

package base;

import java.util.ArrayList;
import java.util.Date;
import java.util.List;
import java.util.Random;

/*
 * Representation av en kortlek, numeriska värden på färg och valör
 */
public class Deck {
	
	/*
	 * Standardstorlek på kortlek
	 */
	private static final int NUM_COLORS = 4;
	private static final int NUM_VALUES = 13; 
	
	/*
	 * Lagra i en ArrayList som tillåter dig att plocka ur element 
	 */
	private List<Card>  cards = new ArrayList<Card>(NUM_COLORS*NUM_VALUES);
	
	/*
	 * Slumptalsgenerator som men initierar med aktuell tid så får man olika 
	 * resultat för varje körning
	 */
	private Random random = new Random(new Date().getTime()); 
	
	public Deck() {
		/*
		 * Initiera kortleken
		 */
		for (int color = 0; color<NUM_COLORS; color++) {
			for (int value = 0; value < NUM_VALUES; value++) {
				/*
				 * Färg och värde börjar på 1
				 */
				cards.add(new Card(color+1,value+1));
			}
		}
	}
	
	/*
	 * Finns det kort kvar i leken
	 */
	public boolean more() {
		return cards.size()>0;
	}
	
	/*
	 * Plocka ett kort slumpmässigt ur leken och returnera
	 */
	public Card pick() {
		int cardIdx = random.nextInt(cards.size());
		return cards.remove(cardIdx);
	}
}

Main-klassen

package base;

/*
 * Drivare för att köra ett antal rundor av en patiens och 
 * beräkna sannolikheten att den går ut
 */
public class Patiens {

	private static final int NUM_ITERATIONS = 1000000;
	
	public static void main(String[] args) {
		Patiens pat = new Patiens(); 		
		pat.doRun();
	}
	public void doRun() {
		int iter = 0; 		// Antalet körningar
		int loss = 0;		// Antalet förluster
		
		while (iter < NUM_ITERATIONS) {
			Deck cards = new Deck(); 
			/*
			 * Börja med att räkna 1
			 */
			int stack = 1;  	// Iterator for stacks
			
			/*
			 * Kör så länge det finns kort i leken
			 */
			do {
				/*
				 * Slumpa ett kort
				 */
				Card card = cards.pick();
				
				/*
				 * Om valören stämmer med 1-2-3-räknaren så har gick den inte ut
				 */
				if (card.getValue() == stack) {
					loss++;
					break;
				}
				/*
				 * Öka räknaren till 3 och börja på 1 igen
				 */
				if (++stack > 3) {stack = 1;}
				
			} while (cards.more());
			iter++;
		}
		System.out.println(String.format("Sannolikheten för att patiensen går ut är %f procent, %d loss", 100.0*(NUM_ITERATIONS-loss)/NUM_ITERATIONS, loss));
	}
}