Инициализация динамического массива структур

Question

Это моя структура:

typedef struct ElementToInsert { 
    char *key; 
    char *value; 
} element; 

Это моя функция:

void init_hash(int size, element**arr) { 
    *arr = malloc(size * sizeof(element*)); 
    if (arr == NULL) 
    { 
     printf("Out of memory\n"); 
     return 0; 
    } 
    for (int i = 0; i < size; i++) { 

     (*arr)[i].key = NULL; 
     (*arr)[i].value = NULL; 
    } 

} 

И main():

void main() { 
    element *hash_table = NULL; 
    int hash_table_size = 10; 
    init_hash(hash_table_size, &hash_table); 
} 

Проблема заключается в том, что, когда я запустите отладчик (в visual studio 2015) он не покажет мне все значения массива. Я ожидал увидеть что-то вроде: hash_table[0] = {key = NULL, value = NULL}, hash_table[1] = {key = NULL, value = NULL}, но я вижу это:

Answer 1

Передача двойной указатель на init_hash является излишне сложным. Хотя существует много действительных применений для двойных указателей, это не один из них. Обычно двойные указатели используются для нескольких выходных переменных или для замены существующей памяти, но поскольку у вас нет возвращаемого значения, и вы все равно выделяете память, оно просто вызывает ошибки. Держите вещи простыми и верните указатель на свой список элементов.

element *init_hash(size_t size) { 
    element *hash = calloc(size, sizeof(element)); 

    if (hash == NULL) { 
     fprintf(stderr, "Out of memory\n"); 
     exit(1); 
    } 

    return hash; 
} 

Несколько примечаний. Во-первых, чтобы выделить массив структур, вы выделяете size * sizeof(element). Это выделяет память для size структур. Если вы используете size * sizeof(element*), который выделяет память для sizeУказатели на структуру. Оба действительны, но вы должны выбрать, с кем работаете. Кажется, вы работаете со списком структур.

В отличие от malloc, calloc будет нулевать память для вас. Это позволяет избежать необходимости вручную нуля каждой структуры, сохраняющей некоторый код и время.

Если у вас закончилась нехватка памяти, и у вас нет плана по ее устранению, вероятно, лучше всего сэкономить массу последующих сообщений об ошибках и exit.

И, наконец, размеры лучше всего хранить как size_t. Это тип, который C использует для размеров массива, и он может поймать некоторые проблемы с типом.