GESTION PERSONNEL

GESTION PERSONNEL - C - Programmation

Marsh Posté le 29-05-2018 à 04:55:01    

bonjour ou bonsoir.
jai un mini projet gestion personel en c avec une liste chainé que je doit sauvegardé dans un fichier jai fini les fonction nécéssaire et la liste saffiche normalement mais le problem cest lecriture dans le fichier jy arrive pas merci davance pour votre aide....

 
Code :
  1. #include <stdio.h>
  3. #include <stdlib.h>
  5. #include <string.h>
  7. #include <ctype.h>
  8. typedef struct date {
  9. int j;
  10. int m;
  11. int a;
  12. }date;
  13. typedef struct personne
  14. {
  15. char prenom[20];
  16. char nom[20];
  17. char sexe;
  18. date d;
  19. int nbrenfant;
  20. char identifiant[5];
  21. personne *suiv ;
  22. personne *precd;
  23. }personne;
  24. typedef personne *liste;
  27. FILE *ps;
  28. struct date saisidate(){
  29. struct date d;
  30. do{
  31.  printf("donnez l'annee \n " );
  32.  scanf("%d",&d.a);
  34. }while(d.a<1915||d.a>2018);
  35. do{
  36.  printf(" donnez le mois " );
  37.  scanf("%d",&d.m);
  38. }while(d.m<0||d.m>12);
  39. switch(d.m){
  40.  case 1:
  41.   case 3:
  43.     case 5:
  44.      case 7:
  45.       case 8:
  46.        case 10 :
  47.         case 12:
  48.          do{
  49.           printf(" donner le jours \n" );
  50.           scanf("%d",&d.j);
  51.           }while(d.j<0||d.j>31);
  52.           break;
  53.           case 4:
  54.            case 6:
  55.             case 9:
  56.              case 11:
  57.               do{
  58.                printf("donnez le jour \n " );
  59.                scanf("%d",&d.j);
  60.               }while(d.j<0||d.j>30);
  61.               break;
  64.           case 2:
  65.            if((d.a%4==0)&&(d.a%100==0)){
  66.             do{
  67.                printf("donner le jour:" );
  68.                scanf("%d",&d.j);
  69.               }while(d.j<=0||d.j>29);break;
  71.              }else
  72.              {
  73.               do{
  74.                printf("donner le jour:" );
  75.                scanf("%d",&d.j);
  76.               }while(d.j<=0||d.j>28);break;
  77.              }
  79.            }
  80.            return(d);
  83.            }
  84. personne*alloue(){
  85. personne*pers=(personne*)malloc(sizeof(personne));
  86. pers->suiv=NULL;
  87. return(pers);
  88. }
  89. void ajoutez(liste tete,personne*p)
  90. {
  91. if(tete==NULL)
  92. {
  93.  tete=p;
  94. }
  95. else
  96. {
  97.  p->suiv=tete;
  98.  tete=p;
  99. }
  100. }
  101. liste charge(FILE*ps)
  102. {
  103. liste l=NULL;
  104. personne *p;
  106. ps=fopen("ps.txt","a" );
  107. if (ps==NULL)
  108. {
  109.  printf("le fichier es vide\n" );
  110. }
  111. while(!feof(ps))
  112. {
  113.    p=alloue();
  114.    fscanf(ps,"%s,%s,%c,%d,%d,%,%d,%s",p->prenom,p->nom,p->sexe,&p->d.j,&p->d.m,&p->d.a,p->nbrenfant,p->identifiant);
  115.    ajoutez(l,p);
  116.  
  117. }
  118. fclose(ps);
  119. return l;
  120. }
  121. liste ajouter(liste l)
  122. {
  123. personne *p=alloue();
  124.  printf("donner le prenom:" );
  125.   scanf("%s",p->prenom);
  126.   printf("donner le nom:" );
  127.   scanf("%s",p->nom);
  128.   printf("donner le sexe" );
  129.   scanf("%s",&p->sexe);
  130.   printf("donner le nombre denfant:" );
  131.   scanf("%d",&p->nbrenfant);
  132.   printf("donner lidentifiant:" );
  133.   scanf("%s",p->identifiant);
  134.   printf("\n" );
  135.   p->d=saisidate();
  136.   ajoutez(l,p);
  137. }
  138. void afficher(personne *p){
  139.  printf("\n le nom %s \n prenom %s \n sexe  %c \n date naissance %d %d %d \n nombre d enfant %d \n identifiant%s \n  ",p->nom,p->prenom,p->sexe,p->d.j,p->d.m,p->d.a,p->nbrenfant,p->identifiant);
  140. }
  141. void afficherliste(liste l){
  142.  liste p;
  143.  p=l;
  144.  if(p==NULL){
  145.   printf(" la liste es vide \n " );
  146.  }
  147.  while(p!=NULL){
  148.   afficher(p);
  149.   p=p->suiv;
  150.  }
  151. }
  152. bool recherche(liste l ,personne *precd ,personne *p ,char nom[20]){
  153. precd=NULL;
  154. p=l;
  155. if(p==NULL){
  156.  printf(" la liste vide \n " );
  157. }
  158. while((p!=NULL)&&(strcmp(nom,p->nom)!=0))
  159. {
  160.  precd=p;
  161.  p=p->suiv;
  162. }
  163. if(strcmp(nom,p->nom)==0){return(true);
  164. }else return false;
  165. }
  167. void modifier_nbrdenfant(liste l, char prenom[20]){
  169.     personne*p,*precd;
  170.      if(recherche(l,precd,p,prenom)==true){
  171.       printf(" donnez le nouveax nombre d enfant" );
  172.       scanf("%d",&p->nbrenfant);
  173.  }
  175.  }
  176.  int affichez_nbrenfant(liste l,char prenom[20])
  177.  {
  178.   personne *p,*precd;
  180.   if(recherche(l,precd,p,prenom)==true)
  181.   {
  182.    return(p->nbrenfant);
  183.   }
  184.   else
  185.   {
  186.    printf("cette personne nexist pas" );
  187.   }
  188.  }
  189. liste supprimer(liste l,char prenom[20]){
  190. personne *p,*precd;
  191. if( recherche(l,precd,p,prenom)==true){
  192.  if(precd==NULL){
  193.   l=l->suiv;
  194.   free(p);
  195.   printf("supprimer avec succe\n " );
  196.   return(l);
  197.  }else{
  198.   precd->suiv=p->suiv;
  199.   free(p);
  200.  }
  201. }else{
  202.  printf("cette perssone nexiste pas" );
  203. }
  204. }
  205. char prenom(liste l ,char prenom[20]){
  206.  personne *p,*precd;
  207. if( recherche(l,precd,p,prenom)==true){return(p->nom[20]);}}
  209. int age(liste l,char prenom[20],int n)
  210. {
  211. personne *p,*precd;
  213. if(recherche(l,precd,p,prenom)==true)
  214. {
  215.  return(n-p->d.a);
  216. }
  217. else
  218. {
  219.  printf("cette personne nexist pas" );
  220. }
  221. }
  222. /*void ercirefichier(liste l,FILE*F)
  223. {
  224. personne *p;
  225. FILE *ps=fopen("ps.txt ","w+" );
  227. if(feof(ps)){
  228.  printf("erreur " );}
  229.  p=l;
  230.  while(p!=NULL){
  231.   fprintf(ps,"%s,%s,%c,%s,%d,%d,%d,%d,%s",p->prenom,p->nom,p->sexe,p->d.j,p->d.m,p->d.a,p->nbrenfant,p->identifiant);
  232.   p=p->suiv;
  233.  
  234.  }
  235.  fclose(ps);
  236.  
  237. }*/
  238. void erciredansfichier(personne *p)
  239. {
  240. FILE *ps;
  241. ps=fopen("ps.txt","w" );
  242. if(ps!=NULL)
  243. {
  244.  personne *suiv=NULL;
  245.  personne *precd=NULL;
  246.  while(p!=NULL)
  247.  {
  248.   fseek(ps,0,SEEK_END);
  249.   fwrite(p,sizeof(personne),1,ps);
  251.   printf("writing:%s to file\n",p->nom);
  252.  p=p->suiv;
  254.  }
  256.  suiv=NULL;
  257.  precd=NULL;
  259. }
  260. else
  261. {
  262.  printf("erreur lors de louverture du fichier" );
  263. }
  264. }
  268. void quitter()
  270. {
  272.     exit(0);
  274. }
  276. int main()
  277. {
  279. FILE* ps;
  280. personne p,*precd;
  281. liste l;
  283. int i,n;
  286.  l=ajouter(l);
  288. afficherliste(l);
  289. erciredansfichier(l);
  290. return(0);
  291. }


 
 


Message édité par Harkonnen le 29-05-2018 à 09:21:18
Reply

Marsh Posté le 29-05-2018 à 04:55:01   

Reply

Marsh Posté le 31-05-2018 à 13:45:57    

Je comprends rien à ta fonction saisidate(). En effet, toutes tes boucles do...while {} s'arrêtent avec une condition totalement à coté de la plaque.

 

Exemple :

Code :
  1. do{
  2.  printf(" donnez le mois " );
  3.  scanf("%d",&d.m);
  4. }while(d.m<0||d.m>12);


Je passe sur l'utilisation du scanf() qui est à proscrire. Cette boucle demande à l'utilisateur de saisir un mois, tant que le mois saisi est inférieur à 0 ou supérieur à 12. Qu'est ce que c'est que ce délire d'obliger l'utilisateur à saisir un mois non valide ? Ton while() ne devrait-il pas plutôt ressembler à ceci ?

 
Code :
  1. do {
  2. ...
  3. }
  4. while (d.m >= 1 && d.m <= 12);
 

quand à ton switch pour déterminer l'année bissextile, il est inutile, un "if" suffit largement.

 

essaie de faire aussi un effort sur le nom de tes variables, évite les tailles fixes du style "nom[20]" (si le nom dépasse 20 caractères => BOUM)...

Message cité 1 fois
Message édité par Harkonnen le 31-05-2018 à 13:46:25

---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Marsh Posté le 31-05-2018 à 14:14:33    

Au passage, la construction de la liste implique que chaque nom est unique. Le cas des homonymes n'est donc pas traité. Il faudrait a minima faire une recherche sur l'unicité nom/prénom et pas juste sur le nom.
 
Par ailleurs, la fonction modifier_nbrdenfant() prend en paramètre le prénom et non le nom. Mais c'est le prénom qui est utilisé pour la recherche de la personne. Or, recherche() fait une recherche sur le nom. Tu recherches donc un prénom dans le champ "nom" :/


---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta
Reply

Marsh Posté le 31-05-2018 à 14:40:27    

Et j'avais pas vu ça aussi :

Code :
  1. fseek(ps,0,SEEK_END);


Bravo, tu écris ta structure à partir du dernier caractère du fichier, pas étonnant que ça ne marche pas [:bien]


---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Marsh Posté le 31-05-2018 à 18:03:00    

Harkonnen a écrit :

Je comprends rien à ta fonction saisidate(). En effet, toutes tes boucles do...while {} s'arrêtent avec une condition totalement à coté de la plaque.
 
Exemple :

Code :
  1. do{
  2.  printf(" donnez le mois " );
  3.  scanf("%d",&d.m);
  4. }while(d.m<0||d.m>12);


Je passe sur l'utilisation du scanf() qui est à proscrire. Cette boucle demande à l'utilisateur de saisir un mois, tant que le mois saisi est inférieur à 0 ou supérieur à 12. Qu'est ce que c'est que ce délire d'obliger l'utilisateur à saisir un mois non valide ? Ton while() ne devrait-il pas plutôt ressembler à ceci ?
 

Code :
  1. do {
  2. ...
  3. }
  4. while (d.m >= 1 && d.m <= 12);



 
Euh soit c'est moi, soit c'est toi qui est fatigué mais cela me semble logique que ce soit tant que le mois est inférieur à ou supérieur à. Néanmoins je te l'accorde que sur le signe exacte c'est inférieur ou égale :

Code :
  1. do {
  2.  printf(" donnez le mois " );
  3.  scanf("%d",&d.m);
  4. } while (d.m <= 0 || d.m >= 13);


ou :

Code :
  1. do {
  2.  printf(" donnez le mois " );
  3.  scanf("%d",&d.m);
  4. } while (d.m < 1 || d.m > 12);

Message cité 1 fois

---------------
C'est en écrivant n'importe quoi qu'on devient n'importe qui.
Reply

Marsh Posté le 01-06-2018 à 06:29:53    

RE mrc pour votre aide et vos remarques..
en effet au debut jai dit "la liste saffiche" alors les pro sauront que la fonctions de saisie de date marche parfaitement....
pour ta part Harkonnen avec fseek end etant donné quen ajoujant dans ma liste je veu quel le fasse en fin de liste ta ue autre soluce??  
mrc rufo je pense aussi que le probleme vient de la.

Reply

Marsh Posté le 01-06-2018 à 09:44:45    

MaybeEijOrNot a écrit :


 
Euh soit c'est moi, soit c'est toi qui est fatigué mais cela me semble logique que ce soit tant que le mois est inférieur à ou supérieur à.


non non, c'est moi qui était fatigué [:petrus75]
mais tu as quand même de la chance que ça marche, car quand tu déclares ta structure comme ceci :

Code :
  1. typedef struct date {
  2.     int j;
  3.     int m;
  4.     int a;
  5. }   date;


 
puis que plus loin tu créées une variable de type date sans l'initialiser, tu n'as aucune garantie que les membres de ta structure date soient initialisés par défaut comme tu le souhaites (par exemple avec un mois<0 ou >12. du coup, il serait parfaitement possible que tu ne passes jamais dans tes do... while().
le mieux serait d'initialiser toutes tes structures, par exemple :
 

Code :
  1. typedef struct date {
  2.     int j;
  3.     int m;
  4.     int a;
  5. }   date = {0, 0, 0};


---------------
J'ai un string dans l'array (Paris Hilton)
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed