Mini projet

**DBA_OCP** · 17/12/2007, 13h35

Bonjour,

J'ai un projet à réaliser qui parait assez simple à faire et dans le quel il faut utiliser les listes.

J'ai déjà fais les fonctions principales mais à l'execution ça marche pas...

Je vais vous faire part du projet ainsi que de mon code actuel si vous voulez bien m'aider !

**DBA_OCP** · 17/12/2007, 13h36

Ce qui est demandé ==

Objectif:

Ce mini projet a pour but de développer une application en C organisée de la manière suivante :

-Une interface qui met à disposition de l'utilisateur un menu pour accéder aux différentes tâches que permet de réaliser l'application :

* Consulter un client,
* ajouter un client,
* supprimer un client,
* .....

-Le client sera principalement présenté par :
le nom,prénom,tel,type(fidèle,régulier,saisonnier,occasionnel)

-Une bibliothèque qui regroupe toutes les fonctions conçues pour réaliser les différentes opérations élémentaires.

-Un fichier principal.

Contraintes:

-Il sera tenu compte de la qualité de la présentation, de l'organisation du travail et de son originalité.
-Un rapport contenant le programme principal et les prototypes des fonctions accompagné d'une description concises doit être rendu.

Pour mon code au quel je suis arriver depuis hier soir est le suivant ==

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
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
int c,k,tel,t,e,i;
float h;
char nom,prenom,type,n,p;
 
// LES PROTOTYPES
 
typedef struct client
{
        char nom,prenom,type;
        int tel;
        struct client *suivant;
 
}t_client;
 
t_client *estvide(t_client *c);
t_client *ajouter_client(t_client *c,char nom,char prenom,int tel);
t_client *kiem(t_client *c,int k);
t_client *saisie(t_client *c,char nom,char prenom,int tel);
t_client *sup_client(t_client *c,int k);
void typec(t_client *c,float h);
void afficher(t_client *c);
 
 
main()
{
      t_client *cl;
      cl=estvide(cl);
      cl=(t_client *)malloc(sizeof(t_client));
      cl=saisie(cl,nom,prenom,tel);
      afficher(cl);
      cl=sup_client(cl,k);
      typec(cl,h);
      system("pause");
}
 
// DEFINITION DES FONCTIONS
 
 
t_client *estvide(t_client* c)
{
         return c=NULL;
}
 
t_client *ajouter_client(t_client *c,char nom,char prenom,int tel)
{
     int e,i,t;
     char n,m;
     t_client *tmp;
     tmp=NULL;
 
       tmp=malloc(sizeof(t_client));
       tmp->nom=n;
       tmp->prenom=p;
       tmp->tel=t;
       tmp->suivant=c;
       return tmp;
}
 
 
t_client *saisie(t_client *c,char nom,char prenom,int tel)
{
 
         printf("\n\nEntrez le nbr de clients a ajouter:");
         scanf("%d",&e);
         if(e<=0)
         {
                 printf("Aucun client a ajouter\n");
         }
         else
         {
      for(i=0;i<e;i++)
      {
      printf("\n\n\tCLIENT N%d\n\n",i+1);
      printf("\nEntrez le nom du client N%d:",i+1);
      scanf("%s",&nom);
      printf("\nEntrez le prenom du client N%d:",i+1);
      scanf("%s",&prenom);
      printf("\nEntrez le N de tel du client N%d:",i+1);
      scanf("%d",&tel);
      }
      }
      return(c,nom,prenom,tel);
}
 
 
t_client *kiem(t_client *c,int k)
{
         int p=1;
         t_client *tmp;
         tmp=c;
         while(p<k)
         {
                   tmp=tmp->suivant;
                   p++;
         }
         return tmp;
}
 
t_client *sup_client(t_client *c,int k)
{
         kiem(c,k-1)->suivant=kiem(c,k)->suivant;
         return c;
}
 
void typec(t_client *c, float h)
{
     printf("Entrez le prix total des achats du client:\n");
     scanf("%f",&h);
     if(h<100)
     {
              printf("\nCLIENT OCCASIONNEL\n");
     }
     else if(h>100 && h<500)
     {
          printf("\nCLIENT SAISONNIER\n");
     }
     else if(h<1000 && h>500)
     {
          printf("\nCLIENT REGULIER\n");
     }
     else
     printf("\nCLIENT FIDEL\n");
}
 
void afficher(t_client *c)
{
     t_client *temp;
     temp=c;
 
     printf(" \n\n ");
     if(estvide(temp))
     printf("La liste est vide.");
     else
     {
         while(temp!=NULL)
         {
                          printf("\nLe Nom:  %s",temp->nom);
                          printf("\nLe prenom:  %s",temp->prenom);
                          printf("\nN de tel:  %d",temp->tel);
                          temp=temp->suivant;
         }
     }
}

**Thierry Chappuis** · 17/12/2007, 13h59

Qu'est-ce qui ne marche pas exactement? Obtiens-tu un message d'erreur, une erreur de segmentation, etc.?

Thierry

**corentin59** · 17/12/2007, 14h14

la crainte que j'avais quand j'ai lu la définition de ta structure s'avère vraie quand je lis le reste de tes fonctions : pour le nom, prénom, ... il faut un tableau de caractères, et non un seul caractère !!!

**DBA_OCP** · 17/12/2007, 14h31

Code :

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

1
2
3
4
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

Voilà pour les char c'est bon.

Mon problème pour l'instant c'est quand je donner par exemple 2 clients à ajouter (vs pouvez essayer) il me donner entrez le nom puis entrez le prenom juste après donc pas l'occasion de saisir le nom.

Après pour afficher il n'affiche qu'un seul client !

Enfin executez le et vous verrez bien...

**corentin59** · 17/12/2007, 14h46

1) après un scanf, il faut vider le buffer. tu peux ajouter cette ligne après chaque scanf :

Code :

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

while(getchar()!='\n'); // pour vider le buffer

2) dans ta fonction saisie, il faut faire appel à la fonction ajouter_client juste après avoir lu les données, ie dans la boucle for, sinon, tu demandes plusieurs fois le nom, prénom, ..., tu écrases à chaque fois la demande précédente, et tu ne crées un client qu'une fois.

3) tes fonctions vider et estvide ne servent à rien, ce n'est pas comme cela que l'on gère des listes chainées !!

4) éviter les variables globales

**BainE** · 17/12/2007, 14h52

bonjour,

P.S. : pour le numero de telephone il faudrait egalement un tableau de int. (a moins tu stockes le numero de maniere binaire dans un int

)

P.P.S : il me semble qu il serait de bon tom de recuperer la valeur de retour de la fonction ajouter et de la traiter en consequence. Sans quoi tu aura beau ajouter sans voir de modification

**Thierry Chappuis** · 17/12/2007, 15h05

Envoyé par BainE

bonjour,

P.S. : pour le numero de telephone il faudrait egalement un tableau de int. (a moins tu stockes le numero de maniere binaire dans un int

)

Pour le numéro de téléphone, je conseille plutôt une chaine de caractères.

Thierry

**DBA_OCP** · 17/12/2007, 15h05

Merci pr vos reponses.

Pour le problème de la saisie c'est reglé. Il reste le problème du stockage des clients maintenant..

Code :

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

1
2
3
4
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

**DBA_OCP** · 17/12/2007, 15h12

Envoyé par Thierry Chappuis

Pour le numéro de téléphone, je conseille plutôt une chaine de caractères.

Thierry

Comment ???

**corentin59** · 17/12/2007, 15h22

Envoyé par OthmanX

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
t_client *saisie(t_client *c)
       {
         int nr,tel,i;
         char nom[20],prenom[20],adresse[40];
         printf("Entrez le nbr de clients a ajouter:");
         scanf("%d",&e);
         while(getchar()!='\n'); // pour vider le buffer
         if(e<=0)
         {
                 printf("Aucun Client A Ajouter\n\n");
                 system("pause");
         }
         else
         {
             for(i=0;i<e;i++)
             {
                printf("Entrer le NOM du client N%d: ",i+1);
                fgets(nom, sizeof nom, stdin);
                printf("\nEntrer le PRENOM du client N%d: ",i+1);
                fgets(prenom, sizeof prenom, stdin);
                printf("Entrer l'ADRESSE du client N%d: ",i+1);
                fgets(adresse, sizeof adresse, stdin);
                printf("Entrer le NUMERO TEL. du client N%d: ",i+1);
                scanf("%d",&tel);
                while(getchar()!='\n');
                ajouter_client(c,nom,prenom,adresse,tel);
                printf("\n\n");
             }
         }
         return ajouter_client(c,nom,prenom,adresse,tel);
       }

maintenant, la ligne en rouge ne sert à rien !!

enlève tes fonctions vider et estvide ainsi que ta variable globale, tout cela ne sert à rien. ex :

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
int main(void) {
    t_client *client=NULL;
    int e;
 
    printf("Entrez le nbr de clients a ajouter : ");
    scanf("%d",&e);
    while(getchar()!='\n'); // pour vider le buffer
    printf("\n");
 
    if ( e > 0 ) {
        saisie(&client,e);
    }
 
    afficher(client);
    vider(&client);
 
    return 0;
}

fais une vraie fonction vider pour désallouer ta liste, parce que pour l'instant, tu as une belle fuite de mémoire (pas très grande mais tout de même...)

**corentin59** · 17/12/2007, 15h32

PS : voici les prototypes des fonctions comme je les définis plus haut

Code :

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

1
2
3
4
t_client *ajouter_client(int nrclnt,char *nom,char *prenom,char *adresse,char *fidel,int tel,t_client *cl);
void saisie(t_client **client,int e);
void afficher(t_client *client);
void vider(t_client **client);

PS2 : je ne réponds pas par message privé. Désolé. Par contre, cherche, propose du code (même débile, on te le pardonnera si tu y mets de la bonne volonté) et on (pas que moi, c'est pour cela qu'il vaut mieux passer par le forum) te dira ce qui ne va pas !!!

**DBA_OCP** · 17/12/2007, 15h42

Envoyé par corentin59

maintenant, la ligne en rouge ne sert à rien !!

enlève tes fonctions vider et estvide ainsi que ta variable globale, tout cela ne sert à rien. ex :

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
int main(void) {
    t_client *client=NULL;
    int e;
 
    printf("Entrez le nbr de clients a ajouter : ");
    scanf("%d",&e);
    while(getchar()!='\n'); // pour vider le buffer
    printf("\n");
 
    if ( e > 0 ) {
        saisie(&client,e);
    }
 
    afficher(client);
    vider(&client);
 
    return 0;
}

fais une vraie fonction vider pour désallouer ta liste, parce que pour l'instant, tu as une belle fuite de mémoire (pas très grande mais tout de même...)

J'ai pas bien compris ce qui se passe dans cette reponse !

**corentin59** · 17/12/2007, 15h48

Envoyé par OthmanX

J'ai pas bien compris ce qui se passe dans cette reponse !

je te propose un main (qui ressemble un petit peu au tien) qui te permet de ne pas utiliser de variables globales (e y est définie), te montre que ta fonction vider (telle que tu l'utilisais) ne sert à rien et qu'il suffit d'initialiser ton pointeur à NULL et enfin pour te montrer qu'à la fin de ton programme, il faut faire appel à une fonction (que j'ai appelée vider mais qui n'a rien à voir avec la tienne) pour desallouer ta liste.

profites-en aussi pour voir la structure de la fonction main : type de retour, utilisation de return, ...

**Thierry Chappuis** · 17/12/2007, 23h19

Envoyé par OthmanX

Envoyé par Thierry Chappuis

Pour le numéro de téléphone, je conseille plutôt une chaine de caractères.

Thierry

Comment ???

Je ne pense pas qu'un entier de type int est adapté pour représenter un numéro de téléphone et je trouve plus naturel d'utiliser une chaine de caractères.

Thierry

**jowo** · 18/12/2007, 07h50

Envoyé par OthmanX

Comment ???

Comment mémorises-tu ce numéro de téléphone avec un tableau de int +41 (32) 555 12 34?

Il est assez évident qu'un tableau de char est plus adéquat pour sauver un numéro de téléphone.

**DBA_OCP** · 20/12/2007, 13h42

Salut,

J'éssaie de travailler avec le numéro de tel en tan que char mais le problème c'est que....heu je sais pas comment vous l'expliquez mais essayez d'ajouter un client et après son numéro de tel et quand ça va s'afficher vous verrez quel numéro s'affichera !!

Code :

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

1
2
3
4
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

**DBA_OCP** · 20/12/2007, 13h48

Désolé, problème reglé pour le numéro de tél

juste un peit "%d"

Bon, pour le moment le problème principale c'est le fais quand j'ajouter par exemple 3 clients, il m'affiche que le dernier ajouter. c'est à dire ceux que j'ajouter avant se font ecraser. Pourtant mon code est bon je pense !

Qu'est ce que vous en pensez ??

**DBA_OCP** · 20/12/2007, 14h54

Code :

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

1
2
3
4
5
6
 
 
void afficher(t_client *c);
boolean estvide(t_client *c);
 
}

**diogene** · 20/12/2007, 15h05

Bon, pour le moment le problème principale c'est le fais quand j'ajouter par exemple 3 clients, il m'affiche que le dernier ajouter. c'est à dire ceux que j'ajouter avant se font ecraser.

Voir les commentaires :

Envoyé par OthmanX

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
t_client *saisie(t_client *c)
       {
....
             for(i=0;i<e;i++)
             {
....
                c = ajouter_client(c,nom,prenom,code_cl,tel);
                printf("\n\n");
             }
         }
//          return ajouter_client(c,nom,prenom,code_cl,tel);
//ceci entre deux fois le dernier client
            return c;
       }
 
void afficher(t_client *c)
  {
...
//      int e;
...
//      if(e<=0)   e n'est pas initialisé et inutile !
      if(tmp == NULL)
      printf("\nPAS DE CLIENTS\n");
....
     }

Mini projet

C

Discussions similaires

Partager

Partager