Почему ГПСЧ нужно инициализировать в main()?

Question

[HoShik]

Имеется такой код:

#include <stdio.h>
#include <limits.h>
#include <stdlib.h>
#include <locale.h>
int fill_array(int array[], int l_gen, int u_gen, int size) 
{
    int sum=0;
    for(int i=0; i<size; i++) 
    {
        array[i]=l_gen+rand()%(u_gen-l_gen+1);
        sum+=array[i];
    }
    return sum;
}
void print_array(int array[], int size) 
{
    for(int i=0; i<size; i++) 
    {
        printf("%d ", array[i]);
    }
}
int main(void)
{   
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
    printf("Введите через пробел нижнюю границу генерации, верхнюю и размер массивов: ");
    int l_gen, u_gen, size, sum_max=INT_MIN+1, position;
    scanf("%d %d %d", &l_gen, &u_gen, &size);
    int array[size], big_array[size];
    for(int i=0, current_sum; i<10; i++) 
    {
        current_sum=fill_array(array,l_gen,u_gen,size);
        printf("Сумма массива под номером %d равна %d\n", i+1,current_sum);
        if(current_sum>sum_max) 
        {
            sum_max=current_sum;
            for(int j=0; j<size; j++) 
            {
                big_array[j]=array[j];
            }
            position=i+1;
        } 
    }
    printf("Массив под номером %d имеет самую наибольшую сумму (%d) из всех созданных\n", position, sum_max);
    print_array(big_array, size);
}

Инициализация ГПСЧ через srand в функции main приводит к правильной работе программы. Если инициализировать ГПСЧ в fill_array, происходит ошибка. Кто сможет разъяснить, с чем это связано? Пример неправильного вывода:
Сумма массива под номером 2 равна 73
Сумма массива под номером 3 равна 73
Сумма массива под номером 4 равна 73
Сумма массива под номером 5 равна 73
Сумма массива под номером 6 равна 73
Сумма массива под номером 7 равна 73
Сумма массива под номером 8 равна 73
Сумма массива под номером 9 равна 73
Сумма массива под номером 10 равна 76

Answer 1

[GavriKos]

Не должно быть ошибки. Если вы только не в цикле его инициализируете. Приведите код который приводит к ошибке.

Comments

[HoShik]

В main убираем инициализацию и проводим её в fill_aray

int fill_array(int array[], int l_gen, int u_gen, int size) 
{
    srand(time(NULL));
    int sum=0;
    for(int i=0; i<size; i++) 
    {
        array[i]=l_gen+rand()%(u_gen-l_gen+1);
        sum+=array[i];
    }
    return sum;
}

[HoShik]

Кажется понял, в чем ошибка. fill_array я вызываю в цикле из main и происходит такая ситуация. Но не до конца понимаю почему так критична ин-ция в цикле

[Евгений Шатунов]

HoShik, потому что так работает ГПСЧ. Он не столько инициализируется, сколько принимает некоторое зерно [?], от которого начинает строить последовательность псевдо-случайных чисел.
При установке зерна происходит сброс состояния генератора. Если ты в цикле постоянно устанавливаешь одно и то же зерно, ты в результате получаешь ровно одну и ту же цепочку псевдо-случайных чисел внутри каждой итерации цикла.

Обрати внимание на то как работает функция time [?]. В системах, которые придерживаются спецификации POSIX, результат можно интерпретировать как количество секунд. Т.е. раз в секунду результат time( nullptr ) будет меняться. Именно это и видно в приведенном тобой листинге.

Сумма массива под номером 9 равна 73
Сумма массива под номером 10 равна 76

9й еще был в одной секунде, а 10й - уже в следующей.

[HoShik]

Евгений Шатунов, Понял, благодарю

Answer 2

[Николай]

Потому что в srand() будет передаваться одинаковое число. Функция time возвращает число секунд от начала 1970 года, её значение меняется раз в секунду, а цикл программы выполняется намного быстрее.