Discussion :

[Ricks]

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 :

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
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!

[stephl]

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
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.

[Emmanuel Delahaye]

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 :

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

#include <malloc.h>

n'existe pas. C'est

Code :

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

#include <stdlib.h>

Code :

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

      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 :

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
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 :

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
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...

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

[stephl]

Code :

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

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

[Emmanuel Delahaye]

J'ai aussi stdlib.h dans mes includes.

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

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...

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...

[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

[Ricks]

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.

[Emmanuel Delahaye]

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.

[Ricks]

Code :

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

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.

[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

[Emmanuel Delahaye]

PS : saleté de parenthèses

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