Discussion :

[nicodn02]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
free(*tab)

[Franck.H]

Je ne vois pas de problème:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Franck.H]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[nicodn02]

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

puis-je remplacer les size_t par des int ?

[Franck.H]

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

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

[nicodn02]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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)

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

[Franck.H]

Bin fait simplement un appel du genre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

tab = creer_tab_dynamique (nb_lig, nb_col);

Tout simplement quoi

[Franck.H]

les size_t fonctionne comme des int ?

Bin c'est un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

unsigned int

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

[nicodn02]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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?

[Franck.H]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[nicodn02]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
free(*tab);

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
free(**tab);
free(*tab);

non?

[Franck.H]

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

[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 ?

[Franck.H]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !

[nicodn02]

Merci