бесплатно() массив гольца **

Question

У меня есть это:

struct Library { 
    char letter; 
    int capacity; 
    int size; 
    char** words; 
}; 
typedef struct Library Lib; 

Инициализировать массив библиотеки:

void InitLibrary(Program* startup) { 
    char alphabet[26] = {'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z'}; 
    startup->dictionary = malloc(sizeof(Lib) * 26); 
    int i; 
    for (i = 0; i < 26; i++) { 
     startup->dictionary[i].letter = alphabet[i]; 
     startup->dictionary[i].capacity = INIT_CAPACITY; 
     startup->dictionary[i].size = 0; 
     startup->dictionary[i].words = malloc(sizeof(char*) * startup->dictionary[i].capacity); 
    } 
} 

Здесь я заселить массив words:

void FillDicoFromFile(Program* startup){ 
    while((!feof(startup->f) && !ferror(startup->f))){ 
     char* word = malloc(sizeof(char) * 30); 
     fscanf(startup->f, "%s", word); 
     ToLowerCase(word); 
     int indexLib = word[0] - 97; 
     int sizeLib = startup->dictionary[indexLib].size; 
     startup->dictionary[indexLib].words[sizeLib] = word; 
     startup->dictionary[indexLib].size++; 
    } 
    CountTotalWords(startup); 
    rewind(startup->f); 
} 

и такая функция:

void CleanDico(Program* startup){ 
    int i = 0; 
    for(; i < startup->dictionary[i].size; i++){ 
     int j = 0; 
     for(; j < startup->dictionary[i].size; i++){ 
      free(startup->dictionary[i].words[j]); 
      startup->dictionary[i].words[j] = NULL; 
     } 
     startup->dictionary[i].size = 0; 
    } 
    startup->totalWords = 0; 
} 

Я получил размер моего массива в своей структуре, чтобы получить предел моего массива, и освободил все используемые ячейки, но каждый раз, когда я вызываю CleanDico, код падает. Какие-либо предложения?

Я уже спросил a question on a problem with a char array. Теперь я хочу free() массив. Я читал много сообщений здесь и там и тестировал множество решений, но ни один из них не помог мне решить мою проблему.

Я получаю SEGMENTATION_FAULT на CleanDico, и у меня нет никакой другой информации об ошибке. Отладочный мод по коду :: Блоки с ошибками.

Answer 1

Я думаю, что на вашем секунду вы имели в виду, чтобы сделать

for(; j < startup->dictionary[j].size; j++){ 

вместо

for(; i < startup->dictionary[i].size; i++){ 

Answer 2

Каждый malloc(), calloc() или realloc() должны быть согласованы с соответствующим free() (или realloc()).

У вас есть это:

startup->dictionary[i].words = malloc(sizeof(char*) * N); 

Что дает вам массив указателей, инициализируется (они содержат мусор).

Позже у вас есть это:

free(startup->dictionary[i].words[j]); 

Который освобождает указатель вы никогда не инициализирована. И не удается освободить то, что вы выделили.

В качестве альтернативы вы можете рассмотреть другую структуру данных: выделить один огромный массив из char, который содержит все слова, плюс один массив из char*, который содержит по одному указателю на каждое слово. Таким образом, вы всегда имеете ровно два распределения и освобождения (если вы не знаете максимальный размер всех ваших слов, в этом случае вам может понадобиться realloc()).

Answer 3

Ваш внутренний цикл цикла также увеличивает значение переменной i. Таким образом, он выходит из индекса.

EDIT: внешний контур должен работать 26 раз, так как у вас есть 26 букв.

for(; i < 26; i++){ 
    int j = 0; 
    for(; j < startup->dictionary[i].size; j++){ 
     free(startup->dictionary[i].words[j]); 
     startup->dictionary[i].words[j] = NULL; 
    } 
    startup->dictionary[i].size = 0; 
}