Random и объекты класса (pseudo random)

@Leonman · Регистрация: 17.06.2012

Всем привет.

Есть класс:

C++

typedef unsigned int ui;
 
class Player
{
private:
    ui health;
    ui mentalHealth;
public:
    Player();
    ui getHealth() const;
    bool changeHealth(ui);
    ui getMentalHealth() const;
    bool changeMentalHealth(ui);
};

Есть описание конструктора по умолчанию:

C++

Player::Player()
{
    srand(time(NULL));
 
    this->health = (rand() % 6 + 1) + 4;
    this->mentalHealth = (rand() % 6 + 1) + 4;
}

Я создаю массив объектов класса Player и хочу, чтобы у каждого объекта было рандомное значение health и mental health:

C++

1	Player team[4];

Но, у всех объектов эти поля имеют одно и тоже значение, рандомное каждый раз, но у всех одинаковое.

Что я делаю не так?

Tulosba · 04.06.2015, 14:04

Сообщение от Leonman

Что я делаю не так?

Вынеси

C++

1	srand(time(NULL));

из конструктора или добавь сюда, например, номер создаваемого экземпляра. Иначе все элементы создаются с одинаковым seed и как следствие rand() возвращает одинаковые последовательности.
Чисто формально даже в текущем варианте ты можешь получить разные значения, если время, возвращаемое time(NULL) переместится на секунду в процессе создания очередного элемента массива.

@Leonman · 04.06.2015, 14:16 **[ТС]**

Tulosba, А куда мне вынести srand(time(NULL))? если я выношу его в глобальное поле(не уверен, что это так называется

), то он выдает ошибку

C++

1	error: expected constructor, destructor, or type conversion before '(' token\|

Добавлено через 3 минуты
Tulosba, А, все, понял. Нужно было просто удалить эту строчку.

Опции темы