ich möchte mir ein kleines Programm zur Auswerten von Pokerhänden erstellen. Ich habe eine Klasse CardGame, die die Methode GetNextCard() enthält mit der zufällig eine Karte vergeben wird. Bis das Spiel mit der Methode Shuffle() neu gemischelt wird, wird keine Karte doppelt vergeben. Das funktioniert soweit auch ganz gut. Nun habe ich eine Datei mit Hilfsfunktionen, die die Karten auswerten soll und feststellt was man für eine Hand hat (Paar, Drilling usw). Da Kombinationen wie ein FullHouse oder besser nur sehr selten auftreten, habe ich mir noch eine Datei mit Testfunktionen erstellt. Eine dieser Funktionen gibt einen Vector mit 7 Werten zurück (ein Wert für jede Karte). Diese Funktion rufe ich so oft auf, bis die 7 Werte/ Karten die gewünschte Kombination haben (FullHouse usw). Zumindest soweit die Theorie. Leider erstellt diese Funktion immer die selben 7 Zufallszahlen. Wenn ich mein Kartenspiel ohne diese Testfunktion aufrufe funktioniert alles und ich bekomme auch jedesmal unterschiedliche Werte. Hier die wichtigsten Ausschnitte aus meinem Code.
**
Hier meine Funktion zum erzeugen der Zufallskarten**
int cardGame::GetNextCard(){
srand(time(NULL));
bool bCardDealt = true;
int iNewCard =-1;
do{
iNewCard = (rand() % 52);
if (!baCardsDealt[iNewCard]){
baCardsDealt[iNewCard]= true;
bCardDealt = false;
}
}while(bCardDealt);
return iNewCard;
};

Hier die Funktion die meine 7 Werte in einem Vektor anlegt und immer die selben Werte liefert:
vector <int> testSpiel(int iKartenAnzahl){
// Vektor mit x Karten anlegen
vector <int> vTestVektor;
cardGame Testspiel;
Testspiel.shuffle();
for (int iCounter=0; iCounter < iKartenAnzahl; iCounter++){
vTestVektor.push_back(Testspiel.GetNextCard());
cout << "zufall" << Testspiel.GetNextCard() << endl;
};
return vTestVektor;
};
Irgendwie bin ich zu doof den Fehler zu finden....
Vielleicht könnt ihr mir ja helfen. Vielen Dank schon mal im Vorraus.
MfG

Meiwen

srand(time(NULL));

srand muss man nur ein einziges Mal aufrufen, zum Beispiel beim Aufstarten des Programmes, danach ist es nicht unbenötig mehr nötig. srand setzt ein "seed", also eine Zahl von der aus die Zufallszahlen berechnet wird. Wenn man das gleiche "seed" setzt, kommen auch die gleichen Zufallszahlen raus. Wenn deine Funktion nun schnell hintereinander aufgerufen wird, dann gibt time(NULL) jedes Mal die gleiche Zahl zurück und du bekommst die gleichen Zufallszahlen

Grüssli

http://magazin.c-plusplus.net/artikel/Zuf�lle gibt`sFragezeichenAusrufezeichen - Funktionen rund um rand%2C Random und den Zufall

der Link brachte die Lösung.

Viel hilft viel – nicht immer

Auf eine böse Falle will ich noch hinweisen, es handelt sich hier um einen beliebten Fehler, den man überraschend oft in Programmen findet. Wohlmeinende Programmierer setzen nämlich vor jeder Ziehung den Startwert aus der Zeit erneut. Werden aber nun Zahlen schneller gezogen als die Uhr tickt, erhalten Sie statt vieler unterschiedlicher Zahlen plötzlich immer den gleichen Wert. Das ist umso dramatischer, da time() ein Sekundentimer ist, und innerhalb einer Sekunde kann man ziemlich oft eine Zufallszahl ziehen.

Vielen Dank

Meiwen