Gestion des pointeurs

**koukiya** · 14/12/2008, 16h54

salut
je vous demande de m'aider svp
j'ai un probleme avec les pointeurs
j'ai fait un programme qui gere des rendez vous et quand j'ai voulu le faire avec le formalisme pointeur j'ai rencontré des problemes...

pourriez vous m'aider svp ?
merci

**devatt** · 14/12/2008, 18h48

salut,

Si tu ne nous donnes pas de code, nous ne pourrons malheureusement pas t'aider.

**koukiya** · 14/12/2008, 22h41

ah désolé je l'ai oublié le voila

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
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<malloc.h>
typedef struct 
           { int jour;
             char mois[10];
             int annee;
           }date;
typedef struct 
{  char nom[20];
   char lieu[20];
   char categorie[20];
   double hor_debut;
   double hor_fin;
   date *D;
}rendez_vous;
 
int cpt=0;
rendez_vous *T_rendez[30];
int choix;
 
// menu
int menu()
                   {
                    printf("\n\t\t***********GESTION DES RENDEZ VOUS***********\n\n");     
                    printf("\n      1- ajouter un rendez vous ");
                    printf("\n      2- afficher un rendez vous ");
                    printf("\n      3- lister tout les rendez vous ");
                    printf("\n      0- quitter ");
 
                    printf("\n\nvotre choix: ");
                    fflush(stdin);
                    scanf("%d",&choix);
 
                    return choix;
                    }
 
 
//saisir
rendez_vous saisir()
{
        rendez_vous *r1;
 
 
       printf("\nsaisir le nom  ");
       fflush(stdin);
       gets(r1->nom);
       printf("\nsaisir le lieu  ");
       fflush(stdin);
       gets(r1->lieu);
 
     printf("\nsaisir la date jour/mois/annee\n");
     printf("\nle jour:  ");
     fflush(stdin);
     scanf("%d",r1->D->jour);
     printf("\nle mois:  ");
     fflush(stdin);
     gets(r1->D->mois);
     printf("\nl'annee:  ");
     fflush(stdin);
     scanf("%d",r1->D->annee);
 
       printf("\nHoraire de debut: ");
       scanf("%lf",r1->hor_debut);
       printf("\nHoraire de fin: ");
       scanf("%lf",r1->hor_fin);
       printf("\nCategorie: ");
       gets(r1->categorie);
       return *r1;
       }
 
       //ajouter
 
       void ajouter(rendez_vous *T_rendez[],rendez_vous *r1)
 
         {
         *(T_rendez+cpt)= r1;
          cpt++;
         }
 
 
 
          //lister les comptes
       void lister_rendez_vs(){
                        for(int j=0;j<cpt;j++)
                        {
                         printf("*******le %d rendez_vous*******\n",j);
                         printf("\nnom: %s",(*(T_rendez+j))->nom);
                         printf("\nlieu: %s",(*(T_rendez+j))->lieu);
 
                         printf("\nDate: %d %s %d",(*(T_rendez+j))->D->jour,T_rendez[j]->D->mois,T_rendez[j]->D->annee);
                         printf("\nharaire de debut: %lf",(*(T_rendez+j))->hor_debut); 
                         printf("\nharaire de fin: %lf",(*(T_rendez+j))->hor_fin); 
                         printf("\ncategorie: %s",(*(T_rendez+j))->categorie); 
 
 
                         }
 
                        }
             //afficher
          void afficher()
            {  char *x[20];
                         printf("saisir le nom ");
                         scanf("%s",&x);
                         for(int j=0;j<cpt;j++)
              {if(strcmp((*(T_rendez+j))->nom,*x)==0)
                         {printf("\nnom: %s",(*(T_rendez+j))->nom);
                         printf("\nlieu: %s",(*(T_rendez+j))->lieu);
 
                         printf("\nDate: %d %s %d",(*(T_rendez+j))->D->jour,(*(T_rendez+j))->D->mois,(*(T_rendez+j))->D->annee);
                         printf("\nharaire de debut: %lf",(*(T_rendez+j))->hor_debut); 
                         printf("\nharaire de fin: %lf",(*(T_rendez+j))->hor_fin); 
                         printf("\ncategorie: %s",(*(T_rendez+j))->categorie);}
 
              }
 
 
         }
 
 
 
 
 
 
 
         //programme principal
      main()
       {      int choix;
               rendez_vous *r1;
                 do   {     choix=menu(); 
                                fflush(stdin); 
                                      switch(choix)
                                     {
                                       case 1:    *r1=saisir();
                                                  ajouter(T_rendez,r1);
                                       break;
                                       case 2:    afficher();
                                       break;
                                       case 3:     lister_rendez_vs();
                                       break;
                                       default: printf("entrer un chiffre valable");
                                       break;
                                   }
                        }  
                                       while (choix!=0);
 
 
system("pause");
       return 0;
       }

**Emmanuel Delahaye** · 15/12/2008, 01h08

Envoyé par koukiya

ah désolé je l'ai oublié le voila

Ca fait beaucoup de choses à corriger avant d'aller plus loin...

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
 
 
-------------- Build: Debug in hello ---------------
 
Compiling: main.c
Linking console executable: bin\Debug\hello.exe
C:\dev\hello\main.c:25: warning: function declaration isn't a prototype
C:\dev\hello\main.c:42: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `saisir':
C:\dev\hello\main.c:56: warning: format argument is not a pointer (arg 2)
C:\dev\hello\main.c:62: warning: format argument is not a pointer (arg 2)
C:\dev\hello\main.c:65: warning: format argument is not a pointer (arg 2)
C:\dev\hello\main.c:67: warning: format argument is not a pointer (arg 2)
C:\dev\hello\main.c:43: warning: 'r1' might be used uninitialized in this function
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:85: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `lister_rendez_vs':
C:\dev\hello\main.c:86: error: 'for' loop initial declaration used outside C99 mode
C:\dev\hello\main.c:93: warning: ISO C90 does not support the `%lf' printf format
C:\dev\hello\main.c:94: warning: ISO C90 does not support the `%lf' printf format
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:103: warning: function declaration isn't a prototype
C:\dev\hello\main.c: In function `afficher':
C:\dev\hello\main.c:105: warning: char format, different type arg (arg 2)
C:\dev\hello\main.c:106: error: 'for' loop initial declaration used outside C99 mode
C:\dev\hello\main.c:112: warning: ISO C90 does not support the `%lf' printf format
C:\dev\hello\main.c:113: warning: ISO C90 does not support the `%lf' printf format
C:\dev\hello\main.c: At top level:
C:\dev\hello\main.c:129: warning: return type defaults to `int'
C:\dev\hello\main.c:129: warning: function declaration isn't a prototype
Process terminated with status 1 (0 minutes, 0 seconds)
2 errors, 16 warnings

- réindentation du code
- <malloc.h> n'est pas standard. <stdlib.h> suffit.

Code :

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

1
2
3
4
 
int cpt = 0;
rendez_vous *T_rendez[30];
int choix;

éviter les globales

- Dans rendez_vous saisir (void) :

- r1 est un pointeur non initialisé. Son usage entraine un comportement indéfini.
- fflush(stdin) n'existe pas (fflush n'est défini que pour les flux sortants)
- gets() est un bug potentiel qui ne demande qu'à se produire (pas de limitation de la taille saisie).
... tout va exploser...

Je suggère fgets() suivi de la fonction 'communautaire' fclean() maintes fois publiée.

Code :

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

   char *x[20];

Un tableau de 20 pointeurs sur char pour saisir un nom ? Pourquoi pas un tableau de 20 char ?

Ceci compile sans warnings :

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
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
typedef struct
{
   int jour;
   char mois[10];
   int annee;
}
date;
 
typedef struct
{
   char nom[20];
   char lieu[20];
   char categorie[20];
   double hor_debut;
   double hor_fin;
   date *D;
}
rendez_vous;
 
int cpt = 0;
rendez_vous *T_rendez[30];
int choix;
 
/* menu */
static int menu (void)
{
   printf ("\n\t\t***********GESTION DES RENDEZ VOUS***********\n\n");
   printf ("\n      1- ajouter un rendez vous ");
   printf ("\n      2- afficher un rendez vous ");
   printf ("\n      3- lister tout les rendez vous ");
   printf ("\n      0- quitter ");
 
   printf ("\n\nvotre choix: ");
   fflush (stdin);
   scanf ("%d", &choix);
 
   return choix;
}
 
/* saisir */
static rendez_vous saisir (void)
{
   rendez_vous r1;
 
   printf ("\nsaisir le nom  ");
   gets (r1.nom);
 
   printf ("\nsaisir le lieu  ");
   gets (r1.lieu);
 
   printf ("\nsaisir la date jour/mois/annee\n");
   printf ("\nle jour:  ");
   scanf ("%d", &r1.D->jour);
   printf ("\nle mois:  ");
   gets (r1.D->mois);
   printf ("\nl'annee:  ");
   scanf ("%d", &r1.D->annee);
 
   printf ("\nHoraire de debut: ");
   scanf ("%lf", &r1.hor_debut);
   printf ("\nHoraire de fin: ");
   scanf ("%lf", &r1.hor_fin);
   printf ("\nCategorie: ");
   gets (r1.categorie);
 
   return r1;
}
 
/* ajouter */
void ajouter (rendez_vous * T_rendez[], rendez_vous * r1)
{
   T_rendez[cpt] = r1;
   cpt++;
}
 
          /* lister les comptes */
void lister_rendez_vs (void)
{
   int j;
   for (j = 0; j < cpt; j++)
   {
      printf ("*******le %d rendez_vous*******\n", j);
      printf ("\nnom: %s", (*(T_rendez + j))->nom);
      printf ("\nlieu: %s", (*(T_rendez + j))->lieu);
 
      printf ("\nDate: %d %s %d", (*(T_rendez + j))->D->jour,
              T_rendez[j]->D->mois, T_rendez[j]->D->annee);
      printf ("\nharaire de debut: %f", (*(T_rendez + j))->hor_debut);
      printf ("\nharaire de fin: %f", (*(T_rendez + j))->hor_fin);
      printf ("\ncategorie: %s", (*(T_rendez + j))->categorie);
 
   }
 
}
 
             /* afficher */
void afficher (void)
{
   char x[20];
   printf ("saisir le nom ");
   scanf ("%s", x);
   {
      int j;
      for (j = 0; j < cpt; j++)
      {
         if (strcmp ((*(T_rendez + j))->nom, x) == 0)
         {
            printf ("\nnom: %s", (*(T_rendez + j))->nom);
            printf ("\nlieu: %s", (*(T_rendez + j))->lieu);
 
            printf ("\nDate: %d %s %d", (*(T_rendez + j))->D->jour,
                    (*(T_rendez + j))->D->mois, (*(T_rendez + j))->D->annee);
            printf ("\nharaire de debut: %f", (*(T_rendez + j))->hor_debut);
            printf ("\nharaire de fin: %f", (*(T_rendez + j))->hor_fin);
            printf ("\ncategorie: %s", (*(T_rendez + j))->categorie);
         }
      }
   }
 
}
 
/* programme principal */
int main (void)
{
   int choix;
   rendez_vous r1;
   do
   {
      choix = menu ();
      fflush (stdin);
      switch (choix)
      {
      case 1:
         r1 = saisir ();
         ajouter (T_rendez, &r1);
         break;
      case 2:
         afficher ();
         break;
      case 3:
         lister_rendez_vs ();
         break;
      default:
         printf ("entrer un chiffre valable");
         break;
      }
   }
   while (choix != 0);
 
   system ("pause");
   return 0;
}

Mais ça ne fonctionne probablement pas.

Tu as tendance à utiliser les pointeurs sans raison valable
Ta connaissance du C est naïve.

Je suggère de tout reprendre de 0 en appliquant des méthodes rigoureuses basées sur les principes de base de la programmation objet, à savoir :

Les données sont dans une ou des structures
on créée des fonctions pour manipuler ces données.

Tu as commencé à le faire, mais de manière disparate et non homogène avec des pointeurs non initialisés dans tous les sens. Bref, du code catastrophique.

Sans méthode, n'essaye même pas.

Gestion des pointeurs

C++

Discussions similaires

Partager

Partager