Discussion :

[sylvanus35]

Bonjour à tous,

Je réalise un programme en C qui nécessite de lire et d'écrire dans des fichiers. Une des fonctions me pose un gros problème car elle est, à l'exécution à l'origine d'une erreur de segmentation. Voici cette fonction:

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
 
void RegularExpressionsSearch ()
,{
	FILE *pointeur3;
  	FILE *pointeur5;
 
        pointeur3 = fopen ("b.txt", "r"); 
	pointeur5 = fopen ("Intermediaire2.txt", "w");
 
        char SpecialCharacter1 = 'D';
        char SpecialCharacter2 = 'p';
 
        char *tab1 = (char *)malloc(10000*sizeof(char));
 
	//Tant  que le premier caractère de la ligne est différent de D (cette boucle enregistre dans tab1 toutes les lignes situées avant la phrase commençant par D)
        while ((getc(pointeur3)) != SpecialCharacter1)
	{
		fgets (tab1, 100, pointeur3);
	}
 
        while (fgets(tab1, 100, pointeur3) != NULL)
	{
		fprintf (pointeur5, "%s", tab1);
	}
 
	fclose (pointeur3);
	fclose (pointeur5);
 
	FILE *pointeur6;
  	FILE *pointeur7; 
 
  	pointeur6 = fopen ("Intermediaire2.txt", "r");
        pointeur7 = fopen ("Intermediaire3.txt", "w");
 
 
	while (fgets(tab1, 150, pointeur6) !=NULL)
        {
                if (tab1[0] == SpecialCharacter2)
		{
                            fprintf (pointeur7, "%s", tab1);
 
               }
	}
        fclose (pointeur6);
	fclose (pointeur7);
return;
}

J'ai fait plusieurs tests et je me suis rendu compte que quand je ne fermais pas le pointeur 7 il n'y avait plus d'ereur de segmentation (mais çà ne résoud pas mon problème pour autant car si je ne ferme pas ce pointeur il est alors impossible de réutiliser le fichier vers lequel il pointe, ie Intermediaire3.txt, dans la suite du programme.

De même, si je met la boucle :

Code :

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

1
2
3
4
5
6
7
8
while (fgets(tab1, 150, pointeur6) !=NULL)
        {
                if (tab1[0] == SpecialCharacter2)
		{
                            fprintf (pointeur7, "%s", tab1);
 
               }
	}

en commentaire alors il n'y a plus d'erreur de segmentation.

Voilà, au vu de ce code et de l'exposition de mon problème pouvez-vous me dire d'où vient le souci ??

Je vous remercie par avance pour toute aide (car ça fait maintenant 2 jours que je bloque complètement sur çà).

[Médinoc]

Commence par vérifier les valeurs de retour de tes appels à fopen() et malloc().

free(NULL) est garanti ne rien faire, mais fclose(NULL) est un comportement indéfini...

[sylvanus35]

Ok je vais essayer. Peux-tu me dire comment faire pour vérifier ces valeurs de retours? Merci

[sylvanus35]

J'ai rajouté cette ligne

Code :

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

1
2
if (pointeur7!=NULL)
		printf ("COUCOU");

COUCOU s'affiche bien sur le shell

[sylvanus35]

Je me suis mal expliqué : le test que j'ai fait marche quand je met en commentaire la boucle qui pose problème (àa semble évident).

Mais je n'ai aucun moyen de tester le retour de fopen juste avant de faire le fclose car l'erreur de segmenatation stoppe l'exécution de mon programme

[Emmanuel Delahaye]

Je me suis mal expliqué : le test que j'ai fait marche quand je met en commentaire la boucle qui pose problème (àa semble évident).

Mais je n'ai aucun moyen de tester le retour de fopen juste avant de faire le fclose car l'erreur de segmenatation stoppe l'exécution de mon programme

La forme canonique pour gérer un fichier est ;

Code :

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

1
2
3
4
5
6
7
8
 
FILE *fp = fopen("fichier", "r");
if (fp != NULL)
{
   /* utilisation */
 
   fclose (fp), fp = NULL;
}

Elle garantit un fonctionnement sans erreur. Si il y a 2 fichier, un en lecture, un en écriture, par exemple, il suffit d'imbriquer une copie de cette forme dans celle-ci en renommant les variables correctement.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
FILE *fp_src = fopen("source", "r");
if (fp_src != NULL)
{
   FILE *fp_des = fopen("destination", "w");
   if (fp_des != NULL)
   {
      /* utilisation */
 
      fclose (fp_des), fp_des = NULL;
   }
 
   fclose (fp_src), fp_src = NULL;
}

[spax]

Ta essayé d'utiliser un debugger style Valgrind ou gdb pour localisé vraiment le problème ? ton fichier est bien encodé .

[bulki]

J'ai rajouté cette ligne

Code :

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

1
2
if (pointeur7!=NULL)
		printf ("COUCOU");

COUCOU s'affiche bien sur le shell

Il n'y a pas que pointeur7 à contrôler. Si pointeur3 ou pointeur6 sont nul, la fct getc/fgets va segfaulter.

Hum, essaie ça, tu vas comprendre ce qui va pas:

Code :

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

1
2
3
4
5
6
 
        while ((a = getc(pointeur3)) != SpecialCharacter1) {
	  printf("a: %#x\n", a);
	  fgets(tab1, 2, pointeur3);
	  printf("a: %#x\n", a);
}

On print le code hex, parce que tu peux avoir des surprises. Essaie de prendre un fichier avec des lettres différentes .

+