pourquoi 'segmentation fault'?

Version imprimable

Bonjour!

Alors ce code se compile correctement mais lorsde l'exacution il m'affiche segmentation fault :(

J'arrive pas à deceler le probleme :(
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 #include <stdlib.h> #include <stdio.h> #include <dirent.h> #include <sys/stat.h> int main (void) { struct dirent *lecture; DIR *rep; rep = opendir("/programmationC"); while((lecture=readdir(rep))) { printf("Fichier: %s\n", &lecture->d_name); } closedir(rep); return 1; }

19/11/2006, 09h37
bahaaldine

débugage

Est ce que tu as essayé de débuguer avec ddd, pour voir à quel moment de la boucle il y a une erreur de segmentation ?

tu tapes ddd nom_du_prog

De plus tu as mis /programmationC, est ce que tu est sur que ton répertoire est juste apres la racine "/" ou alors ton intention était d'ouvrir le répertoire programmationC qui est dans le même répertoire que ton programme actuel, dans ce cas là c'est le répertoire courant et tu dois mettre "./programmationC"

PS : en supposant que tu codes sous linux
19/11/2006, 09h41
Jean-Marc.Bourguet

Citation:

Envoyé par thepinguin

Bonjour!

Alors ce code se compile correctement mais lorsde l'exacution il m'affiche segmentation fault :(

J'arrive pas à deceler le probleme :(

Quel est le résultat d'opendir? Ne serait-il pas NULL par hasard?
19/11/2006, 09h44
swirtel

Quel que soit le langage, toujours vérifié les valeurs de retour :(
19/11/2006, 09h55
Emmanuel Delahaye
Citation:

Envoyé par thepinguin

Alors ce code se compile correctement mais lorsde l'exacution il m'affiche segmentation fault :(

Regle d'or. "Toute fonction retournant un pointeur peut retourner NULL" (lire la doc pour confirmation). Il est donc impératif de tester la valeur avant de l'utiliser, sinon, le comportement est indéterminé :
Code:

1 2 3 4 5 6 7 8 9 10 #define REP "/programmationC" DIR *rep = opendir(REP); if (rep != NULL) { /* OK, suite du traitement */ } else { perror (REP); }
Citation:

Code:

while((lecture=readdir(rep)))

OK, c'est très g33k comme écriture, mais pour le moment, on te demande surtout d'écrire du code lisible. Alors :

Code:

while((lecture=readdir(rep)) != NULL)

Que je ne t'y reprennes pas, vilain garnement...
19/11/2006, 09h57
swirtel

lol "vilain garnement" :d
19/11/2006, 10h14
thepinguin

Merci pour les corrections!! En plus je ne devais pas mettre le '/' devant programmationC:oops:
Encore merci..
Maintenant j'ai un autre probleme :aie:
là mon programme m'affiche les fichiers du repertoire..pour qu'il ne m'affiche que les fichiers de type nomfic.c je voudrai inserer la commande

Code:

ls|grep .c

mais je ne sais pas comment.
Mon but final etant de prendre tous les fichier de type nomfic.c et de mettre leurs noms dans une liste chainée:roll:
19/11/2006, 10h21
Emmanuel Delahaye
Citation:

Envoyé par thepinguin

Maintenant j'ai un autre probleme :aie:
là mon programme m'affiche les fichiers du repertoire..pour qu'il ne m'affiche que les fichiers de type nomfic.c je voudrai inserer la commande

Code:

ls|grep .c

mais je ne sais pas comment.
Mon but final etant de prendre tous les fichier de type nomfic.c et de mettre leurs noms dans une liste chainée:roll:

Il ne faut pas tout mélanger. Soit tu écris un script et tu utilises directement la commande qui va bien :

Code:

ls *.c > liste.txt

soit tu écris un programme en C, et dans ce cas, c'est à toi de faire le travail de filtre. Pour celà, il faut donc lister les fichiers, ce que tu as commencé à faire. Pour le filtrage, il suffit de vérifier à chaque fois si l'extension est .c ou non.
1. chercher le dernier '.' avec strrchr().
2. comparer avec strcmp().
et souviens toi de la règle d'or citée au-dessus...

Ecrit l'algorithme avant de coder...
19/11/2006, 10h55
MatRem

Citation:

Quel que soit le langage, toujours vérifié les valeurs de retour

A part si ce langage gère les exceptions et que la fonction en question utilise ce système pour signaler un problème. :mrgreen: (vive le c++)

encore avec mon programme:

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
#include <stdlib.h>
 
#include <stdio.h>
 
#include <dirent.h>
 
#include <sys/stat.h>
 
#include<string.h>
 
#define REP "programmationC"  
//repertoire à traiter
 
//***********************************************************************************************
//prototypes
int comparer(char *ch1, char *ch2);
 
 
//***********************************************************************************************
//fonctio qui compare duex chaines de caracteres, a utiliser
//pour comparer le dernier caractere dans le nom du fichier avec le car 'c'
//afin de reperer les fichier de typer .c executable
int comparer(char *ch1, char *ch2)
{
/*char *ch1;
char *ch2;*/
int n;
int m;
n=(strlen(ch1) - 1);
m=(strlen(ch2) - 1);
strcmp(&ch1[n], &ch2[m]);
if (strcmp(&ch1[n], &ch2[m])==0)
return 1;
else 
return 0;
}
int main (void)
{
char * ch1;
char * ch2;
 
struct dirent *lecture;
DIR *rep = opendir(REP);//j'ouvre mon repertoire
   if (rep != NULL)
   {
 
   }
   else
   { 
      perror (REP);
   }
while((lecture=readdir(rep))!=NULL)
{
ch2='c';
&lecture->d_name=&ch1;
printf("Fichier: %s\n", &lecture->d_name);
if (comparer(ch1, ch2)==1)
scanf("%s", &lecture->d_name);
printf("Fichier: %s\n", &lecture->d_name);
}
 
closedir(rep);
return 1;
}

le probleme est là:

Code:

1
2
3
 
&ch2='c';
&lecture->d_name=&ch1;

le compilateur m'affiche

Code:

1
2
3
4
5
6
 
cc     essai1.c   -o essai1
essai1.c: In function ‘main’:
essai1.c:55: error: invalid lvalue in assignment
essai1.c:56: error: invalid lvalue in assignment
make: *** [essai1] Error 1

c'est certainement un problème basique...mais je bloque depuis tout a l'heure :(
Merci

19/11/2006, 16h20
coca25
&ch2 est une adresse vers un pointeur non initialisé... on peut pas changé une adresse !?
ce que tu veux c'est comparer le dernier caractère du nom de fichier:
Code:

1 2 3 4 5 6 char ch1, ch2; ch1 = 'c'; ch2 = lecture->d_name[strlen (lecture->d_name) - 1]; if (ch1 == ch2) printf ("Fichier: %s\n", lecture->d_name);
j'ai pas compris l'utilité du scanf

dans une boucle rien ne sert de réinitialiser une variable à chaque fois avec la même valeur
19/11/2006, 16h26
zooro
Citation:
Envoyé par thepinguin

le probleme est là:

Code:

1 2 3 &ch2='c'; &lecture->d_name=&ch1;
Que veux-tu faire exactement ?
- ch2 est un char*. Donc &ch2 est l'adresse d'un char*. Or tu lui affectes un char...
- lecture->d_name est sans doute un char*. Donc &lecture->d_name est l'adresse d'un char*. En gros tu prends une adresse (un nombre représentant un emplacement mémoire), et tu lui assignes quelque chose... Ca n'a pas beaucoup de sens !
19/11/2006, 17h18
Franck.H

Moi ce que je comprend pas c'est ca:

Code:

&lecture->d_name=&ch1;

Il ne faut pas toucher à la chaîne contenue dans d_name surtout que ce que tu as fait est faut. Cette chaîne représente le fichier en cours de lecture dans le répertoire ouvert. Je ne vois pas en quoi tu veux modifier cette chaîne !
19/11/2006, 17h19
Franck.H

Citation:

Envoyé par zooro

...lecture->d_name est sans doute un char*.

D'après le man:

Citation:

D'après POSIX, la structure dirent contient un champ char d_name[] de taille non spécifiée, avec au plus NAME_MAX caractères avant le caractère nul final. L'utilisation des autres champs de cette structure compromet la portabilité de votre programme. POSIX-2001 documente aussi le champ ino_t d_ino comme une extension XSI.
Les données renvoyées par readdir() sont écrasées lors de l'appel suivant à readdir() sur le même flux répertoire.
19/11/2006, 17h41
thepinguin

le but de ma manoeuvre est le suivant:
1/ recuperer tout les fichiers d'un repertoire

2/recuperer dans la liste tout les programmes (fichiers) de type nomfichier.c
pour ce faire g fait la fonction comparer qui me compare la derniere lettre du nom du fichier ( ce que je voulais mettre dand le contenu de l'adresse pointée par ch1) et la lettre 'c', (ce que je voulais mattre dans ch2)

3/ la suite serai de mettre ces noms dans une liste chainée ( je suis encore en train de le faire la aussi ça coince:(
19/11/2006, 18h17
Emmanuel Delahaye
Citation:

Envoyé par thepinguin

2/recuperer dans la liste tout les programmes (fichiers) de type nomfichier.c
pour ce faire g fait la fonction comparer qui me compare la derniere lettre du nom du fichier ( ce que je voulais mettre dand le contenu de l'adresse pointée par ch1) et la lettre 'c', (ce que je voulais mattre dans ch2)

Je t'ai pourtant indiqué la marche à suivre :

Pour chaque nom de fichier :

- Chercher le dernier '.' avec strrchr()
- Comparer l'extension avec ".c" à l'aide de strcmp().

Qu'est-ce que tu attends pour le faire ?

Et pour la lecture du répertoire, je t'ai indiqué ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 #define REP "/programmationC" DIR *rep = opendir(REP); if (rep != NULL) { /* OK, suite du traitement */ } else { perror (REP); }
La suite du traitement, à ton avis c'est quoi ?
Parce que dans ton code je vois :
Code:

1 2 3 4 5 6 7 8 9 10 11 DIR *rep = opendir(REP);//j'ouvre mon repertoire if (rep != NULL) { } else { perror (REP); } while((lecture=readdir(rep))!=NULL)
c'est à dire en gros que tu n'as rien compris à mes indications. Ben si tu ne comprends pas ce que je dis, il ne faut pas répéter bêtement, il faut poser des questions.

Et puis je t'ai demandé d'écrire l'algorithme, on peut le voir ?

Ecrire du code à l'arrache, tout le monde peut le faire, et l'expérience montre ça ne marche pas (ou que ça tombe en marche par hasard). Par contre coder selon des règles de conception bien établies, ça demande de la reflexion, et on est quasiment sûr que le code fonctionnera.

Code:

1
2
3
4
5
6
7
8
9
procedure ouvrire/recuperer()
 
debut
 
ouvrire(REP)
recuperer //recuperer les point.c
maille() //mettre chaque nom dans une maille d'une liste chainée
sauvegarde() //mettre le tout dans un fichier de sauvegarde( je ne sais pas si c'est faisable)
fin

Code:

1
2
3
4
5
6
7
8
9
10
11
 
 
procedure parcourliste
debut
parcourir() //parcourire ma liste chainée
choisir fic()//choisir un nom de fichier par ordre
ouvrir() //ouvrir le fichier correspondant
modification()//permet de rajouter un modification a mon fichier
fermer() //fermer le fichier 
enregister() //enregistrer le fichier modifier dans un fichier portant le meme nom, ecraser l'ancien
fin

20/11/2006, 21h27
thepinguin

euh, pour la fonction opendir je l'avais recuperé de la FAQ, et j'ai essayé de la dechiffrer; je debut en C..donc tout ceci est peutere eviden en général mais là j'ai peu du mal à tout saisir:cry:

Citation:
Envoyé par thepinguin
Code:

1 2 3 4 5 6 7 procedure ouvrire/recuperer() debut ouvrire(REP) recuperer //recuperer les point.c maille() //mettre chaque nom dans une maille d'une liste chainée sauvegarde() //mettre le tout dans un fichier de sauvegarde( je ne sais pas si c'est faisable) fin
C'est très général, mais c'est un début.

Il faut maintenant qu'on voit un peu les données circuler, le boucles, les tests... Que ca ressemble à quelque chose de plus concret, sans entrer dans les détails d'implémentation. On appelle ça le pseudo-code ou LDA (Langage de Description d'Algorithme):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 PROCEDURE creer_liste_c() DEBUT repertoire := ouvrir(REP) REPETER fichier := lire_entree(repertoire) ; recuperer les point.c SI extension(fichier) = ".c" ; mettre chaque nom dans une maille d'une liste chainée ajouter(liste, fichier) FIN SI JUSQU'A fin (rep) ; mettre le tout dans un fichier de sauvegarde sauvegarder(liste) FIN
Il est important que les identificateurs aient du sens. Ils risquent fort d'être repris tels quels dans le vrai code (les règles de nommage diffèrent peu d'un langage à l'autre). Autant poser les jalons correctement.

Soigner aussi la présentation. Les commentaires ne doivent pas être en bout de ligne, ça rend le code illisible. Les placer au-dessus du code concerné.

On devrait pouvoir améliorer la fonction en lui passant en paramètres le nom du répertoire et l'extension :

Code:

PROCEDURE creer_liste_c(nom_repertoire: chaine, extension : chaine)

Je te laisse faire la suite...

20/11/2006, 22h53
thepinguin

Bonsoir,
Alors j'ai un probleme avec strrchr...c'est peutêtre banale mais je recupererai un '.' pour comparer avec une chane de taille differente '.c'?

mon probleme est là apparament :(

Code:

1
2
3
4
 
*s1= lecture->d_name[strlen (lecture->d_name) - 1];
*s = strrchr (s1,c);
if( strcmp(s, s1)=0)

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
#include <stdlib.h>
 
#include <stdio.h>
 
#include <dirent.h>
 
#include <sys/stat.h>
 
#include<string.h>
 
#define REP "programmationC"  
 
typedef struct noeud
{
int nomfic;
struct noeud *suiv;
}T_noeud;
 
 
 
 
int main (void)
{
struct dirent *lecture;
 
/*j'ouvre mon repertoire*/
DIR *rep = opendir(REP);
/*si REP non vide*/
   if (rep != NULL)
 
/*je boucle*/
   {
while((lecture=readdir(rep)) != NULL)
     {
/*je recupere un à un les fichier d'extention .c*/
     char *s[50];
     int c;
     char * s1[50];
     *s1= lecture->d_name[strlen (lecture->d_name) - 1];
     *s = strrchr (s1,c);
      if( strcmp(s, s1)=0)
     /*ch2 = lecture->d_name[strlen (lecture->d_name) - 1];
     if(extention(fic) =".c")*/
     printf("Fichier: %s\n", &lecture->d_name);
/*je met un à un les noms des fichiers dans une liste chainée*/
     //ajouter(liste);
     }
 
/*sinon erreur */
   }
   else
   { 
      perror (REP);
   }
/* je ferme le repertoire */
closedir(rep);
return 1;
}

20/11/2006, 23h08
Emmanuel Delahaye
Citation:

Envoyé par thepinguin

Bonsoir,
Alors j'ai un probleme avec strrchr...c'est peutêtre banale mais je recupererai un '.' pour comparer avec une chane de taille differente '.c'?

strrchr() permet de déterminer la position du dernier '.' du nom de fichier. Si il y a un point, l'adresse retournée est celle du point (sinon, c'est NULL, lire la doc). C'est donc le début de la chaine d'extension. Ensuite, tu compares cette chaine avec ".c" et c'est tout. Il suffit de donner cette adresse à strcmp().

C'est sûr que
Code:

1 2 3 4 *s1= lecture->d_name[strlen (lecture->d_name) - 1]; *s = strrchr (s1,c); if( strcmp(s, s1)=0)
ca le fait pas trop là... Il ne faut pas faire n'importe quoi, lire la doc des fonction, comprendre les paramètres, la valeur retournée, le comportement...

Moi, je les connais les docs, et je sais ce qu'il faut faire. Mais je ne peux pas lire la doc à ta place. Je peux juste t'aider à la comprendre si tu as du mal...
21/11/2006, 21h16
thepinguin

Merci pour votre aide!
je devrai lire plus attentivement la doc...mais j'avoue que ce n'est pas evident de saisir le coté pratique.
21/11/2006, 21h18
Emmanuel Delahaye

Citation:

Envoyé par thepinguin

je devrai lire plus attentivement la doc...mais j'avoue que ce n'est pas evident de saisir le coté pratique.

Bienvenu au club des pragmatiques. Expérimente, ça mord pas... Le comportement des fonctions, il faut le mettre en oeuvre, le sentir 'vivre'...
21/11/2006, 22h10
thepinguin

just probleme, là c'est un projet que je dois rendre dans huit jours chrono...et je bloque pour des raisons certainenemt bêtes mais la je ommence à paniquer..est ce que je peux mettre une partie du programme sur ce site?j'avoue que c'est un peu long..mais ça me stresse parce que ça compile toujours faux certainement un probleme de pointeurs entre autre..:(
21/11/2006, 22h23
Emmanuel Delahaye

Citation:

Envoyé par thepinguin

est ce que je peux mettre une partie du programme sur ce site?

Oui, en pièces jointes.