Discussion :

[Iskander81]

Bonjour,

Dans le cadre d'un programme, j'utilise plusieurs fonctions :

La première fonction permet de parcourir un fichier, et de le lire ligne à ligne. Chaque ligne est de la forme 3 caractères pour un mois, un espace, 3 chiffres. ex : "jan 101"

La deuxième fonction traite ces lignes de la façon suivante :
on recupère la ligne, et selon le mois, on ajoute le chiffre de la ligne dans un fichier nommé : mois-datedelancement.txt
La date fait 12 caracteres.
ex d'un fichier: fev-070627180403.txt
mois-AAMMJJHHMinMinSS.txt
Si le fichier n'existe pas ( ce qui est le cas au lancement du programme ), il est crée.

Après utilisation du programme, celui-ci plante. Il arrive à réaliser un appel à la fonction traite_ligne, mais la 2ème ligne n'est pas traitée.
Il s'agit peut être d'un problème de mémoire ?

Je vous remercie d'avance de votre aide,
Isk

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
 
//fonction qui traite un fichier chemin est le chemin complet du fichier et
//date est la date du lancement du programme
 
void traite_fichier(char *chemin,char *date) {
 
    char ligne[256];
    FILE* fic;
 
    fic=malloc(sizeof(FILE));
	fic=fopen(chemin,"r");    
    if ( fic == NULL ) { 
    printf("Erreur de lecture du fichier\n");
    return 0;
    }
 
     //on appelle ici la fonction traite_ligne en boucle jusqu'à ce qu'on ait plus 
     //de ligne
 
    while( fgets(ligne, sizeof ligne ,fic) != NULL )  {
      traite_ligne(ligne,date);
    }
 
    printf("\n");
    printf("%s a ete traite\n",chemin);
    fclose(fic);
    free(fic);
    free(ligne);
}
 
 
 
 
//fonction qui traite les lignes
// char* sub_string(char* entree,int deb, int fin) 
// est une fonction qui ressort les caracteres de entree[deb] à entree[fin]
 
void traite_ligne(char * ligne,char * date) {
	char nomfic[16];
	FILE* fichier;
 
	fichier= malloc(sizeof(FILE));
 
	strcpy(nomfic,sub_string(ligne,0,2));
	strcat(nomfic,"-");
	strcat(nomfic,date);
	strcat(nomfic,".txt");
 
    fichier=fopen(nomfic, "a");
	fprintf(fichier,"%s\n",sub_string(ligne,4,6));
	fclose(fichier);
 
	free(nomfic);
	free(fichier);
}

[Iskander81]

Je viens de trouver, en fait l'espace alloué pour le nom du fichier n'etait pas suffisant, je n'avais pas pris en compte l'ajout du .txt

[crocodilex]

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
void traite_fichier(char *chemin,char *date) {

    char ligne[256];
    FILE* fic;
    
    fic=malloc(sizeof(FILE)); Ce malloc ne sert a rien !!!
    fic=fopen(chemin,"r");    
    if ( fic == NULL ) { 
    printf("Erreur de lecture du fichier\n");
    return 0;
    }

    [...]

    fclose(fic);
    free(fic); Ce free peut faire planter
    free(ligne); ligne est un tableau, le free n'est donc pas necessaire 
}

Meme remarques pour la deuxième fonction.

[souviron34]

je dirais même plus

Code :

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

1
2
3
    free(fic); Ce free  plantera (variable (statique ??) gérée par fopen !!!
    free(ligne); ligne est un tableau, le free plantera !!!! 

[Emmanuel Delahaye]

Après utilisation du programme, celui-ci plante. Il arrive à réaliser un appel à la fonction traite_ligne, mais la 2ème ligne n'est pas traitée.
Il s'agit peut être d'un problème de mémoire ?

Tu utilises malloc() / free() à tord et à travers. Chaque usage doit être justifié... Ton code corrigé :

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <assert.h>
 
/* fonction qui traite les lignes */
/* char* sub_string(char* entree,int deb, int fin) */
/* est une fonction qui ressort les caracteres de entree[deb] à entree[fin] */
 
char *sub_string (char *entree, int deb, int fin)
{
   assert (fin >= deb);
   assert ((int)strlen (entree) >= fin);
   entree[fin] = 0;
   return entree + deb;
}
 
void traite_ligne (char *ligne, char const *date)
{
   char nomfic[16];
 
   nomfic[sizeof nomfic - 1] = 0;
 
   strcpy (nomfic, sub_string (ligne, 0, 2));
   assert (nomfic[sizeof nomfic - 1] == 0);
   strcat (nomfic, "-");
   assert (nomfic[sizeof nomfic - 1] == 0);
   strcat (nomfic, date);
   assert (nomfic[sizeof nomfic - 1] == 0);
   strcat (nomfic, ".txt");
   assert (nomfic[sizeof nomfic - 1] == 0);
   {
      FILE *fichier = fopen (nomfic, "a");
 
      if (fichier != NULL)
      {
         printf ("nomfic*'%s'\n", nomfic);
 
         fprintf (fichier, "%s\n", sub_string (ligne, 4, 6));
         fclose (fichier);
      }
      else
      {
         perror (nomfic);
      }
   }
 
}
 
/* fonction qui traite un fichier chemin est le chemin complet du fichier et */
/* date est la date du lancement du programme */
 
int traite_fichier (char const *chemin, char const *date)
{
   int err = 0;
   FILE *fic = fopen (chemin, "r");
   if (fic != NULL)
   {
      /* on appelle ici la fonction traite_ligne en boucle jusqu'à ce qu'on ait plus */
      /* de ligne */
      char ligne[256];
      while (fgets (ligne, sizeof ligne, fic) != NULL)
      {
         traite_ligne (ligne, date);
      }
 
      printf ("\n");
      printf ("%s a ete traite\n", chemin);
      fclose (fic);
   }
   else
   {
      printf ("Erreur de lecture du fichier\n");
      err = 1;
   }
   return err;
}
 
int main (void)
{
   traite_fichier ("test.txt", "01/01/07");
   return 0;
 
}

Pose des questions si tu ne comprends pas.

[Iskander81]

je comprend pas très bien l'usage de la fonction assert...
je crois avoir saisi que c'est une fonction de controle, prenant une expression logique en entrée.
Pour la fonction sub_string corrigée, j'ai du mal à saisir comment le retour fonctionne... on retourne chaine + deb ? ( char * + int ? ) en sachant qu'on ne veut qu'une sous partie de chaine ?

merci pour vos conseils, j'ai réalisé les modifications en otant les malloc et free à outrance.

isk

[Médinoc]

assert() est une macro qui sert à vérifier tout ce qui ne dépend d'aucun facteur extérieur, tout ce qui doit toujours être vrai.

Si une assertion échoue, on sait que c'est le programmeur qui est à blâmer. Et non pas l'OS, ni l'utilisateur.

Donc ici, si tu appelles la fonction sub_string avec fin inférieur à deb, ou avec fin supérieur à la taille de la chaîne, ça geulera (en mode Debug uniquement) et ce sera ta faute.

[Emmanuel Delahaye]

je comprend pas très bien l'usage de la fonction assert...
je crois avoir saisi que c'est une fonction de controle, prenant une expression logique en entrée.

http://emmanuel-delahaye.developpez....tes.htm#assert

Pour la fonction sub_string corrigée, j'ai du mal à saisir comment le retour fonctionne... on retourne chaine + deb ? ( char * + int ? ) en sachant qu'on ne veut qu'une sous partie de chaine ?

Comme tu n'as pas donné le code de cette fonction, j'en ai bricolé une vite fait à partir du prototype supposé... Ca ne prétend pas être une 'correction' de la tienne.

[Iskander81]

D'accord.. Encore merci de vos conseils et de vos réponses ! Je tache de m'en imprégner pour ne plus faire les mêmes erreurs.

Isk