Функция rand() дает странные результаты

@parsecer · Регистрация: 19.07.2015

Author24 — интернет-сервис помощи студентам

Есть класс Owner с таким конструктором, значения его членов определяются рандомно. В main.cpp создаются и выводятся на экран последовательно 3 объекта Owner:

C++

 Owner::Owner(std::string name)
 {
m_name=name;
    m_goodness=(rand()%10+1);
    m_size=(rand()%10+1);
    m_money=rand()%1001;
    m_agility=(rand()%10+1);
    m_int=(rand()%10+1);
    std::cout<<"Hi! I am the "<<m_name<<std::endl;
    srand(time(NULL))  };

C++

int main()
    {
        Owner breeder("breeder");
        cout<<breeder;
        Owner banker("banker");
        cout<<banker;
        Owner worker("worker");
        cout<<worker;

Результаты дохода (m_money) часто оказываются очень похожими! Результаты последних двух всегда равны, в некоторых случаях, при увеличении к-ва испытаний, результаты равны у последних трех! Часто числа в небольшом диапазоне 300-500 у всех объектов!!!
Я правильно понимаю, что это из-за использования значения времени, которое мало меняется при выполнении программы? И что делать в таком случае?

@Кот Ангенс · 10.01.2016, 19:35

Правильно. Что делать - вызывать srand только один раз, в самом начале main.

@sourcerer · 10.01.2016, 20:33

Сообщение от Кот Ангенс

Правильно. Что делать - вызывать srand только один раз, в самом начале main.

Вызывать srand() можно и в пользовательской функции. Но вызывать её надо перед вызовом функции rand(), и только один раз. Сделать это можно, указав перед srand() ключевое слово static.

C++

Owner::Owner(std::string name)
{
    static srand(time(NULL));
    m_name=name;
    m_goodness=(rand()%10+1);
    m_size=(rand()%10+1);
    m_money=rand()%1001;
    m_agility=(rand()%10+1);
    m_int=(rand()%10+1);
    std::cout << "Hi! I am the " << m_name <<std::endl;
}

@Кот Ангенс · 10.01.2016, 22:21

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

Вызывать srand() можно и в пользовательской функции. Но вызывать её надо перед вызовом функции rand(), и только один раз.

Разумеется. Я лишь предложил самый простой вариант, где это можно сделать. Вообще правильнее было бы вызывать в конструкторе приватного статического объекта-одиночки.

Код с static srand(time(NULL)); не компилируется:
C89/C99/C11/GNU89/GNU99/GNU11:

Код

test.c:21:18: error: expected declaration specifiers or ‘...’ before ‘time’

C++03/C++11/C++14/C++1y:

Код

test.cpp:12:18: error: ISO C++ forbids declaration of ‘srand’ with no type [-fpermissive]

C++ считает, что я пытаюсь объявить прототип функции srand, но забыл тип возвращаемого значения.
Не уверен, что понял правильно сообщение от компилятора C, но, вроде бы, он думает, что я забыл указать тип параметра time в прототипе функции srand.

@daslex · 10.01.2016, 23:03

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

Вызывать srand() можно и в пользовательской функции. Но вызывать её надо перед вызовом функции rand(), и только один раз. Сделать это можно, указав перед srand() ключевое слово static.

последствие моих советов?
На каждом новом запуске программы будет одна и та же последовательность. Это до random из c++11. Такое вот есть отличие.

Сообщение от Кот Ангенс

static srand(time(NULL));

C++

1	static int srand(time(NULL)); //Типа -то и не было, теперь вон он, int есть

но как я написал, на каждом новом запуске программы будут генерироваться одни и те же значения.
Сразу после main srand. Совет поста 2.

@Кот Ангенс · 10.01.2016, 23:09

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

static int srand(time(NULL));

Но ведь это означает static int srand = time(NULL);, то есть мы объявляем локальную статичную переменную srand и присваиваем ей текущее время (еще и с downcast'ом от time_t {aka long} к int).

@daslex · 10.01.2016, 23:57

Ну да. Так и есть. Не нравится? Тогда можно использовать медленный вариант.

C++

1
2
3

#include <ctime>
#include <cstdlib>
#include <windows.h>

C++

    srand(clock());
    Sleep(1);
   //Всякие рэнды
 cout << rand()%50 << ' ';

Добавлено через 1 минуту

C++

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <windows.h> //Sleep
 
using namespace std;
 
int foo() {
    srand(clock());
    Sleep(1);
    cout << rand() % 50 << ' ';
    return 0;
}
 
int main() {
 
    foo();
    foo();
 
    foo();
    foo();
 
    foo();
    foo();
 
    cin.get();
}

Добавлено через 22 минуты
static srand(time(NULL));
Не выйдет, потому как здесь происходит попытка использовать static в месте вызова функции для вызова функции как для переменной, а не для обозначения переменной как переменной.
Это так же как вот эта, весьма странная конструкция (не рабочая и не должна работать)

C++

void foo(){}
int main() {
    static foo();
}

@sourcerer · 11.01.2016, 06:12

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

последствие моих советов?

Ахахаха, точняк

Ну, тогда, вот такой костыль:

C++

Owner::Owner(std::string name)
{
    static bool first_time_in_this_program = true;
    if (first_time_in_this_program)
    {
        srand(time(NULL));
        first_time_in_this_program = false;
    }
    m_name=name;
    m_goodness=(rand()%10+1);
    m_size=(rand()%10+1);
    m_money=rand()%1001;
    m_agility=(rand()%10+1);
    m_int=(rand()%10+1);
    std::cout << "Hi! I am the " << m_name <<std::endl;
}

Добавлено через 1 минуту
Или, как верно сказал Кот Ангенс, использовать Singleton.

@Кот Ангенс 320 / 270 / 128 Регистрация: 24.05.2012 Сообщений: 629
	10.01.2016, 19:35	2
	Сообщение было отмечено parsecer как решение Решение Правильно. Что делать - вызывать `srand` только один раз, в самом начале `main`. 0

@Кот Ангенс 320 / 270 / 128 Регистрация: 24.05.2012 Сообщений: 629
	10.01.2016, 22:21	4
	Сообщение от gru74ik Вызывать srand() можно и в пользовательской функции. Но вызывать её надо перед вызовом функции rand(), и только один раз. Разумеется. Я лишь предложил самый простой вариант, где это можно сделать. Вообще правильнее было бы вызывать в конструкторе приватного статического объекта-одиночки. Код с `static srand(time(NULL));` не компилируется: C89/C99/C11/GNU89/GNU99/GNU11: Код test.c:21:18: error: expected declaration specifiers or ‘...’ before ‘time’ C++03/C++11/C++14/C++1y: Код test.cpp:12:18: error: ISO C++ forbids declaration of ‘srand’ with no type [-fpermissive] C++ считает, что я пытаюсь объявить прототип функции srand, но забыл тип возвращаемого значения. Не уверен, что понял правильно сообщение от компилятора C, но, вроде бы, он думает, что я забыл указать тип параметра time в прототипе функции srand. 0

@Кот Ангенс 320 / 270 / 128 Регистрация: 24.05.2012 Сообщений: 629
	10.01.2016, 23:09	6
	Сообщение от daslex static int srand(time(NULL)); Но ведь это означает `static int srand = time(NULL);`, то есть мы объявляем локальную статичную переменную `srand` и присваиваем ей текущее время (еще и с downcast'ом от `time_t {aka long}` к `int`). 0

	11.01.2016, 06:12

Функция rand() дает странные результаты

Решение