Allocation dynamique d'un tableau de caractères dans une procédure

Version imprimable

Bonjour,

J'ai quelques difficultés avec les pointeurs alors je me suis inspiré d'une explication de koala01 pour mon code. La différence est qu'il le fait sur un float alors que je souhaite le faire sur un tableau de char 2 dimensions.
http://www.developpez.net/forums/d19...s/#post1284261

J'ai réussi à faire l'équivalent de la fonction pour mon tableau de chaines de caractères mais je n'arrive pas à faire la procédure.

Voici mon code: (j'utilise devcpp pour compiler)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 #include <stdio.h> #include <malloc.h> ... //prototypes void recupConfig(FILE *, char ***, char ***, int *, char ***, int *); ... int main() { FILE *fichierConf; char ***chemins=NULL; char ***extensions=NULL; char ***mots=NULL; int *nbJoursApresZip; int nbElements[4]; ... recupConfig(fichierConf, &chemins, &mots, &nbJoursApresZip, &extensions, nbElements); ... return 0; } void recupConfig(FILE *fichier,char ***listeChemins, char ***listeMots, int *nbJAZ, char ***listeExtensions, int nbElements[]) { int i; *listeChemins = (char **)malloc(50*sizeof(char *)); if(*listeChemins!=NULL) printf("mallocOK\n"); for(i=0;i<50;i++) { *listeChemins[i] = (char *)malloc(100*sizeof(char)); printf("TestMalloc\n");getch(); } ... }
A l'exécution du programme j'obtiens:
mallocOK
TestMalloc
TestMalloc
TestMalloc
Et là ça bloque et ça quitte.

Je suppose que je n'alloue pas au bon endroit en mémoire mais je ne comprends pas pourquoi. Quelqu'un peut-il m'aider?

Merci!

Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 void recupConfig(FILE *, char ***, char ***, int *, char ***, int *); ... int main() { FILE *fichierConf; char ***chemins=NULL; char ***extensions=NULL; char ***mots=NULL; int *nbJoursApresZip; int nbElements[4]; ... recupConfig(fichierConf, &chemins, &mots, &nbJoursApresZip, &extensions, nbElements); ... return 0; }
Les types ne concordent pas. &chemins est un char **** d'après la définition de chemins. Idem pour les autres paramètres.

Citation:

Envoyé par Ricks

J'ai quelques difficultés avec les pointeurs alors je me suis inspiré d'une explication de koala01 pour mon code. La différence est qu'il le fait sur un float alors que je souhaite le faire sur un tableau de char 2 dimensions.

Code:

#include <malloc.h>

n'existe pas. C'est

Code:

#include <stdlib.h>

Code:

getch ();

N'est pas standard.

J'ai remis ton code en ordre de façon à ce que les paramètres soient cohérents (trop d'*', ça sent le pifomètre tout ça...)...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
#include <stdio.h>
#include <stdlib.h>
 
void recupConfig (FILE * fichier, char ***listeChemins, char ***listeMots,
                  int *nbJAZ, char ***listeExtensions, int nbElements[])
{
   int i;
   *listeChemins = malloc (50 * sizeof (char *));
   if (*listeChemins != NULL)
      printf ("mallocOK\n");
   for (i = 0; i < 50; i++)
   {
      *listeChemins[i] = malloc (100 * sizeof (char));
      printf ("TestMalloc\n");
   }
 
   (void) fichier;
   (void) listeMots;
   (void) nbJAZ;
   (void) listeExtensions;
   (void) nbElements;
}
 
int main (void)
{
   FILE *fichierConf = NULL;
   char **chemins = NULL;
   char **extensions = NULL;
   char **mots = NULL;
   int nbJoursApresZip;
   int nbElements[4];
 
   recupConfig (fichierConf, &chemins, &mots, &nbJoursApresZip, &extensions,
                nbElements);
 
   return 0;
}

Effectivement, ça crashe sévèrement...

Toutes ces '*', ça rend fou. Je ne fais jamais comme ça. J'utilise des structures, c'est beaucoup plus clair (et pas limité).

Recodé selon ce principe et une utilisation plus claire de malloc() :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
#include <stdio.h>
#include <stdlib.h>
 
struct data
{
   FILE *fichierConf;
   char **chemins;
   char **extensions;
   char **mots;
   int nbJoursApresZip;
   int nbElements[4];
 
};
 
void recupConfig (struct data *p_data)
{
   p_data->chemins = malloc (50 * sizeof *p_data->chemins);
 
   if (p_data->chemins != NULL)
   {
      int i;
      printf ("mallocOK\n");
 
      for (i = 0; i < 50; i++)
      {
         p_data->chemins[i] = malloc (100 * sizeof *p_data->chemins[i]);
         printf ("TestMalloc\n");
      }
   }
}
 
int main (void)
{
   struct data data = { 0 };
 
   recupConfig (&data);
 
   return 0;
}

Curieusement, ça marche du premier coup...

18/08/2008, 16h13
Ricks

J'ai aussi stdlib.h dans mes includes. Pour getch(), je l'utilise souvent sans problème. Je pense que le problème vient des pointeurs. J'ai essayé avec char *** dans la procédure et char ** dans le main mais il y a des warning à la compilation.

Code:

1
2
3
4
5
   for (i = 0; i < 50; i++)
   {
      (*listeChemins)[i] = malloc (100 * sizeof (char));
      printf ("TestMalloc\n");
   }

18/08/2008, 16h22
Emmanuel Delahaye

Citation:

Envoyé par Ricks

J'ai aussi stdlib.h dans mes includes.

Etrange. Pourquoi tu utilises <malloc.h> ? C'est pas très logique...

Citation:

Pour getch(), je l'utilise souvent sans problème.

Oui, mais ce n'est pas portable. Si tu changes de machine ou compilateur, ça peut de plus fonctionner...

Citation:

Je pense que le problème vient des pointeurs. J'ai essayé avec char *** dans la procédure et char ** dans le main mais il y a des warning à la compilation.

Oui. Une réponse a été apportée. Trop d'*', ça rend gaga...
18/08/2008, 16h22
Ricks

J'utiliserais bien volontiers les structures si cela permet d'allouer et de remplir plusieurs chaines dans la procédure mais je sens que je vais m'emmêler les pinceaux dans la suite du programme car je ne suis pas habitué à utiliser des structures.
C'est pourquoi j'ai essayé le passage par pointeurs :?
18/08/2008, 16h26
Ricks

Citation:

Envoyé par Emmanuel Delahaye

Etrange. Pourquoi tu utilises <malloc.h> ? C'est pas très logique... Oui, mais ce n'est pas portable. Si tu changes de machine ou compilateur, ça peut de plus fonctionner... Oui. Une réponse a été apportée. trop d'*', ça rend gaga...

Je ne savais pas que stdlib.h était utilisée pour l'allocation dynamique.
Merci pour l'info sur getch(). Je ne savais pas. Je ne m'en sers que pour débugger. Je les retire après.
Désolé pour les *** je rédigeais en même temps que toi.
18/08/2008, 16h29
Emmanuel Delahaye

Citation:

Envoyé par Ricks

J'utiliserais bien volontiers les structures si cela permet d'allouer et de remplir plusieurs chaines dans la procédure mais je sens que je vais m'emmêler les pinceaux dans la suite du programme car je ne suis pas habitué à utiliser des structures.
C'est pourquoi j'ai essayé le passage par pointeurs :?

L'utilisation des pointeurs doubles ou triples et beaucoup plus complexe. Preuve est faite, non ?

L'usage des structures impose un peu de réflexion préalable (organisation du code, des données, découpe fonctionnel...), mais c'est l'approche professionnelle.

Citation:
Envoyé par stephl
Code:

1 2 3 4 5 for (i = 0; i < 50; i++) { (*listeChemins)[i] = malloc (100 * sizeof (char)); printf ("TestMalloc\n"); }
Tu as peut être trouvé le problème stephl. Je suis en train de tester.

18/08/2008, 16h36
Ricks

A Stephl : C'est bon ça ne plante plus et je récupère bien le contenu des chaines dans le main.

A Emmanuel Delahaye : Je suis encore novice en structure. Je me pencherai dessus plus en détails pour améliorer ma façon de programmer.

Merci à vous deux pour les conseils et la résolution de mon problème!

PS : saleté de parenthèses :lol:
18/08/2008, 16h38
Emmanuel Delahaye

Citation:

Envoyé par Ricks

PS : saleté de parenthèses :lol:

Bah, non. Saleté de pointeurs imbitables...