Discussion :

[seifdev]

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
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
#include<stdio.h>
#include<conio.h>
#include<string.h>
#include<stdlib.h>
struct liste
{ 
char name[10];
  char prenom[10];
  char bdate[10];
  char code[5];
  char sexe;
  int nbe;
  char enfcode[30];
  struct liste *suiv;
};
 
/* fonction qui enregistre la liste chain‚e dans un fichier sur le disque*/
//d‚but
void enregistre (struct liste *temp,char adr[])
{
   FILE * nv;
   struct liste *par;
  par=temp;
   nv=fopen(adr,"w");
 while(par)
 {
   fprintf(nv,"%s",par->name);
   fprintf(nv,"\n%s",par->prenom);
   fprintf(nv,"\n%s",par->bdate);
   fprintf(nv,"\n%s",par->code);
   fprintf(nv,"\n%c",par->sexe);
   fprintf(nv,"\n%d",par->nbe);
   fprintf(nv,"\n%s",par->enfcode);
   par=par->suiv;
 }
  fclose(nv);
 }
//fin
 
 void affiche (struct liste *tec)
 { while(tec)
  { printf("\nle nom: %s",tec->name);
  tec=tec->suiv;
  }
  }
 //fonction qui ajoute un ‚lement a la liste chain‚e
 //d‚but
 void ajout (struct liste **tec)
 {struct liste *par;
 char aux[50];
  int i,cc;
   char cd[20];
  par=*tec;
  while(par->suiv)
   par=par->suiv;
   par->suiv=(struct liste *)malloc(sizeof(struct liste));
   par=par->suiv;
   par->suiv=NULL;
   clrscr();
   printf("\ndonner le nom de la nouvelle personne :");
   scanf("%s",par->name);
   printf("\ndonner le prenom de la nouvelle personne :");
   scanf("%s",par->prenom);
   printf("\ndonner la date de naissance nouvelle personne :");
   scanf("%s",par->bdate);
   printf("\ndonner le code de la nouvelle personne :");
   scanf("%s",par->code);
   printf("\ndonner le sexe de la nouvelle personne :");
   par->sexe=getch();
   printf("\ndonner le nombre d'enfant de cette personne");
   scanf("%d",&cc);
   par->nbe=cc;
 
      strcpy(cd,"");
     printf("--------%d",cc);
   for(i=0;i<cc;i++)
   {
    printf("\ndonner le code de ces enfants");
    scanf("%s",aux);
       strcat(aux,"\t");
      strcat(cd,aux);
    }
    strcpy(par->enfcode,cd);
 }
// fin
//fonction qui charge un fichier dans une liste chain‚e
void charge(struct liste **ob,char adr[]) /*cette fonction ne fonctionne pas normalement je ne comprend pas pourquoi ??*/
{ struct liste *parc;
  struct liste *de;
  FILE *rd;
  rd=fopen(adr,"r");
  de=(struct liste *)malloc(sizeof(struct liste));
parc=de;
 /* chargement du fichier dans une liste chain‚ */
 /*----------------------------------------------*/
 fscanf(rd,"%s",parc->name);
 
fscanf(rd,"\n%s",parc->prenom);
fscanf(rd,"\n%s",parc->bdate);
fscanf(rd,"\n%s",parc->code);
fscanf(rd,"\n%c",parc->sexe);
fscanf(rd,"\n%d",parc->nbe);
fscanf(rd,"\n%s",parc->enfcode);
while(!feof(rd))
{
parc->suiv=(struct liste *)malloc(sizeof(struct liste));
 fscanf(rd,"%s\n",parc->name);
 
  printf("\n--%s",parc->name);
 getch();
fscanf(rd,"%s",parc->prenom);
fscanf(rd,"\n%s",parc->bdate);
fscanf(rd,"\n%s",parc->code);
fscanf(rd,"\n%c",parc->sexe);
fscanf(rd,"\n%d",parc->nbe);
fscanf(rd,"\n%s",parc->enfcode);
parc=parc->suiv;
}
parc->suiv=NULL;
*ob=de;
}
void main ()
{
 
clrscr();
char cd[50];
char fich[50];
struct liste *doss;
 
struct liste *prov;
struct liste *kl;
struct liste *par;
struct liste *verif;
char aux[50];
prov=NULL;
char cc;
int op,ch;
puts("tapez:\n 1- pour ouvrire un fichier existant \n 2 - pour cr‚e un nouveau fichier");
scanf("%d",&op);
if(op == 1)
{
puts("donner l'adresse du fichier contenant les personnes");
scanf("%s",fich);
printf("---%s",fich);
int nb;
     charge(&kl,fich);
 
 
    affiche(kl);
    getch();
 
 }
 struct liste *deb;
 if(op==2)
 {
 puts("donnez le nombre de personne a ajoutez:");
  int n;
  scanf("%d",&n);
 
 
   deb=(struct liste *)malloc(sizeof(struct liste));
   par=deb;
   par->suiv=NULL;
   clrscr();
   printf("\ndonner le nom de la nouvelle personne :");
   scanf("%s",deb->name);
   printf("\ndonner le prenom de la nouvelle personne :");
   scanf("%s",deb->prenom);
   printf("\ndonner la date de naissance nouvelle personne :");
   scanf("%s",par->bdate);
   printf("\ndonner le code de la nouvelle personne :");
   scanf("%s",par->code);
   printf("\ndonner le sexe de la nouvelle personne :");
   par->sexe=getch();
   printf("\ndonner le nombre d'enfant de cette personne");
   int cc,i;
   scanf("%d",&cc);
   par->nbe=cc;
   for(i=0;i<n-1;i++)
   {    par->suiv=(struct liste *)malloc(sizeof(struct liste));
   par=par->suiv;
 
 clrscr();
   printf("\ndonner le nom de la nouvelle personne :");
   scanf("%s",par->name);
   printf("\ndonner le prenom de la nouvelle personne :");
   scanf("%s",par->prenom);
   printf("\ndonner la date de naissance nouvelle personne :");
   scanf("%s",par->bdate);
   printf("\ndonner le code de la nouvelle personne :");
   scanf("%s",par->code);
   printf("\ndonner le sexe de la nouvelle personne :");
   par->sexe=getch();
   printf("\ndonner le nombre d'enfant de cette personne");
   int cc;
   scanf("%d",&cc);
   par->nbe=cc;
      }
      par->suiv=NULL;
       affiche(deb);
       enregistre(deb,"c:\\perso1.txt");
       kl=deb;
        }
   /*fin de la cr‚ation de la liste chain‚*/
   /*-------------------------------------*/
      prov->suiv=NULL;
    clrscr();
    puts("tapez le code de l'op‚ration a faire");
    puts("1- ajoutez une personne");
    puts("2- supprimer une personne");
    puts("3- mise … jour d'une personne");
    puts("4- le sous menu de consultation de la liste du personnel");
   scanf("%d",&op);
   if(op==1)
   { clrscr();
     verif=(struct liste *)malloc(sizeof(struct liste));
     puts("donner le nom de cette personne");
     scanf("%s",verif->name);
     puts("donner le pr‚nom de cette personne");
     scanf("%s",verif->prenom);
     puts("donner la date de naissance de cette personne");
     scanf("%s",verif->bdate);
     par=kl;
     ch=0;
     while(par!=NULL)
    { if((verif->name==par->name)&&(verif->prenom==par->prenom)&&(verif->bdate==par->bdate))
     ch=1;
     par=par->suiv;
     }
     if(ch)
    {puts("cette personne existe d‚ja");
    }
    else
    {
     puts("cette personne n'existe pas veuillez redonner ces informations");
     ajout(&kl);
     }
      enregistre(kl,"c:\\perso1.txt");
    }
 
 
 
getch();
 }/*parfois méme dans l' enregistrement  ça coince*/

[souviron34]

déjà

[Emmanuel Delahaye]

déjà

ensuite ça :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `ajout':
main.c:58: warning: implicit declaration of function `clrscr'
main.c: In function `charge':
main.c:100: warning: format argument is not a pointer (arg 3)
main.c:101: warning: format argument is not a pointer (arg 3)
main.c:113: warning: format argument is not a pointer (arg 3)
main.c:114: warning: format argument is not a pointer (arg 3)
main.c: At top level:
main.c:122: warning: function declaration isn't a prototype
main.c:122: warning: return type of 'main' is not `int'
main.c: In function `main':
main.c:144: warning: unused variable `nb'
main.c:200: warning: passing arg 2 of `enregistre' discards qualifiers from pointer target type
main.c:237: warning: passing arg 2 of `enregistre' discards qualifiers from pointer target type
main.c:125: warning: unused variable `cd'
main.c:127: warning: unused variable `doss'
main.c:133: warning: unused variable `aux'
main.c:135: warning: unused variable `cc'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 10 seconds)
0 errors, 14 warnings

[diogene]

Quel savant désordre !
Pas d'indentation, des variables déclarées n'importe où sans riason logique, voire ne servant à rien. Exemple(l'indentation ajpoutée):

Code :

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

1
2
3
4
5
6
7
8
9
10
if(op == 1)
{
  puts("donner l'adresse du fichier contenant les personnes");
  scanf("%s",fich);
  printf("---%s",fich);
  int nb; //???????
  charge(&kl,fich);
  affiche(kl);
  getch();
}

Du code répété en plusieurs endroits (considérer les boucles do while!)

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
void charge(struct liste **ob,char adr[]) 
{
 struct liste *parc;
 struct liste *de;
 FILE *rd;
 rd=fopen(adr,"r");                // tester l'ouverture du fichier
 de=(struct liste *)malloc(sizeof(struct liste));
 parc=de;
 fscanf(rd,"%s",parc->name);
 [.....]
 while(!feof(rd))
 {
   parc->suiv=(struct liste *)malloc(sizeof(struct liste));
   fscanf(rd,"%s\n",parc->name);   // on va écraser les valeurs entrées précédemment parc->suiv->name  [.....]
   parc=parc->suiv;
 }
 parc->suiv=NULL;
 *ob=de;
}

Code :

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

1
2
3
4
5
6
7
8
9
void ajout (struct liste **tec)
{
 struct liste *par;
 char aux[50];
 int i,cc;
 char cd[20];
 par=*tec;
 while(par->suiv) par=par->suiv;  // par est NULL au mieux si la liste est vide
....

Faire une fonction de remplissage de ces structures (à partir d'un fichier ou de stdin) sécurisée ( sans scanf("%s"...) , qui teste si tout est bien correct et traite la fin de fichier. Ca allégera déjà le code