a propos de char **

a propos de char ** - C - Programmation

Marsh Posté le 28-10-2005 à 11:42:05    

Rebonjour pour ce qui m'ont déjà aidé (  :ange: ). Maintenant je pense bien maitriser les pointeurs et je fait un petit programme qui est chargé de rechercher les fichiers en doubles dans un groupe dossier ( par exemple le dossier personnel ) malgrès tout un problème persiste sur char ** et char *[].
voici le code erroné:
 

Code :
  1. /* DISKCLEAN --[ Trieur/Nettoyeur de dossier personnel ]-- */
  3. #include <stdio.h>
  4. #include <stdlib.h>
  5. #include <dirent.h>
  6. #include <string.h>
  9. /* DISKCLEAN  
  10.    FONCTION --[ diskc_countfile ]--
  11.    Compte le nombre de fichier / dossier
  12.    dans un dossier spécifié en
  13.    argument.
  14.    Retourne le nombre obtenue.
  15. */
  17. int diskc_countfile(const char *directory)
  18. {
  20.     DIR *dir = NULL;
  21.     struct dirent *entry = NULL;
  22.     int result;
  23.     result = 0;
  25. /* Ouverture du répertoire */
  26.     dir = opendir(directory);
  27.     if (dir == NULL)
  28. return -1;
  30. /* Compter les fichiers / dossiers */
  31.     while ((entry = readdir(dir)) != NULL) {
  33. if (strcmp(entry->d_name, "." ) != 0
  34.     && strcmp(entry->d_name, ".." ) != 0) {
  35.     result++;
  36. }
  37.     }
  39.     closedir(dir);
  41.     return result;
  42. }
  45. /* DISKCLEAN  
  46.    FONCTION --[ diskc_readir ]--
  47.    Lecteur de dossier récursif.
  48.    Retourne un pointeur sur  
  49.    tableau des fichiers présent  
  50.    dans le dossier spécifié
  51.    en argument.
  52. */
  54. /* Nombre de fichier analysé */
  55. int nb_file = 0;
  57. char **diskc_readir(const char *directory)
  58. {
  60.     DIR *dir = NULL;
  61.     struct dirent *entry = NULL;
  62.     char **result = NULL;
  63.     int i;
  65. /* Ouverture du répertoire */
  66.     dir = opendir(directory);
  67.     if (dir == NULL)
  68. return NULL;
  70. /* Lecture du dossier */
  71.     i = 0;
  72.     nb_file = diskc_countfile(directory);
  74.     result = malloc(nb_file * sizeof(*result));
  76.     while ((entry = readdir(dir)) != NULL) {
  78. if (strcmp(entry->d_name, "." ) != 0
  79.     && strcmp(entry->d_name, ".." ) != 0) {
  80.     result[i] = entry->d_name;
  82.     i++;
  83. }
  86.     }
  88.     printf("i: %i\n", i);
  89.     printf("nb_file: %i\n", nb_file);
  91.     closedir(dir);
  93.     return result;
  94. }
  97. /* DISKCLEAN  
  98.    FONCTION --[ main ]--
  99.    Appeler par le systeme
  100.    Aucun arguments
  101. */
  102. int main(void)
  103. {
  105.     int i;
  107.     char **file = NULL;
  108.     file = diskc_readir("/home/pierre/" );
  109.     if (file == NULL)
  110. return EXIT_FAILURE;
  112.     printf("Analyse du dossier /home/pierre/\n" );
  113.     for (i = 0; i < nb_file; i++) {
  114. printf("%s\n", *(file + i));
  115.     }
  117.     if (file != NULL)
  118. free(file);
  120.     return EXIT_SUCCESS;
  121. }


 
Le programme affiche bien les fichiers du dossier /home/pierre ms certain sont coupés ou "collé" à d'autre. Il n'y a aucune erreur de segmentation le problème vient surment au niveau de:

Code :
  1. char **file = NULL;
  2.     file = diskc_readir("/home/pierre/" );


 
puisque quand je met un printf("%s\n", *(result + i)); dans la boucle while de la fonction diskc_readir cela me retourne parfaitement les dossiers et fichiers.
 
Merci bcp de votre aide.

Reply

Marsh Posté le 28-10-2005 à 11:42:05   

Reply

Marsh Posté le 28-10-2005 à 11:55:09    

result[i] = entry->d_name;
 
entry->d_name ne t'appartient pas. tu dois le copier
 
 
 
 
 
 
et pour éviter un double parcours, man realloc

Reply

Marsh Posté le 28-10-2005 à 11:56:36    

Code :
  1. *(file + i)


 
 
peine de mort

Message cité 1 fois
Reply

Marsh Posté le 28-10-2005 à 12:01:19    

realloc j'ai essayé ms j'ai absolument rien compris  :( .
*(file+1) on peut aussi faire file[i] ms ca sait pas le plus important.
 
Enfin pour result[i] = entry->d_name tu veux dire que je doit faire un strcpy?

Reply

Marsh Posté le 28-10-2005 à 12:07:58    

entre autres pour le strcpy

Reply

Marsh Posté le 28-10-2005 à 12:09:57    

Un strcpy cause un segment fault dans mon cas  :non: .
Peux-tu m'explique le realloc?

Message cité 1 fois
Message édité par super-tupapau le 28-10-2005 à 12:11:15
Reply

Marsh Posté le 28-10-2005 à 12:10:32    

strdup

Reply

Marsh Posté le 28-10-2005 à 12:12:46    

pas standard.

Message cité 1 fois
Reply

Marsh Posté le 28-10-2005 à 12:13:55    

super-tupapau a écrit :

Un strcpy cause un segment fault dans mon cas  :non: .


t'es un marrant, copier oui, mais copier où ...

Reply

Marsh Posté le 28-10-2005 à 12:14:41    

Merci cela fonctionne parfaitement mais j'ai pas compris pourquoi je devais dupliquer la chaine?


Message édité par super-tupapau le 28-10-2005 à 12:16:47
Reply

Marsh Posté le 28-10-2005 à 12:14:41   

Reply

Marsh Posté le 28-10-2005 à 12:19:31    

voila avec un realloc:
 

Code :
  1. /* DISKCLEAN --[ Trieur/Nettoyeur de dossier personnel ]-- */
  3. #include <stdio.h>
  4. #include <stdlib.h>
  5. #include <dirent.h>
  6. #include <string.h>
  8. /* DISKCLEAN  
  9.    FONCTION --[ diskc_readir ]--
  10.    Lecteur de dossier récursif.
  11.    Retourne un pointeur sur  
  12.    tableau des fichiers présent  
  13.    dans le dossier spécifié
  14.    en argument.
  15. */
  17. /* Nombre de fichier analysé */
  18. int nb_file = 1;
  20. char **diskc_readir(const char *directory)
  21. {
  23.     DIR *dir = NULL;
  24.     struct dirent *entry = NULL;
  25.     char **result = NULL;
  26.     int i;
  28. /* Ouverture du répertoire */
  29.     dir = opendir(directory);
  30.     if (dir == NULL)
  31. return NULL;
  33. /* Lecture du dossier */
  34.     i = 0;
  36.     while ((entry = readdir(dir)) != NULL) {
  38. if (strcmp(entry->d_name, "." ) != 0
  39.     && strcmp(entry->d_name, ".." ) != 0) {
  40.             result = realloc( result, nb_file * sizeof(*result));
  41.     result[i] = strdup(entry->d_name);
  42.             nb_file++;
  43.     i++;
  44. }
  46.     }
  48.     printf("i: %i\n", i);
  49.     printf("nb_file: %i\n", nb_file);
  51.     closedir(dir);
  53.     return result;
  54. }
  57. /* DISKCLEAN  
  58.    FONCTION --[ main ]--
  59.    Appeler par le systeme
  60.    Aucun arguments
  61. */
  62. int main(void)
  63. {
  65.     int i;
  67.     char **file = NULL;
  68.     file = diskc_readir("/home/pierre/" );
  69.     if (file == NULL)
  70. return EXIT_FAILURE;
  72.     printf("Analyse du dossier /home/pierre/\n" );
  73.     for (i = 0; i < nb_file-1; i++) {
  74. printf("%s\n", file[i]);
  75.     }
  77.     if (file != NULL)
  78. free(file);
  80.     return EXIT_SUCCESS;
  81. }

Reply

Marsh Posté le 28-10-2005 à 12:21:48    

voir 1er message.

Reply

Marsh Posté le 28-10-2005 à 12:28:30    

Il ne m'appartient pas?
c'est à dire? explique stp.

Reply

Marsh Posté le 28-10-2005 à 12:44:11    

man 3 readdir

Reply

Marsh Posté le 28-10-2005 à 13:20:52    

Après avoir lu le man sur readdir j'ai pu voir que c'etait un tableau de caractères qui était retourné et pas un pointeur sur un tableau de caractère.

Reply

Marsh Posté le 28-10-2005 à 13:23:03    

et quand bien même : ton entry, il pointe on ne sait où ... encore une fois, voir la doc.

Reply

Marsh Posté le 28-10-2005 à 20:36:14    

[...]
       The data returned by readdir() may be overwritten by  subsequent  calls
       to readdir() for the same directory stream.
[...]

Reply

Marsh Posté le 28-10-2005 à 21:56:33    

Taz a écrit :

pas standard.

spa standard strdup :ouch:
j'ai vérifié t'as raison :jap:
j'ai appris un truc, pourtant je m'en sers souvent et j'aurais mis ma main à couper que c'était standard.
En plus c'est bien utile comme fonction :( dommage.

Message cité 1 fois
Reply

Marsh Posté le 28-10-2005 à 23:10:48    

jesus_christ a écrit :

spa standard strdup :ouch:
j'ai vérifié t'as raison :jap:
j'ai appris un truc, pourtant je m'en sers souvent et j'aurais mis ma main à couper que c'était standard.
En plus c'est bien utile comme fonction :( dommage.


 
Pas standard ISO, mais c'est quand même POSIX.2
Et vu qu'il utilise dirent.h (POSIX.1), de toute manière, il peut utiliser strdup(). ;)

Reply

Marsh Posté le 28-10-2005 à 23:12:50    

ouais mais bon c'est l'ISO que je considère comme standard, POSIX c'est gentil mais fork() aussi c'est POSIX et ça marche pas terrible sous win ;)
merci pour l'info :jap:

Reply

Marsh Posté le 28-10-2005 à 23:16:58    

Euh oui si on veut vraiment du code portable, c'est ISO.
Je disais juste que vu qu'il s'était déjà engagé dans du POSIX, un strdup() de plus ou de moins ne faisait pas grande différence. [:petrus75]

Reply

Marsh Posté le 29-10-2005 à 08:39:45    

chrisbk a écrit :

Code :
  1. *(file + i)


 
 
peine de mort


juste pour être aussi HS : sémantiquement c'est l'écriture la + juste :jap:
 
edit
 [:aloy]


Message édité par lkolrn le 29-10-2005 à 08:41:52
Reply

Marsh Posté le 30-10-2005 à 15:04:52    

je voudrais savoir si il etait possible de faire ceci:

Code :
  1. struct ma_struct ** ptr = NULL;


et apres de l'allouer avec malloc ptr en sachat que ma_struct et déclararé avant.

Message cité 1 fois
Message édité par super-tupapau le 30-10-2005 à 15:07:00
Reply

Marsh Posté le 30-10-2005 à 17:04:44    

[:undertaker666]

Reply

Marsh Posté le 30-10-2005 à 17:41:37    

oui tu peux. Je reste perplexe sur la fin de ta phrase.

Reply

Marsh Posté le 30-10-2005 à 18:01:15    

ok ms je n'y arrive pas:

Code :
  1. struct {
  3. char *name;
  4. char *chemin;
  5. struct arch *next;
  6. struct arch *subdir;
  8. } arch;
  10. typedef struct arch * diskc_arch;
  12. ...
  14. diskc_arch **list = NULL;
  16. ...
  18. list = realloc( list, nb_file*sizeof(*list));
  20. ...
  22. list[i]->name = malloc( sizeof( entry->d_name ) );
  23. list[i]->name = strdup( entry->d_name );
  25. ...


 
Le compilateur me retourne erreur: request for member ‘name’ in something not a structure or union

Reply

Marsh Posté le 30-10-2005 à 18:50:33    

c'est moi ou la définition de ta struct est foireuse.
De plus selon toi, quel est le type de *list ?

Reply

Marsh Posté le 30-10-2005 à 18:56:06    

Pour le type *list de toute facon c'un pointeur donc c 4 qui sera retourner koi qu'il arrive.
Pour la definition de ma structure explique stp?

Reply

Marsh Posté le 30-10-2005 à 19:07:41    

arch est une variable globale dans ton programme, je doute que c'est ça que tu voulais obtenir !
(pour t'en convaincre, tu peux très bien faire arch.name = NULL; )
 
J'aurais plutôt écrit struct arch {...};
 
Ensuite, ton list aura come type "struct arch ***"
Donc list[i] n'aura pas "arch*" comme type mais bien "arch**"
Et arch** n'a pas le type d'un pointeur sur une structure, tu ne peux donc pas écrire list[i]->name;
 
remarque : il est très rare de se servir de "pointeurs de pointeurs de pointeurs"


Message édité par leneuf22 le 30-10-2005 à 19:08:48
Reply

Marsh Posté le 30-10-2005 à 19:19:01    

ok ca se complique trop je vais faire autrement c'est pas tres pratique comme ca.

Reply

Marsh Posté le 30-10-2005 à 19:52:44    

je trouve pas d'autre solution  :sweat:  en fait je voulais faire un tableau de pointeur sur des structures.

Message cité 1 fois
Reply

Marsh Posté le 30-10-2005 à 20:56:14    

personne ne sait comment faire?

Reply

Marsh Posté le 31-10-2005 à 01:15:05    

super-tupapau a écrit :

je voudrais savoir si il etait possible de faire ceci:

Code :
  1. struct ma_struct ** ptr = NULL;


et apres de l'allouer avec malloc ptr en sachat que ma_struct et déclararé avant.


Oui, et tu peux même utiliser realloc(). Avec NULL c'est pareil que malloc().


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 31-10-2005 à 01:16:39    

super-tupapau a écrit :

je trouve pas d'autre solution  :sweat:  en fait je voulais faire un tableau de pointeur sur des structures.


Comme d'hab :  


T *p = malloc(sizeof *p * n);



---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 31-10-2005 à 12:06:40    

Le problème ne vient pas du malloc voila le code:

Code :
  1. diskc_arch * diskc_readir(const char *directory)
  2. {
  4.     DIR *dir = NULL;
  5.     struct dirent *entry = NULL;
  6.     diskc_arch **list = NULL;
  7.     int i=0;
  9. /* Ouverture du répertoire */
  10.     dir = opendir(directory);
  11.     if (dir == NULL)
  12. return NULL;
  14. /* Lecture du dossier */
  15.     while ((entry = readdir(dir)) != NULL) {
  17. if (strcmp(entry->d_name, "." ) != 0
  18.     && strcmp(entry->d_name, ".." ) != 0) {
  19.             list = realloc( list, sizeof(*list) * nb_file);
  21.     /* Compléter les informations du fichiers: nom, chemin, pointeur suivant, pointeur précédent */
  22.     list[i]->name = malloc( sizeof( entry->d_name ) );
  23.     list[i]->name = strdup( entry->d_name );
  24.     list[i]->chemin = malloc( strlen( directory )+1 );
  25.             list[i]->chemin = strdup( directory );
  26.             list[i]->next = list[i+1];
  27.     list[i]->subdir = NULL;
  29.             nb_file++;
  30. i++;
  31. }
  33.     }
  35.     closedir(dir);
  37.     return list[i];
  38. }


 
ligne de compilation: gcc -Wall -W -O2 -pedantic -o diskclean main.c
 
ca donne:
 
main.c:58: erreur: request for member ‘name’ in something not a structure or union
main.c:59: erreur: request for member ‘name’ in something not a structure or union
main.c:60: erreur: request for member ‘chemin’ in something not a structure or union
main.c:61: erreur: request for member ‘chemin’ in something not a structure or union
main.c:62: erreur: request for member ‘next’ in something not a structure or union
main.c:63: erreur: request for member ‘subdir’ in something not a structure or union

Message cité 1 fois
Reply

Marsh Posté le 31-10-2005 à 12:26:27    

super-tupapau a écrit :

Le problème ne vient pas du malloc voila le code:


Ce code ne compile pas.  

  • Quelle est la définition de 'diskc_arch' ?
  • Où est défini 'nb_file' ?


Il faut poster du code complet, compilable (sauf erreurs), mais réduit au minimum qui montre le défaut.


Message édité par Emmanuel Delahaye le 31-10-2005 à 12:26:52

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 31-10-2005 à 12:40:59    

ok voila tous le programme:

Code :
  1. /* DISKCLEAN --[ Trieur/Nettoyeur de dossier personnel ]-- */
  3. #include <stdio.h>
  4. #include <stdlib.h>
  5. #include <dirent.h>
  6. #include <string.h>
  8. /* DISKCLEAN  
  9.    STRUCTURE --[ diskc_arch ]--
  10.    Structure chainée
  11. */
  13. struct arch {
  15. char *name;
  16. char *chemin;
  17. struct arch *next;
  18. struct arch *subdir;
  20. };
  22. typedef struct arch * diskc_arch;
  25. /* DISKCLEAN  
  26.    FONCTION --[ diskc_readir ]--
  27.    Lecteur de dossier récursif.
  28.    Retourne un pointeur sur  
  29.    tableau des fichiers présent  
  30.    dans le dossier spécifié
  31.    en argument.
  32. */
  34. /* Nombre de fichier analysé */
  35. int nb_file = 1;
  37. diskc_arch * diskc_readir(const char *directory)
  38. {
  40.     DIR *dir = NULL;
  41.     struct dirent *entry = NULL;
  42.     diskc_arch **list = NULL;
  43.     int i=0;
  45. /* Ouverture du répertoire */
  46.     dir = opendir(directory);
  47.     if (dir == NULL)
  48. return NULL;
  50. /* Lecture du dossier */
  51.     while ((entry = readdir(dir)) != NULL) {
  53. if (strcmp(entry->d_name, "." ) != 0
  54.     && strcmp(entry->d_name, ".." ) != 0) {
  55.             list = realloc( list, sizeof(*list) * nb_file);
  57.     /* Compléter les informations du fichiers: nom, chemin, pointeur suivant, pointeur précédent */
  58.     list[i]->name = malloc( sizeof( entry->d_name ) );
  59.     list[i]->name = strdup( entry->d_name );
  60.     list[i]->chemin = malloc( strlen( directory )+1 );
  61.             list[i]->chemin = strdup( directory );
  62.             list[i]->next = list[i+1];
  63.     list[i]->subdir = NULL;
  65.             nb_file++;
  66. i++;
  67. }
  69.     }
  71.     closedir(dir);
  73.     return list[i];
  74. }
  77. /* DISKCLEAN  
  78.    FONCTION --[ main ]--
  79.    Appeler par le systeme
  80.    Aucun arguments
  81. */
  82. int main( int argc, char ** argv )
  83. {
  84.     int i;
  85.     diskc_arch *file = NULL;
  87.     if(argv[1] == NULL)
  88. return EXIT_FAILURE;
  90.     file = diskc_readir(argv[1]);
  91.     if (file == NULL)
  92. return EXIT_FAILURE;
  96.     if (file != NULL)
  97. free(file);
  99.     return EXIT_SUCCESS;
  100. }

Message cité 1 fois
Reply

Marsh Posté le 31-10-2005 à 13:05:27    

super-tupapau a écrit :

ok voila tous le programme:

typedef struct arch * diskc_arch;
    diskc_arch **list = NULL;



Une étoile de trop...

   diskc_arch *list = NULL;


Citation :

    list[i]->name = malloc( sizeof( entry->d_name ) );
     list[i]->name = strdup( entry->d_name );
     list[i]->chemin = malloc( strlen( directory )+1 );
            list[i]->chemin = strdup( directory );



Ben non. Relire la doc de strdup(). Il fait l'allocation tout seul...

    list[i]->name = strdup( entry->d_name );
            list[i]->chemin = strdup( directory );


 
Quand à la libération, un simple free() est bien optimiste!
 
Il faut tout parcourir et libérer tout ce qui a été alloué sans couper la branche sur laquelle on est assis...


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le 31-10-2005 à 13:37:16    

j'ai compris!
 
quand je même ceci:

Code :
  1. typedef struct arch * diskc_arch;
  2. diskc_arch **list = NULL;


 
ca donne un pointeur sur un pointeur sur un pointeur.
j'ai donc supprimer mit

Code :
  1. typedef struct arch diskc_arch;


 
comme ca j'ai plus d'erreur de compilation ms un segment fault avec list[i]->name = strdup( entry->d_name );
 
et pour le free je ferais comme ca?

Code :
  1. if (file != NULL)
  2.     for( i = 0; i<nb_file-2; i++ )
  3. free(file->next+i);
  4.     free(file);

Message cité 1 fois
Message édité par super-tupapau le 31-10-2005 à 13:43:43
Reply

Marsh Posté le 31-10-2005 à 13:45:54    

super-tupapau a écrit :

j'ai compris!
 
quand je même ceci:

Code :
  1. typedef struct arch * diskc_arch;
  2. diskc_arch **list = NULL;


 
ca donne un pointeur sur un pointeur sur un pointeur.
j'ai donc supprimer mit

Code :
  1. typedef struct arch diskc_arch;


comme ca j'ai plus d'erreur de compilation ms un segment fault avec list[i]->name = strdup( entry->d_name );


Ben oui. Tu as alloué le tableau mais pas les éléments...

Citation :


et pour le free je ferais comme ca?

Code :
  1. if (file != NULL)
  2.     for( i = 0; i<nb_file-1; i++ )
  3. free(file->next+i);
  4.     free(file);




Insuffisant. Chaque strdup() doit être compensé par un free(). C'est dans la doc, tu ne l'as donc pas lue malgré mon insistance ? Mais qu'est-ce qu'il faut faire pour se faire entendre ?


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed