C программируемые указатели массивов read file

Question

Строка *array[cnt] = thing вызывает ошибку seg, и я не знаю почему. Любые идеи, чтобы исправить это?

long *Load_File(char *Filename, int *Size) 
{ 
    FILE *fp; 

    if((fp = fopen(Filename,"r")) == NULL) 
    { 
     printf("Cannot open file.\n"); 
     exit(-1); 
    } 

    fscanf(fp,"%d",Size); 

    int cnt = 0; 
    int items = *Size; 
    long * array[items]; 
    int thing; 

    while (!feof(fp)) 
    { 
     fscanf(fp,"%d",&thing); 
     *array[cnt] = thing; 
     cnt++; 
    } 

    fclose(fp); 

    return *array; 
} 

Answer 1

long * array[items]; 

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

Когда вы

*array[cnt] = thing; 

вы разыменования указателя, который является неправильным, так как они не указывают на что-либо значимое.

Вы можете динамически выделить память для массива, как:

long * array = (long*) malloc(size(long) * items); 

, а затем сделать:

while (!feof(fp)) { 
     fscanf(fp,"%d",&arr[cnt++]); 
    } 

, а затем возвращают массив как:

return array; 

Answer 2

Во-первых, этот код не может скомпилировать. Поскольку items не является константой, его нельзя использовать для определения размера массива. Как вы добрались до него, не говоря уже о seg-fault? Помимо этого, и в дополнение к проблеме @codaddict подчеркивает ...

feof не вернется с ошибкой до тех пор, пока fscanf не сработает. Это приведет вас к концу массива. Лучше написать так:

while (cnt < items && fscanf(fp, "%d", &thing)) 
{ 
    /* ... */ 
} 

WRT массив, я думаю, что вы хотели это:

long *array = malloc(sizeof(long)*items); 
/* ... */ 
    array[cnt] = thing; 
/* ... */ 
return array; 

Answer 3

Изменить

long * array[items]; 

в

long * array = (long *) malloc(sizeof(long) * items); 

Мы динамически выделять память для длинные предметы и храните адрр ess в нашей переменной массива. Ваш синтаксис означает «массив указателей на длину». Новый синтаксис означает «указатель на длинный» (первый из динамического «массива»).

Изменить

*array[cnt] = thing; 

в

array[cnt] = thing; 

Ставим последний читал долго в правильном месте.

Изменить

return *array; 

в

return array; 

Возвращаемся массив, который является таким же, как указатель на первый в памяти надолго зарезервированное место. Обязательно укажите free позже.

EDIT:

Благодаря ergosys за напоминание мне, что Власа выделяется в стеке. Удалены предлагаемые изменения в заголовке функции.