Mini projet

Version imprimable

17/12/2007, 13h35
DBA_OCP

Mini projet

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 !

Citation:

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:

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;
         }
     }
}

17/12/2007, 13h59
Thierry Chappuis

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

Thierry
17/12/2007, 14h14
corentin59

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 !!!
17/12/2007, 14h31
DBA_OCP
Code:

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...
17/12/2007, 14h46
corentin59

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

Code:

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
17/12/2007, 14h52
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 :aie:)

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
17/12/2007, 15h05
Thierry Chappuis

Citation:

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 :aie:)

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

Thierry
17/12/2007, 15h05
DBA_OCP
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:

1 2 3 4 #include<stdio.h> #include<stdlib.h> #include<string.h>
17/12/2007, 15h12
DBA_OCP

Citation:

Envoyé par Thierry Chappuis

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

Thierry

Comment ???

Citation:

Envoyé par OthmanX

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
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:

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...)

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

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 !!!

Citation:
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:

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 !

17/12/2007, 15h48
corentin59

Citation:

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, ...
17/12/2007, 23h19
Thierry Chappuis

Citation:

Envoyé par OthmanX

Citation:

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
18/12/2007, 07h50
jowo

Citation:

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.
20/12/2007, 13h42
DBA_OCP
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:

1 2 3 4 #include<stdio.h> #include<stdlib.h> #include<string.h>
20/12/2007, 13h48
DBA_OCP

Désolé, problème reglé pour le numéro de tél :king: 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 ??

Code:

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

Citation:

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 :
Citation:
Envoyé par OthmanX
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 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"); .... }

20/12/2007, 15h06
BainE

Bonjour,

comme je te l ai deja dis 6 post avant, tant que tu ne recuperes pas le retour de ta fonction ajouter (ou faire une insertion en fait de chaine) t aura toujours le meme probleme
21/12/2007, 19h48
DBA_OCP

Bonsoir,

Merci encore pour vos reponses.
J'ai fais exactement ce que diogene m'a dis sans bien comprendre car déjà je n'ai jamais vu 2 return en une seule fonctions...

Enfin bref, je laisse les questions pour plustard tant que ça marche pour le moment ;)

Rebonsoir,

En ce moment je suis dans l'idée pour que l'utilisateur puisse faire une rechèrche par code ou bien par nom dans les clients qui existent.

Ensuite pouvoir supprimer un client en entrant son nom.

Au début biensur ça devra lui donner désolé aucun client enregistrer/à supprimer..

Je trouve pas l'idée pour le faire meme en essayant avec cette fonction.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 t_client *recherche_code(t_client *c,char *CODE_CL) { char code_cl[10]; printf("Entrez le code du client que vous cherchez:"); fgets(code_cl,sizeof code_cl,stdin); if(code_cl(/*condition*/)) printf("Ce code client n'existe pas!!"); else /*....C'est dans ce else qu'il y a les problèmes, je trouve meme pas l'idée ou bien c'est très compliquer....*/ ................

22/12/2007, 17h03
diogene
Citation:

Envoyé par OthmanX

Rebonsoir,
En ce moment je suis dans l'idée pour que l'utilisateur puisse faire une rechèrche par code ou bien par nom dans les clients qui existent.
Ensuite pouvoir supprimer un client en entrant son nom.
Au début biensur ça devra lui donner désolé aucun client enregistrer/à supprimer..
Je trouve pas l'idée pour le faire meme en essayant avec cette fonction.

La fonction reçoit en argument le début de la liste et le code à chercher.
Il faut parcourir la liste jusqu'à ce qu'on trouve le code ( et on renvoie alors le t_client * où on l'a trouvé) ou jusqu'à ce qu'on arrive à la fin de la liste (et on renvoie alors NULL pour dire qu'il n'a pas été trouvé). Pour comparer le code d'un client et le code cherché, utiliser strcmp() qui compare les chaînes de caractères. L'opérateur == comparerai lui les adresses des chaînes ce qui n'est pas le but.
Code:

1 2 3 4 5 t_client *recherche_code(t_client *c,char *CODE_CL) { while(c != NULL && strcmp(c->code_cl ,CODE_CL) != 0) c = c->suivant; return c; }

Merci encore diogène..

Je galère depuis plus d'une heure pour pouvoir faire fonctionner la recherche mais j'y arrive pas !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
void recherche_code(t_client *c,char *CODE_CL)
{
             t_client* tmp;
             tmp=c;
             char code_cl[10];
            // while(c != NULL && strcmp(c->code_cl ,CODE_CL) != 0) 
            printf("\nEntrez le code client que vous chercher:");
            scanf("%s",&code_cl);
            if(strcmp(c->code_cl ,CODE_CL))
            {
             printf("Nom:       %s\n",tmp->nom);
             printf("Prenom:    %s\n",tmp->prenom);
             printf("Code Client:    %s\n",tmp->code_cl);
             printf("Tel:       %s\n\n",tmp->tel);
            }
            else
            printf("Ce code n'existe pas!!");
            system("pause");
}