menu par notion de fichier

Version imprimable

Voir 40 message(s) de cette discussion en une page

bonjour,

j ai en fait a ecrire une fonction qui permet de trier l ensemble des etudiant,
etudiant est une structure definie par le nom le prenom et le code de l etudiant:
mon assai est le voila :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 void tri() { etudiant p,pm,t; if((f=fopen("etudiant.txt","r"))!=NULL) { fseek(f,0,0); while(fread(&p,sizeof(p),1,f)&&!feof(f)) {pm=p; fseek(f,1,1); while(fread(&pm,sizeof(pm),1,f)&&!feof(f)) if(pm.nom>p.nom) pm=p; t=p; p=pm; pm=t; } while(fread(&p,sizeof(p),1,f)&&!feof(f)) printf("%s\n",p.nom); } fclose(f); }
mais j'ai eu un probleme lors de l'execution !:(
j'aimerais bien savoir s'il ya quelque chose qui manque dans mon code(logiquement parlant)
merci!!

25/01/2008, 16h31
Médinoc

Je dirais de l'ordre.

Tu devrais séparer ton code en trois fonctions: Chargement, Enregistrement et Tri.

De plus, n'utilise pas fread()/fwrite() si c'est du texte...
25/01/2008, 16h32
souviron34

Citation:

Envoyé par s-ehtp

j'aimerais bien savoir s'il ya quelque chose qui manque dans mon code

et de l'indentation et des commentaires...
25/01/2008, 16h34
cmoibal

Citation:

Envoyé par Médinoc

Je dirais de l'ordre.

Tu devrais séparer ton code en trois fonctions: Chargement, Enregistrement et Tri.

De plus, n'utilise pas fread()/fwrite() si c'est du texte...

Mais c'es le standard C, pouquoi pas ?
25/01/2008, 16h40
Médinoc

Parce que ces fonctions sont plus propices à la lecture binaire qu'à la lecture de texte.

ben,j'ai essayé durant 6h a ameliorer le code pour le faire executer mais ca na pas marchr:?,
mais bon je me suis dit je passe a une autre fonction (pour ne pas perdre le temps)qui a pour but de supprimer un enregistrement dans un fichier,je me suis en fait inspiré de la methode expliqué comme suit:je copie mes enregistrements dans un autre fichier laissons seulement l element a supprimer!
le probleme que j'ai trouvé c'est que quand je veux tester ma suppression de l'un des etudiants je fais recours au choix "edition" pour afficher la nouvelle liste des eleves(qui est une autre fonction que j'ai deja etabli) ,mais ce que je obtient c'est l ancienne liste des eleves!:?
bon je donne tout le code dont j'ai pas encore terminé:

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
#include<string.h>
#include<ctype.h>
 
typedef struct
{
	char nom[20];
	char prenom[20];
	int code;
}etudiant;
FILE*f=NULL;
void main()
{   char choix;
    void saisie(), suppression(),edition(),tri();
	//affichage de menu
	do
	{
		puts("\t____________________________");
		puts("\t gestion des etudiants");
		puts("\t____________________________");
		puts("\t\t 1-saisie");
		puts("\t\t 2-suppression");
		puts("\t\t 3-edition");
		puts("\t\t 4-tri");
		puts("\t\t 5-fin");
		printf("                             choix: ");
		do
	    	choix=getchar();
	     while(( choix<'1')||(choix>'5'));
 
	switch(choix)
	{
    	case '1':saisie();break;
    	case '2':suppression();break;
    	case '3':edition();break;
    	case '4':tri();break;
		case '5':printf("au revoir!\n");break;
	}
	}while(choix!='5');
}
////////////////////////////////////////////////////////////////////////////////////////////
 
void saisie()
{  etudiant e;
   char reponse;
 	f=fopen("etudiant.txt","w");
 
	 if(f!=NULL)
	 	do 
	    {
		printf("entrer le nom de votre etudiant\n");
	    scanf("%s",e.nom);
        printf("entrer le prenom de votre etudiant\n");
		scanf("%s",e.prenom);
		printf("entrer le code de votre etudiant\n");
		scanf("%s",&e.code);
		fwrite(&e,sizeof(e),1,f);
		printf("voulez vous continuez?[o/n]");
		 do
	      reponse=getchar();
	       while((toupper(reponse)!='N')&&( toupper(reponse)!='O'));
		puts("\n");
		}
        while(toupper(reponse)!='N');
		fclose(f);
 
 
 
	}
//////////////////////////////////////////////////////////////////////////////////////////////
void suppression()
 
{ 
     FILE *OUTFILE=NULL;
	 OUTFILE=fopen("eleve.txt","w");
	 int i;
	 etudiant e;
char name[20]; 
if(!OUTFILE); 
/* Saisie de l'enregistrement à supprimer */ 
printf("Enregistrement à supprimer : "); 
scanf("%s",name); 
/* Traitement */ 
/* Copie de tous les enregistrements à */ 
/* l'exception de celui à supprimer. */ 
i=0; 
while (fread(&e,sizeof(e),1,f)&&!feof(f)) 
{ 
 
fscanf(f,"%s\n", e.nom); 
 
if (strcmp(e.nom, name) != 0) 
{ 
fprintf(OUTFILE, "%s\n", e.nom); 
fprintf(OUTFILE, "%s\n", e.prenom); 
fprintf(OUTFILE, "%d\n", e.code); 
 
} 
i++; 
} 
/* Fermeture des fichiers */ 
fclose(OUTFILE); 
fclose(f); 
} 	
 
void edition()
{  int i;
   etudiant e;
 
	f=fopen("etudiant.txt","r");
 if(f!=NULL)
 {	 i=1;
	 while(fread(&e,sizeof(e),1,f)&&!feof(f))
	 {   puts("\n");
		 printf("la personne numero %d est:\n",i);
	     printf("le nom  est %s:\n",e.nom);
		 printf("le prenom  est %s:\n",e.prenom);
		 printf("le code  est %d:\n",e.code);
		 i++;
	 }
	 fseek(f,0,2);
	 printf("le fichier contient %d element \n",ftell(f)/sizeof(e));
 
 }
 fclose(f);
}
 
///////////////////////////////////////////////////////////////////////////////////////////
void tri()	
{printf("hi");
}

Tu participes au concours du code le moins lisible de la semaine ?

Fais un effort car c'est pénible de lire du code mal formaté !

Sinon :

aucun contrôle de la taille des saisies "%s" depuis stdin via scanf : c'est à dire que si mon prénom et/mon nom font plus de 20 caractères, ton appli va surement planter !
aucun test de succès de lecture ou d'écriture dans les fichiers
aucun intérêt de " if(!OUTFILE); " dans la fonction suppression()
aucun contrôle de la taille des lectures "%s" depuis les fichiers via fscanf
Appel à suppression() : la variable f n'est pas initialisée avec fopen() dont valeur nulle si saisie() non apellée avant suppression()
fseek() : utilise les constantes SEEK_xxx au lieu de passer des valeurs numériques
déclarer tes fonctions dans le main comme des variables ne te sert à rien ! Relis ton bouquin ou tes cours de C

Bon, c'est juste après un survol rapide du code en diagonal...
Y a du taf !!!

Voici ton code reformaté un minimum :

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
 
#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
#include<string.h>
#include<ctype.h>
 
typedef struct
{
    char nom[20];
    char prenom[20];
    int code;
}etudiant;
 
void saisie();
void suppression();
void edition();
void tri();
 
FILE *f = NULL;
 
void main()
{
    char choix;
 
    //affichage de menu
    do
    {
        puts("\t____________________________");
        puts("\t gestion des etudiants");
        puts("\t____________________________");
        puts("\t\t 1-saisie");
        puts("\t\t 2-suppression");
        puts("\t\t 3-edition");
        puts("\t\t 4-tri");
        puts("\t\t 5-fin");
        printf("                             choix: ");
 
        do
        {
            choix=getchar();
        }
        while(( choix<'1')||(choix>'5'));
 
        switch(choix)
        {
            case '1':
                saisie();
                break;
            case '2':
                suppression();
                break;
            case '3':
                edition();
                break;
            case '4':
                tri();
                break;
            case '5':
                printf("au revoir!\n");
                break;
        }
    }
    while(choix!='5');
}
 
////////////////////////////////////////////////////////////////////////////////
 
void saisie()
{ 
    etudiant e;
    char reponse;
 
    f=fopen("etudiant.txt","w");
 
    if(f!=NULL)
    {
        do 
        {
            printf("entrer le nom de votre etudiant\n");
            scanf("%s",e.nom);
 
            printf("entrer le prenom de votre etudiant\n");
            scanf("%s",e.prenom);
 
            printf("entrer le code de votre etudiant\n");
            scanf("%s",&e.code);
 
            fwrite(&e,sizeof(e),1,f);
 
            printf("voulez vous continuez?[o/n]");
 
            do
            {
                reponse=getchar();
            }
            while((toupper(reponse)!='N')&&( toupper(reponse)!='O'));
 
            puts("\n");
        }
        while(toupper(reponse)!='N');
 
        fclose(f);
    }
 
////////////////////////////////////////////////////////////////////////////////
 
void suppression()
{ 
    FILE *OUTFILE=NULL;
    OUTFILE=fopen("eleve.txt","w");
    int i;
    etudiant e;
    char name[20]; 
 
    if(!OUTFILE); 
 
    /* Saisie de l'enregistrement à supprimer */ 
    printf("Enregistrement à supprimer : "); 
    scanf("%s",name); 
 
    /* Traitement */ 
    /* Copie de tous les enregistrements à */ 
    /* l'exception de celui à supprimer. */ 
 
    i=0; 
    while (fread(&e,sizeof(e),1,f) &&! feof(f)) 
    { 
 
        fscanf(f,"%s\n", e.nom); 
 
        if (strcmp(e.nom, name) != 0) 
        { 
            fprintf(OUTFILE, "%s\n", e.nom); 
            fprintf(OUTFILE, "%s\n", e.prenom); 
            fprintf(OUTFILE, "%d\n", e.code); 
 
        } 
        i++; 
    } 
 
    /* Fermeture des fichiers */ 
    fclose(OUTFILE); 
    fclose(f); 
}     
 
void edition()
{ 
    int i;
    etudiant e;
 
    f=fopen("etudiant.txt","r");
 
    if(f!=NULL)
    {
        i=1;
 
        while(fread(&e,sizeof(e),1,f)&&!feof(f))
        { 
            puts("\n");
            printf("la personne numero %d est:\n",i);
            printf("le nom  est %s:\n",e.nom);
            printf("le prenom  est %s:\n",e.prenom);
            printf("le code  est %d:\n",e.code);
            i++;
        }
 
        fseek(f,0,2);
        printf("le fichier contient %d element \n",ftell(f)/sizeof(e));
    }
 
    fclose(f);
}
 
////////////////////////////////////////////////////////////////////////////////
 
void tri()    
{
    printf("hi");
}

Citation:

Envoyé par s-ehtp

ben,j'ai essayé durant 6h a ameliorer le code pour le faire executer mais ca na pas marchr:?,

Un compilateur bien réglé montre bien des problèmes :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:16: warning: function declaration isn't a prototype
main.c:16: warning: return type of 'main' is not `int'
main.c: In function `main':
main.c:18: warning: function declaration isn't a prototype
main.c:18: warning: function declaration isn't a prototype
main.c:18: warning: function declaration isn't a prototype
main.c:18: warning: function declaration isn't a prototype
main.c: At top level:
main.c:60: warning: function declaration isn't a prototype
main.c: In function `saisie':
main.c:73: warning: char format, different type arg (arg 2)
main.c: At top level:
main.c:88: warning: function declaration isn't a prototype
main.c: In function `suppression':
main.c:94: warning: empty body in an if-statement
main.c: At top level:
main.c:122: warning: function declaration isn't a prototype
main.c: In function `edition':
main.c:140: warning: int format, long unsigned int arg (arg 2)
main.c: At top level:
main.c:148: warning: function declaration isn't a prototype
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 13 warnings

Tu ne devrais pas utiliser de variables globales. Ça rend le code incompréhensible.

Dans suppression(), il y a f et OUTFILE. C'est normal ?

Et ceci

Code:

if (!OUTFILE);

n'a aucun sens. Il faut retirer ce ';'.

Citation:

Envoyé par Emmanuel Delahaye

Un compilateur bien réglé montre bien des problèmes :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:16: warning: function declaration isn't a prototype
main.c:16: warning: return type of 'main' is not `int'
main.c: In function `main':
main.c:18: warning: function declaration isn't a prototype
main.c:18: warning: function declaration isn't a prototype
main.c:18: warning: function declaration isn't a prototype
main.c:18: warning: function declaration isn't a prototype
main.c: At top level:
main.c:60: warning: function declaration isn't a prototype
main.c: In function `saisie':
main.c:73: warning: char format, different type arg (arg 2)
main.c: At top level:
main.c:88: warning: function declaration isn't a prototype
main.c: In function `suppression':
main.c:94: warning: empty body in an if-statement
main.c: At top level:
main.c:122: warning: function declaration isn't a prototype
main.c: In function `edition':
main.c:140: warning: int format, long unsigned int arg (arg 2)
main.c: At top level:
main.c:148: warning: function declaration isn't a prototype
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 13 warnings

Tu ne devrais pas utiliser de variables globales. Ça rend le code incompréhensible.

Dans suppression(), il y a f et OUTFILE. C'est normal ?

Et ceci

Code:

if (!OUTFILE);

n'a aucun sens. Il faut retirer ce ';'.

en fait je me suis inspiré d'une methode qui dit:on peut bien supprimer un element de fichier tout en creant un nouveau fichier qui va contenir tous les elements de l ancien fichier sauf l 'element a supprimer!!

27/01/2008, 02h59
Emmanuel Delahaye

Citation:

Envoyé par s-ehtp

en fait je me suis inspiré d'une methode qui dit:on peut bien supprimer un element de fichier tout en creant un nouveau fichier qui va contenir tous les elements de l ancien fichier sauf l 'element a supprimer!!
voila le site ou j'ai trouvé cette idée

Cette méthode est bonne mais on a pas besoin de globales. Il faut gérer les 2 fichiers dans la même fonction avec des pointeurs locaux, du fopen() au fclose().
27/01/2008, 16h26
s-ehtp

Citation:

Envoyé par Emmanuel Delahaye

Cette méthode est bonne mais on a pas besoin de globales. Il faut gérer les 2 fichiers dans la même fonction avec des pointeurs locaux, du fopen() au fclose().

vous voulez dire que l instruction FILE*f doit figurer localement non pas globalement?c'est bien ça?
sinon ,j'ai utilisé ce pointeur globalement parce que j'aurai besoin de ce pointeur dans toutes les fonctions que je vais faire(pas donc la peine de definir a chaque fois ce pointeur) est ce n'est pas correcte??
merci;)
27/01/2008, 17h17
diogene

Citation:

Envoyé par s-ehtp

vous voulez dire que l instruction FILE*f doit figurer localement non pas globalement?c'est bien ça?
sinon ,j'ai utilisé ce pointeur globalement parce que j'aurai besoin de ce pointeur dans toutes les fonctions que je vais faire(pas donc la peine de definir a chaque fois ce pointeur) est ce n'est pas correcte??
merci;)

C'est un très mauvaise raison.
Les variables globales rendent le code diffile à lire, puisque chaque fonction dépend alors de quelque chose qui lui est extérieur. De plus, lorsque le code est un peu copieux, on ne sait plus qui modifie ni qui utilise ces variables : il faut chercher partout dans le code ce qui transforme le dépannage ou la maintenance du code en un cauchemar.

Règle à adopter au plus tôt : Eviter les variables globales .
27/01/2008, 17h53
s-ehtp

oui ça sera une regle a retenir....
sinon,mon vrai probleme ici c'est que si je supprime mon element tout en creant le nouveau fichier dans lequel je fais transferer mes elements sauf celui à supprimer, j'arrive pas a visualiser ma suppression tout en choisissant le choix edition ,parce que tout simplement ma fonction edition fait apparaitre mon ancien fichier!!!
et moi je dois visualiser le nouveau fichier! et je sais pas comment faire?:cry:
27/01/2008, 19h13
Emmanuel Delahaye

Citation:

Envoyé par s-ehtp

vous voulez dire que l instruction FILE*f doit figurer localement non pas globalement?

Tout à fait. Un bon principe de codage (et de conception) est de limiter la portée des objets et deds fonctions au strict minimum.

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

Citation:

sinon ,j'ai utilisé ce pointeur globalement parce que j'aurai besoin de ce pointeur dans toutes les fonctions que je vais faire(pas donc la peine de definir a chaque fois ce pointeur) est ce n'est pas correcte??

C'est techniquement correct, mais c'est déplorable sur le plan de la qualité du codage. En effet, comment interpréter l'état de f ?

Ouvert ? comment ? fermé ?
27/01/2008, 19h50
s-ehtp

oui ça sera une regle a retenir....
sinon,mon vrai probleme ici c'est que si je supprime mon element tout en creant le nouveau fichier dans lequel je fais transferer mes elements sauf celui à supprimer, j'arrive pas a visualiser ma suppression tout en choisissant le choix edition ,parce que tout simplement ma fonction edition fait apparaitre mon ancien fichier!!!
et moi je dois visualiser le nouveau fichier! et je sais pas comment faire?
27/01/2008, 23h52
s-ehtp

ah une idée ma tombé en tete ,je crois que je dois utiliser remove("ancien fichier") puis je fais rename("nouveau fichier","ancien fichier")
je peux les utiliser je crois...:roll:
28/01/2008, 09h52
Emmanuel Delahaye

Citation:

Envoyé par s-ehtp

ah une idée ma tombé en tete ,je crois que je dois utiliser remove("ancien fichier") puis je fais rename("nouveau fichier","ancien fichier")
je peux les utiliser je crois...:roll:

Oui.

j'ai en fait eu un probleme que j'ai essayé de regler mais en vain:quand je choisis dans mon menu ,la saisie ,l'edition j'arrive toujours à revenir à mon menu principale,mais dans la suppression j'arrive toujours pas a revenir à mon menu!

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
#include<stdio.h>
#include<stdlib.h>
#include<conio.h>
#include<string.h>
#include<ctype.h>
 
typedef struct
{
	char nom[20];
	char prenom[20];
	int code;
}etudiant;
 
void main()
{   char choix;
    void saisie(), suppression(),edition(),tri();
	//affichage de menu
	do
	{
		puts("\t____________________________");
		puts("\t gestion des etudiants");
		puts("\t____________________________");
		puts("\t\t 1-saisie");
		puts("\t\t 2-suppression");
		puts("\t\t 3-edition");
		puts("\t\t 4-tri");
		puts("\t\t 5-fin");
		printf("                             choix: ");
		do
	    	choix=getchar();
	     while(( choix<'1')||(choix>'5'));
 
	switch(choix)
	{
    	case '1':saisie();break;
    	case '2':suppression();break;
    	case '3':edition();break;
    	case '4':tri();break;
		case '5':printf("au revoir!\n");break;
	}
	}while(choix!='5');
}
////////////////////////////////////////////////////////////////////////////////////////////
 
void saisie()
{  etudiant e;
   char reponse;
   FILE*f;
 	f=fopen("etudiant.txt","a+");
 
	 if(f!=NULL)
	 	do 
	    {
		printf("entrer le nom de votre etudiant\n");
	    scanf("%s",e.nom);
        printf("entrer le prenom de votre etudiant\n");
		scanf("%s",e.prenom);
		printf("entrer le code de votre etudiant\n");
		scanf("%s",&e.code);
		fwrite(&e,sizeof(e),1,f);
		printf("voulez vous continuez?[o/n]");
		 do
	      reponse=getchar();
	       while((toupper(reponse)!='N')&&( toupper(reponse)!='O'));
		puts("\n");
		}
        while(toupper(reponse)!='N');
		fclose(f);
 
 
 
	}
//////////////////////////////////////////////////////////////////////////////////////////////
void suppression()
 
{   
 
  char ANCIEN[30], NOUVEAU[30]; 
  FILE *INFILE, *OUTFILE;
 
  char  NOM_SUPPR[30];
  etudiant e;
  /* Ouverture de l'ancien fichier en lecture */
  do{
 
     printf("Nom de l'ancien fichier : ");
     scanf("%s", ANCIEN);
     INFILE = fopen(ANCIEN, "r"); 
     if (!INFILE) 
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", ANCIEN);
    }
  while (!INFILE);
  /* Ouverture du nouveau fichier en écriture */
  do
    {
     printf("Nom du nouveau fichier : ");
     scanf("%s", NOUVEAU);
     OUTFILE = fopen(NOUVEAU, "w"); 
     if (!OUTFILE) 
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", NOUVEAU);
    }
  while (!OUTFILE);
 
 
 
  printf("Enregistrement a supprimer : ");
  scanf("%s",NOM_SUPPR);
 
 
 
 
  while (!feof(INFILE))
    {
     fscanf(INFILE, "%s\n", e.nom);
     if (strcmp(e.nom, NOM_SUPPR) != 0)
          fprintf(OUTFILE, "%s\n", e.nom);
    }
 
 
  /* Fermeture des fichiers */
 
  fclose(OUTFILE);
  fclose(INFILE);
  remove("etudiant.txt");
  rename("eleve.txt","etudiant.txt");
 
 
 
} 	
/////////////////////////////////////////////////////////////////////////////////////////////////
 
 
 
 
void edition()
{  int i;
   etudiant e;
   FILE*f;
 
	f=fopen("etudiant.txt","r");
 if(f!=NULL)
 {	 i=1;
	 while(fread(&e,sizeof(e),1,f)&&!feof(f))
	 {   puts("\n");
		 printf("la personne numero %d est:\n",i);
	     printf("le nom  est %s:\n",e.nom);
		 printf("le prenom  est %s:\n",e.prenom);
		 printf("le code  est %d:\n",e.code);
		 i++;
	 }
	 fseek(f,0,2);
	 printf("le fichier contient %d element \n",ftell(f)/sizeof(e));
 
 }
 fclose(f);
}
 
///////////////////////////////////////////////////////////////////////////////////////////
void tri()	
{
	printf("hi");
}

29/01/2008, 14h06
Médinoc

feof() ne s'emploie pas ainsi.
29/01/2008, 14h08
Médinoc
De plus, il y a plusieurs monstruosités dans ton code, ainsi que des problèmes de forme:
1. Utilisation de scanf("%s") sans limite de taille.
2. main() retourne un int. Toujours.
3. Pourquoi "a+" au lieu de simplement "a" ?
4. Les majuscules, c'est pour les constantes. Les noms de variables doivent être en minuscules ou en camelCase.
29/01/2008, 14h13
s-ehtp
Citation:
Envoyé par Médinoc

De plus, il y a plusieurs monstruosités dans ton code, ainsi que des problèmes de forme:

Utilisation de scanf("%s") sans limite de taille.
main() retourne un int. Toujours.
Pourquoi "a+" au lieu de simplement "a" ?
Les majuscules, c'est pour les constantes. Les noms de variables doivent être en minuscules ou en camelCase.
vous pouvez me clarifier de quel main() vous parlez?
merci!
29/01/2008, 14h16
s-ehtp

Citation:

Envoyé par Médinoc

feof() ne s'emploie pas ainsi.

while(!feof(INFILE)),veut dire tant que je me suis pas encore arrivé à la fin de mon fichier?
pourquoi donc c'est pas bien utilisé?:roll:
29/01/2008, 14h26
Médinoc

Il n'y a qu'une seule fonction main() dans un programme, que je sache... :roll:. Eh bien, apprend que cette fonction-là doit retourner un int. Généralement zéro.
Ben non, ça ne veut pas dire ça. La valeur de feof() est indéfinie tant que les fonctions de lecture n'ont pas retourné une erreur.
29/01/2008, 14h30
s-ehtp

Citation:

Envoyé par Médinoc

Il n'y a qu'une seule fonction main() dans un programme, que je sache... :roll:. Eh bien, apprend que cette fonction-là doit retourner un int. Généralement zéro.
Ben non, ça ne veut pas dire ça. La valeur de feof() est indéfinie tant que les fonctions de lecture n'ont pas retourné une erreur.

pour le main() je sais bien que ça doit retourner un entier.je parle en fait des remarques que vous avez fait sur mon code(merci bien!!) et vous avez signalé que le main doit retourner un entier,je voulais savoir de quel main vous parlez dans mon programme?
merci!
29/01/2008, 14h37
s-ehtp

Citation:

Envoyé par Médinoc

Il n'y a qu'une seule fonction main() dans un programme, que je sache... :roll:. Eh bien, apprend que cette fonction-là doit retourner un int. Généralement zéro.
Ben non, ça ne veut pas dire ça. La valeur de feof() est indéfinie tant que les fonctions de lecture n'ont pas retourné une erreur.

D'accord j ai bien saisi ça!!
29/01/2008, 14h39
Médinoc

Pourquoi me demandes-tu de quel main je parle puisqu'il n'y en a qu'un seul ?
29/01/2008, 15h19
s-ehtp

Citation:

Envoyé par Médinoc

Pourquoi me demandes-tu de quel main je parle puisqu'il n'y en a qu'un seul ?

il ya un seul void main(),pourquoi donc dois-je retourner une valeur?:roll:
29/01/2008, 15h42
corentin59
En C, il doit toujours y avoir une fonction main. Cette fonction doit toujours renvoyer un entier (souvent 0). Le code minimum est donc
Code:

1 2 3 int main(void) { return 0; }
Le void signifie ici que tu n'utilises pas d'argument pour la fonction main. Ton "void main ()" n'a pas de sens.
29/01/2008, 16h02
Emmanuel Delahaye

Citation:

Envoyé par s-ehtp

il ya un seul void main(),pourquoi donc dois-je retourner une valeur?:roll:

http://emmanuel-delahaye.developpez....s.htm#typemain

et pour le problème du feof() :

http://emmanuel-delahaye.developpez....es.htm#fichier

Voici ton programme un peu (beaucoup) remanié:

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
#include <stdio.h>
#include <stdlib.h>
#if 0 /* -tc- inclusion inutile */
#   include<conio.h>
#endif
#include <string.h>
#include <ctype.h>
 
typedef struct
{
    char nom[20];
    char prenom[20];
    int code;
} etudiant;
 
/* -tc- je recommande de declarer les fonctions hors de main() ET sous forme de
   prototypes (une liste vide d'arguments est spécifiee par void) */
void saisie(void);
void suppression(void);
/* -tc- j'ai renomme la fonction edition qui ne faisait que de l'affichage */
void affichage(void);
void tri(void);
/* -tc- quelques fcts pour securiser les saisies*/
int fget_c(FILE *fp);
int fget_s(char *buffer, size_t bufsize, FILE *fp);
int fget_int(int *value, FILE *fp);
/* -tc- recupere les donnees enregistrees par etudient_enreg() */
int etudiant_get(etudiant *self, FILE *fp);
/* -tc- enregistre la description d'un etudiant au format texte dans un 
   fichier */
void etudiant_enreg(etudiant *self, FILE *fp);
 
/* -tc- main() retourne un entier de type int, TOUJOURS! */
int main(void)
{
    int choix;
    int err = 0;
 
    /* affichage de menu */
    do
    {
        printf("____________________________\n"
               "gestion des etudiants\n"
               "____________________________\n"
               "    1-saisie\n"
               "    2-suppression\n"
               "    3-affichage\n"
               "    4-tri\n"
               "    5-fin\n");
 
        do
        {
            printf("choix: ");
            fflush(stdout);
            /* -tc- Saisie pas tres heureuse: laisse un caracterec de fin de
               ligne dans le tampon de stdin. Je prefere convertir en entier.
            choix = getchar();
            */
            err = fget_int(&choix, stdin);
        }
        while (err == 1 || ( choix < 1) || (choix > 5));
 
        switch (choix)
        {
        case 1:
            saisie();
            break;
        case 2:
            suppression();
            break;
        case 3:
            affichage();
            break;
        case 4:
            tri();
            break;
        case 5:
            printf("au revoir!\n");
            break;
        }
    }
    while (choix != 5);
 
    /* -tc- Les 3 valeurs portables que peut retourner main() sont 0,
       EXIT_SUCCESS et EXIT_FAILURE */
    return 0;
}
 
void saisie(void)
{
    etudiant e;
    char reponse;
    FILE *f = NULL;
 
    f = fopen("etudiant.txt", "a");
    if (f != NULL)
    {
        do
        {
            printf("Entrer le nom de votre etudiant: ");
            /* -tc- on force l'affichage de l'invite */
            fflush(stdout);
            /* -tc- pour saisir une chaine proprement */
            fget_s(e.nom, sizeof e.nom, stdin);
            printf("Entrer le prenom de votre etudiant: ");
            fflush(stdout);
            fget_s(e.prenom, sizeof e.prenom, stdin);
            printf("Entrer le code de votre etudiant: ");
            fflush(stdout);
            fget_int(&e.code, stdin);
            etudiant_enreg(&e, f);
            do
            {
                printf("Voulez vous continuez?[o/n] ");
                fflush(stdout);
                /* -tc- il reste au moins un caractere de fin de ligne sur
                   le flux d'entree standard. Il faut faire le menage... 
                reponse = getchar();
                */
                reponse = fget_c(stdin);
            }
            while ((toupper(reponse) != 'N') && ( toupper(reponse) != 'O'));
        }
        while (toupper(reponse) != 'N');
 
        fclose(f), f = NULL;
    }
}
 
void suppression(void)
{
    FILE *infile = NULL;
    FILE *outfile = NULL;
    etudiant  e_suppr = {"", "", 0};
    etudiant e = {"", "", 0};
 
    /* -tc- on n'a pas le choix du nom de fichier dans la fonction saisie().
       Pourquoi le donner dans suppression() ? */
    infile = fopen("etudiant.txt", "r");
    if (infile == NULL)
    {
        fprintf(stderr, "\aERREUR: Impossible d'ouvrir "
                "le fichier etudiant.txt");
    }
    else
    {
 
        outfile = fopen("etudiant-tmp.txt", "w");
        if (!outfile)
        {
            printf("\aERREUR: Impossible d'ouvrir "
                   "le fichier etudiant-tmp.txt");
        }
        else
        {
            /* -tc- On identifie l'etudiant a supprimer avec nom et prenom */
            printf("Nom de l'etudiant a supprimer: ");
            fflush(stdout);
            fget_s(e_suppr.nom, sizeof e_suppr.nom, stdin);
            printf("Prenom de l'etudiant a supprimmer: ");
            fflush(stdout);
            fget_s(e_suppr.prenom, sizeof e_suppr.prenom, stdin);
 
            while (etudiant_get(&e, infile) != EOF)
            {
                /* -tc- on ne conserve que les etudiants dont le nom ET le
                   prenom different de ceux entres par l'utilisateur */
                if (strcmp(e.nom, e_suppr.nom) != 0 || 
                        strcmp(e.prenom, e_suppr.prenom) != 0)
                {
                    etudiant_enreg(&e, outfile);
                }
            }
 
            /* Fermeture des fichiers */
 
            remove("etudiant.txt");
            rename("etudiant-tmp.txt","etudiant.txt");
 
            fclose(outfile);
        }
        fclose(infile);
    }
}
 
void affichage(void)
{
    int i;
    etudiant e= {"", "", 0};
    FILE *f = NULL;
 
    f = fopen("etudiant.txt","r");
    if (f != NULL)
    {
        i = 1;
        while (etudiant_get(&e, f) != EOF)
        {
            printf("La personne numero %d est:\n", i);
            printf("    Le nom  est %s:\n", e.nom);
            printf("    Le prenom  est %s:\n", e.prenom);
            printf("    Le code  est %d:\n", e.code);
            i++;
        }
        printf("Le fichier contient %d element%s\n", i - 1, (i > 2) ? "s": "");
 
    }
    fclose(f);
}
 
void tri(void)
{
    printf("hi\n");
}
 
/* -tc- elimine le caractere de fin de ligne saisi par fgets() et fait le
 * menage sur le flux entrant en cas de saisie tronquee. */
int fclean(char *buffer, FILE *fp)
{
    int err = 0;
 
    if (buffer != NULL && fp != NULL)
    {
        char *pc = strchr(buffer, '\n');
 
        if (pc != NULL)
        {
            *pc = 0;
        }
        else
        {
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {
            }
 
            err = 1;
        }
    }
 
    return err;
}
 
/* -tc- permet de saisir proprement un caractere sur le flux entrant fp */
int fget_c(FILE *fp)
{
    int ret = 0;
 
    if (fp != NULL)
    {
        ret = fgetc(fp);
        if (ret != '\n' && ret != EOF)
        {
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {
            }
        }
    }
 
    return ret;
}
 
/* -tc- permet de saisir proprement une chaine sur le flux d'entree standard */
int fget_s(char *buffer, size_t bufsize, FILE *fp)
{
    int err = 0;
 
    if (buffer != NULL && bufsize > 1 && fp != NULL)
    {
        if (fgets(buffer, bufsize, fp) != NULL)
        {
            err = fclean(buffer, fp);
        }
        else
        {
            if (feof(fp) == 1)
            {
                /* -tc- EOF */
                err = EOF;
            }
        }
    }
    else
    {
        /* -tc- erreur: buffer et fp ne doivent pas etre NULL */
        err = 2;
    }
    return err;
}
 
/* -tc- permet la saisie securisee d'un entier sur le flux d'entree standard */
int fget_int(int *value, FILE *fp)
{
    int err = 0;
 
    if (value != NULL && fp != NULL)
    {
        char buffer[15] = "";
        char *pend = NULL;
 
        if (fget_s(buffer, sizeof buffer, fp) != 1)
        {
            /* -tc- conversion de la chaine en entier */
            *value = strtol(buffer, &pend, 0);
            if (*pend != 0)
            {
                /* -tc- erreur de format */
                err = 1;
            }
        }
    }
    else
    {
        /* -tc- erreur: value et fp ne doivent pas etre NULL */
        err = 2;
    }
    return err;
}
 
/* -tc- permet la saisie d'une entree a partir du fichier contenant la
   liste des etudiants */
int etudiant_get(etudiant *self, FILE *fp)
{
    int err = 0;
 
    if (self != NULL && fp != NULL)
    {
        char buffer[60] = "";
 
        if ((err = fget_s(buffer, sizeof buffer, fp)) == 0)
        {
            int rv = sscanf(buffer, "%19[^,],%19[^,],%d", 
                            self->nom, self->prenom, &self->code);
            if (rv != 3)
            {
                err = 1;
            }
        }
    }
 
    return err;
}
 
/* -tc- permet d'ajouter une entree dans le fichier contenant la liste
   des etudiants */
void etudiant_enreg(etudiant *self, FILE *fp)
{
    if (self != NULL && fp != NULL)
    {
        fprintf(fp, "%s,%s,%d\n", self->nom, self->prenom, self->code);
    }
}

Pose des questions si tu ne comprends pas.

Thierry

bonsoir tout le monde,
bon je vous poste ici seulement ma nouvelle suppression qui m'a permis de revenir au menu principal,mais la ya un autre probleme ,aucune suppression n'a pu etre faite:?:?:,je sais pas pourquoi parce que le tout Me parait logiquement bien fait!
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 void suppression() { FILE*p;\\le pointeur sur le nouveau fichier FILE*f;\\le pointeur sur l'ancien fichier char name[20]; etudiant e; \controle de l'ouverture des 2 fichiers do { f=fopen("etudiant.txt","w"); if(f==NULL) printf("impossible d'ouvrir le fichier"); } while(f==NULL); do { p=fopen("eleve.txt","w"); if(f==NULL) printf("impossible d'ouvrir le fichier"); } while(p==NULL); printf("quel etudiant voulez vous supprimez?"); scanf("%s",name); while(!feof(f)) { fread(&e,sizeof(e),1,f); if(strcmp(e.nom,name)!=0) fwrite(&e,sizeof(e),1,p); } fclose(f); fclose(p); rename("eleve.txt","etudiant.txt"); }
la methode est aussi expliqué dans le site que Monsieur Emmanuel m'a donné
http://emmanuel-delahaye.developpez....es.htm#fichier
(fread est la,le if aussi ,le fwrite....)

Citation:

Envoyé par Thierry Chappuis

Voici ton programme un peu (beaucoup) remanié:

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
#include <stdio.h>
#include <stdlib.h>
#if 0 /* -tc- inclusion inutile */
#   include<conio.h>
#endif
#include <string.h>
#include <ctype.h>
 
typedef struct
{
    char nom[20];
    char prenom[20];
    int code;
} etudiant;
 
/* -tc- je recommande de declarer les fonctions hors de main() ET sous forme de
   prototypes (une liste vide d'arguments est spécifiee par void) */
void saisie(void);
void suppression(void);
/* -tc- j'ai renomme la fonction edition qui ne faisait que de l'affichage */
void affichage(void);
void tri(void);
/* -tc- quelques fcts pour securiser les saisies*/
int fget_c(FILE *fp);
int fget_s(char *buffer, size_t bufsize, FILE *fp);
int fget_int(int *value, FILE *fp);
/* -tc- recupere les donnees enregistrees par etudient_enreg() */
int etudiant_get(etudiant *self, FILE *fp);
/* -tc- enregistre la description d'un etudiant au format texte dans un 
   fichier */
void etudiant_enreg(etudiant *self, FILE *fp);
 
/* -tc- main() retourne un entier de type int, TOUJOURS! */
int main(void)
{
    int choix;
    int err = 0;
 
    /* affichage de menu */
    do
    {
        printf("____________________________\n"
               "gestion des etudiants\n"
               "____________________________\n"
               "    1-saisie\n"
               "    2-suppression\n"
               "    3-affichage\n"
               "    4-tri\n"
               "    5-fin\n");
 
        do
        {
            printf("choix: ");
            fflush(stdout);
            /* -tc- Saisie pas tres heureuse: laisse un caracterec de fin de
               ligne dans le tampon de stdin. Je prefere convertir en entier.
            choix = getchar();
            */
            err = fget_int(&choix, stdin);
        }
        while (err == 1 || ( choix < 1) || (choix > 5));
 
        switch (choix)
        {
        case 1:
            saisie();
            break;
        case 2:
            suppression();
            break;
        case 3:
            affichage();
            break;
        case 4:
            tri();
            break;
        case 5:
            printf("au revoir!\n");
            break;
        }
    }
    while (choix != 5);
 
    /* -tc- Les 3 valeurs portables que peut retourner main() sont 0,
       EXIT_SUCCESS et EXIT_FAILURE */
    return 0;
}
 
void saisie(void)
{
    etudiant e;
    char reponse;
    FILE *f = NULL;
 
    f = fopen("etudiant.txt", "a");
    if (f != NULL)
    {
        do
        {
            printf("Entrer le nom de votre etudiant: ");
            /* -tc- on force l'affichage de l'invite */
            fflush(stdout);
            /* -tc- pour saisir une chaine proprement */
            fget_s(e.nom, sizeof e.nom, stdin);
            printf("Entrer le prenom de votre etudiant: ");
            fflush(stdout);
            fget_s(e.prenom, sizeof e.prenom, stdin);
            printf("Entrer le code de votre etudiant: ");
            fflush(stdout);
            fget_int(&e.code, stdin);
            etudiant_enreg(&e, f);
            do
            {
                printf("Voulez vous continuez?[o/n] ");
                fflush(stdout);
                /* -tc- il reste au moins un caractere de fin de ligne sur
                   le flux d'entree standard. Il faut faire le menage... 
                reponse = getchar();
                */
                reponse = fget_c(stdin);
            }
            while ((toupper(reponse) != 'N') && ( toupper(reponse) != 'O'));
        }
        while (toupper(reponse) != 'N');
 
        fclose(f), f = NULL;
    }
}
 
void suppression(void)
{
    FILE *infile = NULL;
    FILE *outfile = NULL;
    etudiant  e_suppr = {"", "", 0};
    etudiant e = {"", "", 0};
 
    /* -tc- on n'a pas le choix du nom de fichier dans la fonction saisie().
       Pourquoi le donner dans suppression() ? */
    infile = fopen("etudiant.txt", "r");
    if (infile == NULL)
    {
        fprintf(stderr, "\aERREUR: Impossible d'ouvrir "
                "le fichier etudiant.txt");
    }
    else
    {
 
        outfile = fopen("etudiant-tmp.txt", "w");
        if (!outfile)
        {
            printf("\aERREUR: Impossible d'ouvrir "
                   "le fichier etudiant-tmp.txt");
        }
        else
        {
            /* -tc- On identifie l'etudiant a supprimer avec nom et prenom */
            printf("Nom de l'etudiant a supprimer: ");
            fflush(stdout);
            fget_s(e_suppr.nom, sizeof e_suppr.nom, stdin);
            printf("Prenom de l'etudiant a supprimmer: ");
            fflush(stdout);
            fget_s(e_suppr.prenom, sizeof e_suppr.prenom, stdin);
 
            while (etudiant_get(&e, infile) != EOF)
            {
                /* -tc- on ne conserve que les etudiants dont le nom ET le
                   prenom different de ceux entres par l'utilisateur */
                if (strcmp(e.nom, e_suppr.nom) != 0 || 
                        strcmp(e.prenom, e_suppr.prenom) != 0)
                {
                    etudiant_enreg(&e, outfile);
                }
            }
 
            /* Fermeture des fichiers */
 
            remove("etudiant.txt");
            rename("etudiant-tmp.txt","etudiant.txt");
 
            fclose(outfile);
        }
        fclose(infile);
    }
}
 
void affichage(void)
{
    int i;
    etudiant e= {"", "", 0};
    FILE *f = NULL;
 
    f = fopen("etudiant.txt","r");
    if (f != NULL)
    {
        i = 1;
        while (etudiant_get(&e, f) != EOF)
        {
            printf("La personne numero %d est:\n", i);
            printf("    Le nom  est %s:\n", e.nom);
            printf("    Le prenom  est %s:\n", e.prenom);
            printf("    Le code  est %d:\n", e.code);
            i++;
        }
        printf("Le fichier contient %d element%s\n", i - 1, (i > 2) ? "s": "");
 
    }
    fclose(f);
}
 
void tri(void)
{
    printf("hi\n");
}
 
/* -tc- elimine le caractere de fin de ligne saisi par fgets() et fait le
 * menage sur le flux entrant en cas de saisie tronquee. */
int fclean(char *buffer, FILE *fp)
{
    int err = 0;
 
    if (buffer != NULL && fp != NULL)
    {
        char *pc = strchr(buffer, '\n');
 
        if (pc != NULL)
        {
            *pc = 0;
        }
        else
        {
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {
            }
 
            err = 1;
        }
    }
 
    return err;
}
 
/* -tc- permet de saisir proprement un caractere sur le flux entrant fp */
int fget_c(FILE *fp)
{
    int ret = 0;
 
    if (fp != NULL)
    {
        ret = fgetc(fp);
        if (ret != '\n' && ret != EOF)
        {
            int c;
 
            while ((c = fgetc(fp)) != '\n' && c != EOF)
            {
            }
        }
    }
 
    return ret;
}
 
/* -tc- permet de saisir proprement une chaine sur le flux d'entree standard */
int fget_s(char *buffer, size_t bufsize, FILE *fp)
{
    int err = 0;
 
    if (buffer != NULL && bufsize > 1 && fp != NULL)
    {
        if (fgets(buffer, bufsize, fp) != NULL)
        {
            err = fclean(buffer, fp);
        }
        else
        {
            if (feof(fp) == 1)
            {
                /* -tc- EOF */
                err = EOF;
            }
        }
    }
    else
    {
        /* -tc- erreur: buffer et fp ne doivent pas etre NULL */
        err = 2;
    }
    return err;
}
 
/* -tc- permet la saisie securisee d'un entier sur le flux d'entree standard */
int fget_int(int *value, FILE *fp)
{
    int err = 0;
 
    if (value != NULL && fp != NULL)
    {
        char buffer[15] = "";
        char *pend = NULL;
 
        if (fget_s(buffer, sizeof buffer, fp) != 1)
        {
            /* -tc- conversion de la chaine en entier */
            *value = strtol(buffer, &pend, 0);
            if (*pend != 0)
            {
                /* -tc- erreur de format */
                err = 1;
            }
        }
    }
    else
    {
        /* -tc- erreur: value et fp ne doivent pas etre NULL */
        err = 2;
    }
    return err;
}
 
/* -tc- permet la saisie d'une entree a partir du fichier contenant la
   liste des etudiants */
int etudiant_get(etudiant *self, FILE *fp)
{
    int err = 0;
 
    if (self != NULL && fp != NULL)
    {
        char buffer[60] = "";
 
        if ((err = fget_s(buffer, sizeof buffer, fp)) == 0)
        {
            int rv = sscanf(buffer, "%19[^,],%19[^,],%d", 
                            self->nom, self->prenom, &self->code);
            if (rv != 3)
            {
                err = 1;
            }
        }
    }
 
    return err;
}
 
/* -tc- permet d'ajouter une entree dans le fichier contenant la liste
   des etudiants */
void etudiant_enreg(etudiant *self, FILE *fp)
{
    if (self != NULL && fp != NULL)
    {
        fprintf(fp, "%s,%s,%d\n", self->nom, self->prenom, self->code);
    }
}

Pose des questions si tu ne comprends pas.

Thierry

mercii infiniment ,mais vraiment ya des trucs que vous avez utilisez qu'apparement je les ai pas encore appris!
si vous me demandez que je pose des questions je vais poser peut etre 100000000000000000000 questions :lol::lol:
Merci!!;)

30/01/2008, 22h50
s-ehtp

sinon pour le feof ,j'arrive toujours pas à saisir en quoi elle n'est pas bien utilisé dans mon code?:roll:
j'ai lu le http://emmanuel-delahaye.developpez....es.htm#fichier mais j'aarive toujours pas à detecter la faute!

Citation:

Envoyé par s-ehtp

la methode est aussi expliqué dans le site que Monsieur Emmanuel m'a donné
http://emmanuel-delahaye.developpez....es.htm#fichier
(fread est la,le if aussi ,le fwrite....)

Ahem... Dans mon article, il n'est certainement pas dit d'utiliser feof() comme ça :
Citation:
Code:

1 2 3 4 5 6 7 while(!feof(f)) { fread(&e,sizeof(e),1,f); if(strcmp(e.nom,name)!=0) fwrite(&e,sizeof(e),1,p); }
Euh, c'est quoi çà ?
Citation:
Code:

1 2 FILE*p;\\le pointeur sur le nouveau fichier
je rappelle qu'en C, les commentaires, c'est /* ... */

Quand à
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 do { f=fopen("etudiant.txt","w"); if(f==NULL) printf("impossible d'ouvrir le fichier"); } while(f==NULL); do { p=fopen("eleve.txt","w"); if(f==NULL) printf("impossible d'ouvrir le fichier"); } while(p==NULL);
Ca n'a aucun sens. Pourquoi ces boucles ? Pourquoi ouvrir les 2 fichiers en mode "w" (et donc les détruire) ?

Reprenons pas à pas .
1 - On commence par ouvrir les 2 fichiers correctement :
- on donne un nom correct et clair aux pointeurs
- on ouvre l'ancien fichier le lecture et le nouveau en écriture.
- on utilise le mode binaire ("rb", "wb"), car il s'agit visiblement d'un fichier binaire composé d'une séquence structures de type 'etudiant'.
Code:

1 2 3 FILE *fp_old = fopen ("etudiant.txt", "rb"); FILE *fp_new = fopen ("eleve.txt", "wb");
ensuite, on vérifie que tout s'est bien passé, et on en profite pour placer le code pour fermer correctement les fichiers. (le traitement se fera entre les deux, évidemment) :
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 void suppression (void) { FILE *fp_old = fopen ("etudiant.txt", "rb"); if (fp_old != NULL) { FILE *fp_new = fopen ("eleve.txt", "wb"); if (fp_new != NULL) { /* traitement a venir */ fclose (fp_new), fp_new = NULL; } else { printf ("impossible d'ouvrir le nouveau fichier"); } fclose (fp_old), fp_old = NULL; } else { printf ("impossible d'ouvrir l'ancien fichier"); } }
On ajoute une saisie correcte du nom de l'étudiant :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void suppression (void) { <...> if (fp_new != NULL) { char nom[20]; printf ("quel etudiant voulez vous supprimer ? "); fgets (nom, sizeof nom, stdin); fclean (nom, stdin); /* traitement a venir */ <...>
enfin, il faut boucler correctement et faire le filtre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 { etudiant e; while (fread (&e, sizeof e, 1, fp_old) == 1) { if (strcmp (e.nom, name) != 0) { fwrite (&e, sizeof e, 1, fp_new); } else { printf ("etudiant '%s' supprime\n", name); } } }

30/01/2008, 23h44
Emmanuel Delahaye

Citation:

Envoyé par s-ehtp

sinon pour le feof ,j'arrive toujours pas à saisir en quoi elle n'est pas bien utilisé dans mon code?:roll:
j'ai lu le http://emmanuel-delahaye.developpez....es.htm#fichier mais j'aarive toujours pas à detecter la faute!

Qu'estce que tu ne comprends pas dans :

Citation:

Envoyé par -ed-

Détection d'une erreur

La fonction fgets() retourne une valeur de type char *. Si la lecture a réussi, la valeur retournée est l'adresse du tableau de char passé en paramètre. En cas d'échec, la valeur NULL est retournée. Il suffit donc de surveiller cette valeur pour savoir si on peut continuer ou non. Comme une des causes d'échec est la "fin de fichier atteinte", on peut donc parfaitement intégrer ce test dans une boucle de lecture "ligne par ligne".

Une fois l'échec de la lecture constaté, il est possible d'en identifier la cause. Le langage C met à disposition les deux fonctions feof() et ferror() qu'il faut appeler après la boucle de lecture, mais avant la fermeture du fichier.
31/01/2008, 02h00
s-ehtp

Citation:

Ca n'a aucun sens. Pourquoi ces boucles ? Pourquoi ouvrir les 2 fichiers en mode "w" (et donc les détruire) ?

c'est seulement une erreur de frappe!j'ouvre l'un en ecriture l'autre en lecture bien sur!
pour les boucles c'est une sorte d'essai pour ouvrir le fichier.

Code:

1 2 3 4 5 6 while(!feof(f)) { fread(&e,sizeof(e),1,f); if(strcmp(e.nom,name)!=0) fwrite(&e,sizeof(e),1,p); }
s'explique par:tant que je suis pas encore arrivée a la fin de mon fichier
je lis mon enregistrement(par fread()),si je trouve que mon nom a supprimer concide avec le nom de mon enregitrement je le transfert pas au nouveau fichier.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 while (fread (&e, sizeof e, 1, fp_old) == 1) { if (strcmp (e.nom, name) != 0) { fwrite (&e, sizeof e, 1, fp_new); } else { printf ("etudiant '%s' supprime\n", name); } }
ici vous avez utilisez une autre methode:tant que je suis en train de lire mon enregistrement je fais....
donc les 2 boucles auront le meme effet au niveau de l'execution de mon programme ... c'est bien ça non?

Citation:
Envoyé par s-ehtp
Code:

1 2 3 4 5 6 while(!feof(f)) { fread(&e,sizeof(e),1,f); if(strcmp(e.nom,name)!=0) fwrite(&e,sizeof(e),1,p); }
s'explique par:tant que je suis pas encore arrivée a la fin de mon fichier
je lis mon enregistrement(par fread()),si je trouve que mon nom a supprimer concide avec le nom de mon enregitrement je le transfert pas au nouveau fichier.
Décidément, malgré toutes les explications données, tu n'arrives pas à comprendre le fonctionnement de feof(). Je ne sais plus quoi faire. Il est dit que feof() n'a de sens qu'après que la détection de la fin de lecture ait eu lieu. La séquence

feof()
fread()

est donc une absurdité étant donné que c'est fread() qui détecte la fin de lecture. La valeur retournée par feof() avant la fin de lecture n'est pas définie.

D'autre part, feof() est rarement utile, puisqu'il ne sert qu'à identifier la cause de la fin de lecture, qui peut être 'Fin de fichier atteinte' (feof() retourne alors 1) ou 'erreur de lecture' (ferror() retourne 1).
Citation:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 while (fread (&e, sizeof e, 1, fp_old) == 1) { if (strcmp (e.nom, name) != 0) { fwrite (&e, sizeof e, 1, fp_new); } else { printf ("etudiant '%s' supprime\n", name); } }
ici vous avez utilisez une autre methode:tant que je suis en train de lire mon enregistrement je fais....
donc les 2 boucles auront le meme effet au niveau de l'execution de mon programme ... c'est bien ça non?
Je n'ai pas utilisé une 'autre' méthode. J'ai utilisé une méthode correcte qui s'appuie sur le fait que semble ne pas comprendre : c'est la fonction de lecture qui indique la fin de lecture et rien d'autre. Ton code est faux. Que se passe-t-il si il y a une erreur de lecture ? Boucle infinie.

Bon, c'est clair maintenant ? Comment tu te formes au langage C ?

31/01/2008, 16h28
s-ehtp

Citation:

Bon, c'est clair maintenant ? Comment tu te formes au langage C ?

oui oui c'est trés clair maintenant! je vous remercie infiniment Monsieur Emmanuel d'avoir accordé tout ce temps pour mes problemes;)
sinon comment je me forme?:en fait le cours de prof n'est pas toujours complet ,pas toujours d'explication...(on s'est meme pas arretés sur la façon dont le feof fonctionne... et beaucoup d'autre chose)
je me suis apres tombé sur le cours que vous venez de me deconseillez..(merci!!)
et ça me sera trés utile si vous me conseillez un bon cours...:D
31/01/2008, 16h35
s-ehtp

Citation:

Je ne sais plus quoi faire.

vous avez tout fait Monsieur...le probleme reside dans mes capacités trés ,disant trop faible(mais que j'essaie d'ameliorer ....enfin esperons!)

Voir 40 message(s) de cette discussion en une page