Discussion :

[acik59]

bonjour a tous

alors voila mon programe avec des manipulation de fichier dont j'ai comme erreur : erreur de segmentation

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
 
FILE *tab=NULL;
FILE *rep=NULL;
FILE *file1,*file2,*file3;
char *p = NULL ;
char lecture[100],lit[100];
char *param;
char racine[100];
char fp[100],fp2[100];
 
 
 
 param="/opt/local/proddm/newcat/niveaufin.tab";
tab=fopen(param,"r"); 
fgets(lecture,100,tab);
strcpy ( racine, "/opt/local/proddm/newcat/");
if ( (p = strchr(lecture, '\n')) != NULL )
       *p = '\0' ;
 
 
 strcpy(fp,racine); 
 strcat(fp,lecture);
 
 file1=fopen(fp,"r");     
 strcat(fp,"/cst");
 
 file2=fopen(fp,"r");
 strcpy(fp2,fp);
 
 strcat(fp,"/dxf.cst");
 file3=fopen(fp,"r");    
 
while (!feof(tab))
  {
   if (file1==NULL || file2==NULL) 
      sprintf(lecture,"fond");	
   if (file3!=NULL) 
   {
 
   rep=fopen(fp,"r");
   } 
   else
   {
 
   strcat(fp2,"/couleur.cst");
   rep=fopen(fp2,"r");
   }
fgets(lit,100,rep);
 
}
fclose(file1);
fclose(file2);
fclose(file3);
fclose(rep);
fclose(tab);
}

si quelqu'un a une idee de l'origine du probleme

merci

[Médinoc]

C'est très confus, mais ce que je vois pour commencer, c'est que tu ne testes pas le retour de fopen(), et que tu n'emploies pas feof() correctement.

De plus, param devrait être un const char *.

PS: Et tes strcat() semblent générer au moins un chemin où le / intermédiaire est doublé.

[acik59]

pour ce qui est du / intermediaire j'ai verifier et je ne l'est jamais double

pour ce qui est du fopen je les tester c bon

maintenant comment utiliser correctement la fonction feof()

[crocodilex]

Appeler fopen() plusieurs fois dans une boucle while() n'est pas très malin....

[crocodilex]

maintenant comment utiliser correctement la fonction feof()

Il me semble qu'on en avait déjà parler dans un de tes posts : http://www.developpez.net/forums/sho...d.php?t=333757

Une autre réponse dans la FAQ : ici

[gl]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
 strcpy(fp,racine); 
 strcat(fp,lecture);
 
 file1=fopen(fp,"r");     
 strcat(fp,"/cst");
 
 file2=fopen(fp,"r");
 strcpy(fp2,fp);
 
 strcat(fp,"/dxf.cst");
 file3=fopen(fp,"r");

Mis a part tout ce qui a deja ete signale (particulierement le test manaquant sur les ouvertures de fichiers), il y a un truc qui me chiffone dans ce que je vois au dessus, qu'essaies tu de faire exactement ?
La j'ai l'impression (de la maniere dont sont construit les noms de fichier) que tu cherches a ouvrir des repertoires.

[acik59]

pour le file 1 c'est en effet un test d'ouverture de repertoire!

[Fabllot]

Ce serait un peu plus joli...

http://c.developpez.com/faq/c/?page=...IERS_dir_exist

[gl]

pour le file 1 c'est en effet un test d'ouverture de repertoire!

file2 aussi non ?

Sinon je doute fortement que fopen() sur un repertoire fonctionne bien. Et en plus c'est inutile, si un des repertoires n'existe pas, le fichier n'existera pas non plus (ben oui, ou le mettre) et son ouverture echouera.

En outre, si ne ouverture a echouee, le pointeur vaut NULL et fclose(NULL) est un comportement indetermine (c'est probablement la cause de ton segfault)

[Didier77]

pour les erreur de segmentation utilise gdb c'est mieux