pourquoi 'segmentation fault'?

**thepinguin** · 19/11/2006, 07h51

Bonjour!

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

J'arrive pas à deceler le probleme

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
 
#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;
}

**bahaaldine** · 19/11/2006, 09h37

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

**Jean-Marc.Bourguet** · 19/11/2006, 09h41

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?

**swirtel** · 19/11/2006, 09h44

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

**Emmanuel Delahaye** · 19/11/2006, 09h55

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 :

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

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);
   }

Code :

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

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 :

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

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

Que je ne t'y reprennes pas, vilain garnement...

**swirtel** · 19/11/2006, 09h57

lol "vilain garnement" :d

**thepinguin** · 19/11/2006, 10h14

Merci pour les corrections!! En plus je ne devais pas mettre le '/' devant programmationC

Encore merci..
Maintenant j'ai un autre probleme

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 :

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

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

**Emmanuel Delahaye** · 19/11/2006, 10h21

Envoyé par thepinguin

Maintenant j'ai un autre probleme

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 :

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

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

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

Code :

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

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.

chercher le dernier '.' avec strrchr().
comparer avec strcmp().

et souviens toi de la règle d'or citée au-dessus...

Ecrit l'algorithme avant de coder...

**MatRem** · 19/11/2006, 10h55

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.

(vive le c++)

**thepinguin** · 19/11/2006, 16h08

encore avec mon programme:

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

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

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

le compilateur m'affiche

Code :

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

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

**coca25** · 19/11/2006, 16h20

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

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

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

**zooro** · 19/11/2006, 16h26

Envoyé par thepinguin

le probleme est là:

Code :

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

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 !

**Franck.H** · 19/11/2006, 17h18

Moi ce que je comprend pas c'est ca:

Code :

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

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

**Franck.H** · 19/11/2006, 17h19

Envoyé par zooro

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

D'après le man:

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.

**thepinguin** · 19/11/2006, 17h41

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

**Emmanuel Delahaye** · 19/11/2006, 18h17

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 :

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

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 :

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

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.

**thepinguin** · 20/11/2006, 21h25

Code :

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

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 :

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

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

**thepinguin** · 20/11/2006, 21h27

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

**Emmanuel Delahaye** · 20/11/2006, 21h46

Envoyé par thepinguin

Code :

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

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 :

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

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

PROCEDURE creer_liste_c(nom_repertoire: chaine, extension : chaine)

Je te laisse faire la suite...

**thepinguin** · 20/11/2006, 22h53

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'?

pourquoi 'segmentation fault'?

C

Discussions similaires

Partager

Partager