a propos de char **

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 Page Suivante Page Précédente Bas de page
Auteur	Sujet : a propos de char **

super-tupapau

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 :

/* DISKCLEAN --[ Trieur/Nettoyeur de dossier personnel ]-- */
#include <stdio.h>
#include <stdlib.h>
#include <dirent.h>
#include <string.h>
/* DISKCLEAN
FONCTION --[ diskc_countfile ]--
Compte le nombre de fichier / dossier
dans un dossier spécifié en
argument.
Retourne le nombre obtenue.
*/
int diskc_countfile(const char *directory)
{
DIR *dir = NULL;
struct dirent *entry = NULL;
int result;
result = 0;
/* Ouverture du répertoire */
dir = opendir(directory);
if (dir == NULL)
return -1;
/* Compter les fichiers / dossiers */
while ((entry = readdir(dir)) != NULL) {
if (strcmp(entry->d_name, "." ) != 0
&& strcmp(entry->d_name, ".." ) != 0) {
result++;
}
}
closedir(dir);
return result;
}
/* DISKCLEAN
FONCTION --[ diskc_readir ]--
Lecteur de dossier récursif.
Retourne un pointeur sur
tableau des fichiers présent
dans le dossier spécifié
en argument.
*/
/* Nombre de fichier analysé */
int nb_file = 0;
char **diskc_readir(const char *directory)
{
DIR *dir = NULL;
struct dirent *entry = NULL;
char **result = NULL;
int i;
/* Ouverture du répertoire */
dir = opendir(directory);
if (dir == NULL)
return NULL;
/* Lecture du dossier */
i = 0;
nb_file = diskc_countfile(directory);
result = malloc(nb_file * sizeof(*result));
while ((entry = readdir(dir)) != NULL) {
if (strcmp(entry->d_name, "." ) != 0
&& strcmp(entry->d_name, ".." ) != 0) {
result[i] = entry->d_name;
i++;
}
}
printf("i: %i\n", i);
printf("nb_file: %i\n", nb_file);
closedir(dir);
return result;
}
/* DISKCLEAN
FONCTION --[ main ]--
Appeler par le systeme
Aucun arguments
*/
int main(void)
{
int i;
char **file = NULL;
file = diskc_readir("/home/pierre/" );
if (file == NULL)
return EXIT_FAILURE;
printf("Analyse du dossier /home/pierre/\n" );
for (i = 0; i < nb_file; i++) {
printf("%s\n", *(file + i));
}
if (file != NULL)
free(file);
return EXIT_SUCCESS;
}

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 :

char **file = NULL;
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.

Publicité

Taz

bisounours-codeur

result[i] = entry->d_name;

entry->d_name ne t'appartient pas. tu dois le copier

et pour éviter un double parcours, man realloc

chrisbk

Code :

*(file + i)

peine de mort

Message cité 1 fois

super-tupapau

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?

Taz

bisounours-codeur

entre autres pour le strcpy

super-tupapau

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

chrisbk

strdup

Taz

bisounours-codeur

pas standard.

Message cité 1 fois

Taz

bisounours-codeur

super-tupapau a écrit :

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

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

super-tupapau

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

Publicité

super-tupapau

voila avec un realloc:

Code :

/* DISKCLEAN --[ Trieur/Nettoyeur de dossier personnel ]-- */
#include <stdio.h>
#include <stdlib.h>
#include <dirent.h>
#include <string.h>
/* DISKCLEAN
FONCTION --[ diskc_readir ]--
Lecteur de dossier récursif.
Retourne un pointeur sur
tableau des fichiers présent
dans le dossier spécifié
en argument.
*/
/* Nombre de fichier analysé */
int nb_file = 1;
char **diskc_readir(const char *directory)
{
DIR *dir = NULL;
struct dirent *entry = NULL;
char **result = NULL;
int i;
/* Ouverture du répertoire */
dir = opendir(directory);
if (dir == NULL)
return NULL;
/* Lecture du dossier */
i = 0;
while ((entry = readdir(dir)) != NULL) {
if (strcmp(entry->d_name, "." ) != 0
&& strcmp(entry->d_name, ".." ) != 0) {
result = realloc( result, nb_file * sizeof(*result));
result[i] = strdup(entry->d_name);
nb_file++;
i++;
}
}
printf("i: %i\n", i);
printf("nb_file: %i\n", nb_file);
closedir(dir);
return result;
}
/* DISKCLEAN
FONCTION --[ main ]--
Appeler par le systeme
Aucun arguments
*/
int main(void)
{
int i;
char **file = NULL;
file = diskc_readir("/home/pierre/" );
if (file == NULL)
return EXIT_FAILURE;
printf("Analyse du dossier /home/pierre/\n" );
for (i = 0; i < nb_file-1; i++) {
printf("%s\n", file[i]);
}
if (file != NULL)
free(file);
return EXIT_SUCCESS;
}

Taz

bisounours-codeur

voir 1er message.

super-tupapau

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

Taz

bisounours-codeur

man 3 readdir

super-tupapau

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.

Taz

bisounours-codeur

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

matafan

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

jesus_christ

votre nouveau dieu

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

Elmoricq

Modérateur

jesus_christ a écrit :

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

jesus_christ

votre nouveau dieu

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:

Elmoricq

Modérateur

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]

lkolrn

chrisbk a écrit :

Code :

*(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

super-tupapau

je voudrais savoir si il etait possible de faire ceci:

Code :

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

super-tupapau

[:undertaker666]

Taz

bisounours-codeur

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

super-tupapau

ok ms je n'y arrive pas:

Code :

struct {
char *name;
char *chemin;
struct arch *next;
struct arch *subdir;
} arch;
typedef struct arch * diskc_arch;
...
diskc_arch **list = NULL;
...
list = realloc( list, nb_file*sizeof(*list));
...
list[i]->name = malloc( sizeof( entry->d_name ) );
list[i]->name = strdup( entry->d_name );
...

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

leneuf22

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

super-tupapau

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?

leneuf22

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

super-tupapau

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

super-tupapau

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

Message cité 1 fois

super-tupapau

personne ne sait comment faire?

Emmanuel Delahaye

C is a sharp tool

super-tupapau a écrit :

je voudrais savoir si il etait possible de faire ceci:

Code :

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/

Emmanuel Delahaye

C is a sharp tool

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/

super-tupapau

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

Code :

diskc_arch * diskc_readir(const char *directory)
{
DIR *dir = NULL;
struct dirent *entry = NULL;
diskc_arch **list = NULL;
int i=0;
/* Ouverture du répertoire */
dir = opendir(directory);
if (dir == NULL)
return NULL;
/* Lecture du dossier */
while ((entry = readdir(dir)) != NULL) {
if (strcmp(entry->d_name, "." ) != 0
&& strcmp(entry->d_name, ".." ) != 0) {
list = realloc( list, sizeof(*list) * nb_file);
/* Compléter les informations du fichiers: nom, chemin, pointeur suivant, pointeur précédent */
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 );
list[i]->next = list[i+1];
list[i]->subdir = NULL;
nb_file++;
i++;
}
}
closedir(dir);
return list[i];
}

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

Emmanuel Delahaye

C is a sharp tool

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/

super-tupapau

ok voila tous le programme:

Code :

/* DISKCLEAN --[ Trieur/Nettoyeur de dossier personnel ]-- */
#include <stdio.h>
#include <stdlib.h>
#include <dirent.h>
#include <string.h>
/* DISKCLEAN
STRUCTURE --[ diskc_arch ]--
Structure chainée
*/
struct arch {
char *name;
char *chemin;
struct arch *next;
struct arch *subdir;
};
typedef struct arch * diskc_arch;
/* DISKCLEAN
FONCTION --[ diskc_readir ]--
Lecteur de dossier récursif.
Retourne un pointeur sur
tableau des fichiers présent
dans le dossier spécifié
en argument.
*/
/* Nombre de fichier analysé */
int nb_file = 1;
diskc_arch * diskc_readir(const char *directory)
{
DIR *dir = NULL;
struct dirent *entry = NULL;
diskc_arch **list = NULL;
int i=0;
/* Ouverture du répertoire */
dir = opendir(directory);
if (dir == NULL)
return NULL;
/* Lecture du dossier */
while ((entry = readdir(dir)) != NULL) {
if (strcmp(entry->d_name, "." ) != 0
&& strcmp(entry->d_name, ".." ) != 0) {
list = realloc( list, sizeof(*list) * nb_file);
/* Compléter les informations du fichiers: nom, chemin, pointeur suivant, pointeur précédent */
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 );
list[i]->next = list[i+1];
list[i]->subdir = NULL;
nb_file++;
i++;
}
}
closedir(dir);
return list[i];
}
/* DISKCLEAN
FONCTION --[ main ]--
Appeler par le systeme
Aucun arguments
*/
int main( int argc, char ** argv )
{
int i;
diskc_arch *file = NULL;
if(argv[1] == NULL)
return EXIT_FAILURE;
file = diskc_readir(argv[1]);
if (file == NULL)
return EXIT_FAILURE;
if (file != NULL)
free(file);
return EXIT_SUCCESS;
}

Message cité 1 fois

Emmanuel Delahaye

C is a sharp tool

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/

super-tupapau

j'ai compris!

quand je même ceci:

Code :

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

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

Code :

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 :

if (file != NULL)
for( i = 0; i<nb_file-2; i++ )
free(file->next+i);
free(file);

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

Emmanuel Delahaye

C is a sharp tool

super-tupapau a écrit :

j'ai compris!

quand je même ceci:

Code :

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

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

Code :

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 :

if (file != NULL)
for( i = 0; i<nb_file-1; i++ )
free(file->next+i);
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/

Publicité

Page : 1 2

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Programmation

a propos de char **

Sujets relatifs
à propos de cadres.. ou recherche d'une autre solution	aide à propos strstr
Fonction qui modifie un char *	a propos des combobox
[HTML] question toute bete à propos des tableaux	Question a propos de Visual C++ 2005
[batch] longueur d'une chaine de char	Convertir un char en int ?!
PB: Fonction de conversion Unicode->char	error: invalid operands of types 'const char [15]' and 'short ..
Plus de sujets relatifs à : a propos de char **

Page générée en 0.165 secondes