[C++] Probleem met random.

[JordyOnrust]

Beste mensen,

Ik ondervind wat problemen met het genereren van random getallen in C++.
Mijn bedoeling is het ontwikkelen van een soort Yahtzee game.
Binnen deze game heb ik een class Dobbelsteen, welke een random getal kan genereren tussen 1 en 6.

De class Yahtzee maakt 5 objecten van Dobbelsteen aan, en met een for-loop roep ik van elke dobbelsteen een methode aan voor het genereren van een random getal.

Zie onderstaande code.

C++: Constanten.h

#ifndef CONSTANTEN
#define CONSTANTEN

const int NUMBER_ON_DICE = 6;
const int NUMBER_OF_DICES = 5;

#endif

C++: Dobbelsteen.h

#ifndef DOBBELSTEEN
#define DOBBELSTEEN

#include "Constanten.h"
#include <cstdlib>
#include <ctime> 

class Dobbelsteen
{
  public:
     // Getal dat de dobbelsteen aanwijst.
     int gegooid;
     int gooiDobbelsteen();
};

#endif

C++: Dobbelsteen.cpp

#include "Dobbelsteen.h"

using namespace std;

int Dobbelsteen::gooiDobbelsteen()
{
     srand((unsigned)time(0));    
     this->gegooid = rand()%NUMBER_ON_DICE + 1;
     return this->gegooid;
}

C++: Yahtzee.h

#ifndef YAHTZEE
#define YAHTZEE

#include "Constanten.h"
#include "Dobbelsteen.h"
#include <cstdlib>

class Yahtzee
{
     private:
          Dobbelsteen* dobbelstenen[NUMBER_OF_DICES];
     public:
          Yahtzee();
          void gooiDobbelstenen();
};

#endif

C++: Yahtzee.cpp

#include "Yahtzee.h"
#include <iostream>

using namespace std;

Yahtzee::Yahtzee()
{
     for(int i = 0; i < NUMBER_OF_DICES; i++)
     {
          Dobbelsteen* dobbelsteen = new Dobbelsteen();
          dobbelstenen[i] = dobbelsteen;
     }
}

void Yahtzee::gooiDobbelstenen()
{
     for(int i = 0; i < NUMBER_OF_DICES; i++)
     {
          int blaat = dobbelstenen[i]->gooiDobbelsteen();
          cout << blaat << endl;
     }
     cin.get();
}

C++: Game.cpp

#include <iostream>
#include "Yahtzee.h"

using namespace std;

int main()
{
     Yahtzee yahtzee;
     yahtzee.gooiDobbelstenen();
}

Wat mijn probleem met bovenstaande code nou is, is dat ik bij het genereren van dus 5 keer een random getal 5 keer het zelfde getal terug krijg.

Ik zou niet weten hoe ik dit zou kunnen oplossen.

Heeft iemand een idee?

[PrisonerOfPain]

Je hoeft srand maar een keer aan te roepen in plaats van iedere keer dat je rand aanroept.

[kunnen]

De seed (srand) zorgt ervoor dat je random-functie op een bepaalde plek begint. Nu geef je hem steeds dezelfde plek mee (namelijk de huidige tijd), dus hij geeft ook steeds hetzelfde getal.

Oplossing is hierboven genoemd.

[Creepy]

Je moet je srand maar 1 keer aanroepen in je programma en daarna nooit meer. Dat is de initialisatie voor de random() functie. Waarschijnlijkt levert de time(0) call dezelfde waarde op voor alle dobbelstenen.
[/spuit11, sneller posten....]

[ Voor 7% gewijzigd door Creepy op 01-04-2009 17:22 ]

[.oisyn]

Het meervoud van dice is overigens... tatatataaaa: dice . In US-english is het enkelvoud overigens die, maar in de UK zeggen ze ook dan dice.

Overigens zou ik je de tip willen geven om een taal te kiezen en daar ook bij te blijven. En dus niet NUMBER_ON_DICE (Engels) en gooiDobbelsteen() (Nederlands)

[ Voor 37% gewijzigd door .oisyn op 01-04-2009 17:44 ]

[JordyOnrust]

Eey,

Super bedankt.
Ik heb nu srand uit de loop gehaald en nu werkt het prima.

Over de spel / taalfouten, jullie hebben gelijk, zal er wat aan doen.

Super bedankt.

[.oisyn]

http://thedailywtf.com/Articles/More-or-Less-Random.aspx

[leuk_he]

nog meer offtopic


(moet kunnen terwijl vandaag de dag m'n pagina door lemmings wordt opgegeten. )

[ Voor 26% gewijzigd door leuk_he op 01-04-2009 18:31 ]

[Soultaker]

Zal ik de (betere, imo) Dilbert cartoon ook nog even posten of zullen we weer on-topic gaan?

[writser]

.oisyn schreef op woensdag 01 april 2009 @ 17:45:
http://thedailywtf.com/Articles/More-or-Less-Random.aspx

Damn, ik wou hem net posten. "Keep it random, but the same every time!"

[Zoijar]

writser schreef op woensdag 01 april 2009 @ 18:41:
Damn, ik wou hem net posten. "Keep it random, but the same every time!"

Wel grappig uitgedrukt, maar op zich niet zo'n vreemd verzoek... repeatable randomness is best een bekend iets.

[Hydra]

Zoijar schreef op woensdag 01 april 2009 @ 19:27:
Wel grappig uitgedrukt, maar op zich niet zo'n vreemd verzoek... repeatable randomness is best een bekend iets.

Sterker nog, ik vind het uberhaupt geen WTF. Het is volledig logisch dat je een scenario opnieuw af kunt draaien zodat je bijvoorbeeld mensen kunt tonen wat ze verkeerd gedaan hebben.

[.oisyn]

Dan neem je dat op. Kun je nog vooruit- en terugspoelen ook, en bovendien de acties van de leerling erin mee kunnen nemen. Jouw argument voor niet-random data vind ik een stuk minder zinnig dan die van die admiraals, namelijk dat ze elke leerling aan dezelfde test willen onderwerpen.

[Hydra]

.oisyn schreef op donderdag 02 april 2009 @ 10:40:
Dan neem je dat op. Kun je nog vooruit- en terugspoelen ook, en bovendien de acties van de leerling erin mee kunnen nemen. Jouw argument voor niet-random data vind ik een stuk minder zinnig dan die van die admiraals, namelijk dat ze elke leerling aan dezelfde test willen onderwerpen.

Ik bedoel dat ze dus hetzelfde scenario nog een keer aan kunnen bieden, dus nog steeds interactief, in plaats van als een 'film'.