libre() un tableau de char **

Question

je ceci:

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

Initialiser le tableau de la bibliothèque:

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); 
    } 
} 

Ici je remplir le tableau 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); 
} 

et une fonction comme celle-ci:

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; 
} 

J'ai obtenu la taille de mon tableau sur ma structure pour obtenir la limite de mon tableau, et libéré toutes les cellules utilisées, mais chaque fois que j'appelle CleanDico, le code se bloque. Aucune suggestion?

J'ai déjà demandé a question on a problem with a char array. Maintenant, je veux free() le tableau. J'ai lu beaucoup de posts ici et là et testé beaucoup de solutions mais aucune m'a aidé à résoudre mon problème.

Je reçois un SEGMENTATION_FAULT sur CleanDico, et je n'ai pas d'autres informations sur l'erreur. Déboguer mod sur Code :: Blocks sont pauvres avec des messages d'erreur.

Answer 1

Je pense que votre deuxième pour vous l'intention de faire

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

au lieu de

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

Answer 2

Chaque malloc(), calloc() ou realloc() doit être associé à un free() correspondant (ou realloc()).

Vous avez ceci:

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

Ce qui vous donne un tableau de pointeurs, initialisés (ils contiennent des déchets).

Plus tard, vous avez ceci:

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

un pointeur qui désalloue vous n'initialisés. Et ne parvient pas à libérer ce que vous avez attribué. En outre, vous pouvez envisager une structure de données différente: allouer un grand tableau de char qui contient tous les mots, plus un tableau de char* qui contient un pointeur sur chaque mot. De cette façon, vous avez toujours exactement deux allocations et désallocations (à moins que vous ne connaissiez la taille maximale de tous vos mots, auquel cas vous pourriez avoir besoin de realloc() occasionnellement).

Answer 3

Votre boucle for interne incrémente également la variable i. Donc ça sort de l'index.

EDIT: la boucle externe devrait fonctionner 26 fois puisque vous avez 26 lettres.

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; 
}