Каков самый чистый способ инициализации указателей в C?

Question

Я хочу иметь функцию C, которая читает файл и возвращает динамический массив из int. Каков наилучший способ инициализации указателей для этого массива?

Моя текущая версия:

void readFile(int* array,int* size){ 
    // ...read the file and see there are X integers... 
    array = malloc(X * sizeof(int)); 
    *size = X; 
    // ... fill the array ... 
    } 
int main() { 
    int* array; 
    int size; 
    readFile(array,&size); 
    //Work with array 
    return 0; 
} 

Это дает мне инициализированы предупреждения, это нормально, просто сделать

int* array = NULL 

или есть лучше и чище способ для достижения этой цели?

Answer 1

Технический комментарий

Во-первых, у вас возникли проблемы, потому что readFile принимает int *, не int **. Если у вас

void readFile(int **array, int *size) { 
    *array = malloc(X * sizeof(int)); 
    ... 

вы на самом деле быть в состоянии назвать это как в

int main() { 
    int *array, size; 
    readFile(& array, & size); 
    ... 

так вызывающего абонента array будет указывать на то, что readFile выделено.

Дизайн вопрос

К вашему конкретному вопросу, технически и возможно, до тех пор, как ответственность за инициализации/выделения и очистки понятны:

• main выделяет массив, и readFile заливок Это.
• readFile выделяет и заполняет массив содержимым вашего файла.

main может выделить память, потому что это функция, которая будет знать, когда очищать (освобождать память). В противном случае блок, в котором реализован readFile, также может реализовать функцию freeFile для других пользователей.