[C++] Classe che genera numeri a caso

[The_ouroboros]

E da un po che non programmavo e oggi ho ripreso...

#include <cstdlib>
#include <ctime>

class Gen
{
private:
    int  *list;
    int  range;
    int  seed;
    int  gen_casual();
    void SetRange(int v=100){ range=v;  };
    int  GetRange(){ return range;  };
public:
    Gen(int c, int s=time(NULL));
    ~Gen();
    int *GetList(int n=50);
};

Gen::Gen(int c, int s)
{
    seed = s;
    SetRange(c);
    srand(seed);
}

Gen::~Gen()
{
    // TODO
    delete list;
}
int Gen::gen_casual()
{
    return (int)( rand() % GetRange() + 1 );
}

int *Gen::GetList(int n)
{
    list =  new int[n];
    for(int i=0; i < n; i++)
    {
        list[i] = gen_casual();
    }
    return list;
}

Va bene come progettazione di classe o è troppo incasinata???

[andbin]

Tieni presente che la impostazione del 'seed' con srand è una cosa "globale", non ha quindi senso farla nel costruttore della classe.

Io, come base, farei così:

#include <cstdlib>
#include <ctime>

class Rnd
{
private:
    int min;
    int max;
public:
    Rnd (int min, int max);
    int next ();

    static void seed ();
    static void seed (unsigned int seed);
};

Rnd::Rnd (int min, int max)
{
    this->min = min;
    this->max = max;
}

int Rnd::next ()
{
    double d = rand () / (RAND_MAX+1.0);
    return ((int) (d * (max-min+1))) + min;
}

void Rnd::seed ()
{
    srand ((unsigned int) time (NULL));
}

void Rnd::seed (unsigned int seed)
{
    srand (seed);
}

[cionci]

Io ci metterei anche qualcosa di questo tipo:

#include <cstdlib>
#include <ctime>

class Rnd
{
private:
    int min;
    int max;

    static boolean seeded;

public:
    Rnd (int min, int max);
    int next ();

    static void seed ();
    static void seed (unsigned int seed);
};

Rnd::Rnd (int min, int max)
{
    this->min = min;
    this->max = max;
    if(!Rnd::seeded)
    {        
        Rnd::seed();
    }
}

int Rnd::next ()
{
    double d = rand () / (RAND_MAX+1.0);
    return ((int) (d * (max-min+1))) + min;
}

void Rnd::seed ()
{
    seed((unsigned int) time (NULL));
}

void Rnd::seed (unsigned int seed)
{
    srand (seed);
    Rnd::seeded = true;
}

boolean Rnd::seeded = false;

[andbin]

cionci, ottima idea!
Così non ci si deve preoccupare di impostare il seed esplicitamente.