malloc() a l'aide d'une fonction + tab2D

je veus creer un tableau 2 dimensions dynamiquement, à l'aide de la focntion malloc... Mon code fonctionne très bien dans mon main() mais quand j'essaye de le mettre dans une fonctione creer_tab_dynamique, je n'y arrive pas.

En voici mon code de mon main :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

tab = malloc (nb_lig * sizeof (* tab));   if (tab != NULL) { int i = 0; for (i = 0; i < nb_lig; i++) { tab[i] = malloc (nb_col * sizeof (**tab)); } }

---

Citation:

---

NB: il manque la gestion si la 2eme allocation de la 2nd dimension a bien été réaliser ou non...

---

Aussi, si la 2eme allocation dynamique échou, j'ai jsute besoin de libérer la 1ere allocation ? :

Code:

---

1 2	free(*tab)

---

Je ne vois pas de problème:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

char ** creer_tab (size_t nb_lig, size_t nb_col) { char ** tab = malloc (nb_lig * sizeof (* tab));   if (tab != NULL) { size_t i = 0; for (i = 0; i < nb_lig; i++) { tab[i] = malloc (nb_col * sizeof (**tab)); } }   return tab; }

---

Citation:

---

Envoyé par nicodn02

Aussi, si la 2eme allocation dynamique échou, j'ai jsute besoin de libérer la 1ere allocation ? :

Code:

---

1 2	free(*tab)

---

---

Non, pour chaque allocation dans ta boucle il faut un free, la règle en C est simple: 1 *alloc = 1 free ;)

Par exemple:

Code:

---

1 2 3 4 5 6 7 8 9

for (i = 0; i < nb_lig; i++) { if (tab[i] != NULL) { free (tab[i]); } } free (tab);

---

pourquoi as tu déclaré la fonction en tant que char ** creer_tab_dynamique ?
( ** )

puis-je remplacer les size_t par des int ?

Citation:

---

Envoyé par nicodn02

pourquoi as tu déclaré la fonction en tant que char ** creer_tab_dynamique ?
( ** )

---

Bin si tu veux un tableau de chaîne dynamique c'est un pointeur sur pointeur sur char qu'il te faut, comme il faut un pointeur sur char pour créer une chaîne dont l'espace est alloué dynamiquement ! Donc si tu veux retourner ce tableau de chaîne, c'est ainsi qu'il faut procéder ;)

Citation:

---

Envoyé par nicodn02

puis-je remplacer les size_t par des int ?

---

Non, il est préférable d'utiliser ce type pour les données de taille et d'index, c'est d'ailleurs le type attendu par malloc ;)

De plus, après j'ai dautre fonction qui vont interagir avec le tableau, donc je ne voit pas comment faire dans mon main .... moi j'avais penser a :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

int main(){ /*Déclaration du tableau 2D */ char ** tab = NULL; /*Déclaration des variables des dimensions */ int nb_col=0, nb_lig=0;   /*Demande des dimension a l'utilisateur */ scanf("%d", &nb_col); scanf("%d", &nb_lig);   /*Appel de la fonction pour creer le tableau */ /* Je ne sais pas ici que marquer... j'orai penser a **tab=**creer_tab(nb_col, nb_lig); */   initialisation_grille(tab, nb_col, nb_lig);   draw_grille(tab, nb_col, nb_lig);   ...   return 0; }

---

Mais je pense que mes initialisation_grille et draw_grille, il ne faut pas mettre dedans **tab ?( a la place de tab)

Citation:

---

les size_t fonctionne comme des int ?
#include<stdlib.h> (malloc avec size_t et free a l'interieur de cette bibliotheque, non ? )

---

Bin fait simplement un appel du genre:

Code:

---

tab = creer_tab_dynamique (nb_lig, nb_col);

---

Tout simplement quoi :lol:

Citation:

---

les size_t fonctionne comme des int ?

---

Bin c'est un

Code:

---

unsigned int

---

tout simplement, du moins suivant les implémentations je pense mais c'est le cas général me semble-t-il.

Et en ce qui concerne mes autres fonction, qui sont initalisation et draw, voici leur déclaration :

Code:

---

1 2 3

void initialisation_grille(char **grille, nb_lig, nb_col); void draw_grille(char **grille, nb_lig, nb_col);

---

Est-ce bien cela?

Citation:

---

Envoyé par nicodn02

Et en ce qui concerne mes autres fonction, qui sont initalisation et draw, voici leur déclaration :

Code:

---

1 2 3

void initialisation_grille(char **grille, nb_lig, nb_col); void draw_grille(char **grille, nb_lig, nb_col);

---

Est-ce bien cela?

---

Tant que tu ne modifies pas la taille du tableau, cela ne devrait pas poser de problèmes particuliers.

Et en ce qui concerne maintenant les free
on dit que la 1ere allocation échoue, je fait :

Code:

---

1 2	free(*tab);

---

et si la deuxieme allocation echou, je libere le tout :

Code:

---

1 2 3

free(**tab); free(*tab);

---

non?

Non, je t'ai montré plus haut comment il faut procéder pour une libération correcte suivant ce type d'allocation !

Donc meme si la 2eme allocation a echouer, il faut que je fasse comme si elle avait réussit et que je libere tout ?

Citation:

---

Envoyé par nicodn02

Donc meme si la 2eme allocation a echouer, il faut que je fasse comme si elle avait réussit et que je libere tout ?

---

Bin si une seule allocation échoue, il faut de toute manière tout libérer sous peine d'avoir des comportements indéterminés par la suite dans l'exécution de ton programme. Ici:

Code:

---

1 2 3 4 5 6 7 8

for (i = 0; i < nb_lig; i++) { if (tab[i] != NULL) { free (tab[i]); } } free (tab);

---

Je libère tout mais je test dans la boucle for à chaque itération si l'adresse courant est valide ou non, si elle l'est alors je la libère sinon je passe à la suivante. Ceci libère les espaces pour les chaînes de caractères dans ton tableau de chaînes. Une fois que la boucle est terminée, je libère le tableau lui même tout simplement.

Il est préférable de procéder ainsi car une allocation peut très bien échouer en plein milieu du tableau !

Merci