генерация случайных массивов в c ++ [дубликаты]

Question

генерация случайных массивов в c ++ [дубликаты]

0

Я пытаюсь написать простую функцию, которая генерирует массив со случайными целыми числами.

Я столкнулся с интересной задачей, когда я запускаю эту программу, выходы -each cout statements- кажутся одинаковыми.

Тем не менее, когда я отлаживаю и смотрю его, пошаговые значения массива меняются.

Я делаю что-то неправильно?

Благодарю.

void generateRandomArray( int *&inputArray, int size){

    if(inputArray != NULL){
        delete []inputArray;

    }

    //create a new array
    inputArray = new int [size];

    //fill with random numbers   
    srand(unsigned (time (NULL)));
    for (int i = 0; i < size ; i++)     
        inputArray[i] = (rand() % 100) ;

}
int main(){

    //Variables
    int *inputArray = NULL;
    int size;


    //Test
    size = 10;

    //first call
    generateRandomArray( inputArray, size);

    for(int i = 0 ; i < size; i++){
        cout << inputArray[i] << endl;
    }

    cout << "------------------" << endl;

    //second call
    generateRandomArray( inputArray, size);

    //output is the same with previous
    for(int i = 0 ; i < size; i++){
        cout << inputArray[i] << endl;
    }
    return 0;
}

selpak 12 июль 2014, в 00:22

Источник

7

Позвони srand один раз.
chris 11 июль 2014, в 21:31
2

Вызов srand(time(NULL)); сразу после запуска
Martin York 11 июль 2014, в 21:31
1

Не используйте эту устаревшую C-версию. en.cppreference.com/w/cpp/numeric/random
n0rd 11 июль 2014, в 21:32
0

Как только это работает. Перейти к обзору кода, чтобы получить информацию о лучших методах.
Martin York 11 июль 2014, в 21:34
1

Если вы во время отладки пошагово выполняете код, вы увидите тот же результат, что и без подключенного отладчика :)
Praetorian 11 июль 2014, в 21:35
1

Прочитайте о новых псевдослучайных классах и std::generate и std::generate_n .
Some programmer dude 11 июль 2014, в 21:35
1

@sp2danny: sp2danny: параметр является ссылкой на указатель. (Типы читаются справа налево). Указатель на ссылку недопустим и не будет компилироваться.
Martin York 11 июль 2014, в 21:35
0

Кстати, используйте std::vector вместо необработанного массива.
Jarod42 11 июль 2014, в 21:37
0

но в чем причина разницы между выводом отладки и фактическим выводом?
selpak 11 июль 2014, в 21:38
0

время, которое требуется, и следующая секунда произведет новое значение
sp2danny 11 июль 2014, в 21:39
1

хотя не выполняется отладка, оба обращения ко time(NULL) происходят в течение одного такта, поэтому для каждого вызова генерируется одна и та же последовательность. Во время отладки между вызовами проходит некоторое время, поэтому генерируется другая последовательность.
n0rd 11 июль 2014, в 21:40

Показать ещё 9 комментариев

Теги:

c++

arrays

4 ответа

Ещё вопросы

Вызов srand(time(NULL)); сразу после запуска
Не используйте эту устаревшую C-версию. en.cppreference.com/w/cpp/numeric/random
Как только это работает. Перейти к обзору кода, чтобы получить информацию о лучших методах.
Если вы во время отладки пошагово выполняете код, вы увидите тот же результат, что и без подключенного отладчика :)
Прочитайте о новых псевдослучайных классах и std::generate и std::generate_n .
@sp2danny: sp2danny: параметр является ссылкой на указатель. (Типы читаются справа налево). Указатель на ссылку недопустим и не будет компилироваться.
Кстати, используйте std::vector вместо необработанного массива.
но в чем причина разницы между выводом отладки и фактическим выводом?
время, которое требуется, и следующая секунда произведет новое значение
хотя не выполняется отладка, оба обращения ко time(NULL) происходят в течение одного такта, поэтому для каждого вызова генерируется одна и та же последовательность. Во время отладки между вызовами проходит некоторое время, поэтому генерируется другая последовательность.

Jarod42 · Answer 1 · 2014-07-11T21-17-00.000Z

С С++ 11 вы можете использовать что-то вроде: (https://ideone.com/EIb0nY)

#include <iostream>
#include <random>
#include <vector>

std::vector<int> generateRandomArray(std::default_random_engine& generator, int size)
{
    std::uniform_int_distribution<int> distribution(1, 100);
    std::vector<int> res(size);

    for (auto& e : res) {
        e = distribution(generator);
    }
    return res;
}

int main()
{
    std::default_random_engine generator;
    const int size = 10;

    //first call
    for (auto e : generateRandomArray(generator, size)) {
        std::cout << e << std::endl;
    }

    std::cout << "------------------" << std::endl;

    //second call
    for (auto e : generateRandomArray(generator2, size)) {
        std::cout << e << std::endl;
    }
    return 0;
}

Это использует новые случайные объекты.
Это позволяет избежать ручного управления памятью (с помощью std::vector).

Hoang Minh · Answer 2 · 2014-07-11T20-28-00.000Z

Вызовите srand(time(NULL)) и оно будет высевать текущее время и сделать из него случайное число. Поскольку время меняется все время, оно будет давать вам разное количество каждый раз.

Some programmer dude · Answer 3 · 2014-07-11T20-03-00.000Z

Проблема в том, что если вызовы srand все происходят с той же секундой, генератор будет засеяться с одинаковыми значениями и поэтому генерирует ту же последовательность, всегда.

Когда вы отлаживаете, он будет медленнее и позволит время проходить и иметь разное семя для генератора. Если отладка не выполняется, время между вызовами будет слишком быстрым.

Не вызывайте srand более одного раза, если вы не знаете, что делаете. Поместите его в начало main функции и не вызывайте его снова.

Dieter Lücking · Answer 4 · 2014-07-11T19-04-00.000Z

Программа выполняется слишком быстро, srand(unsigned (time (NULL))); получит семя с теми же секундами (с эпохи). Следовательно, rand последовательности одинаковы.