valeur de gets

**darkwall_37** · 05/04/2008, 19h27

Bonjour,

J'ai un souci avec le premier gets du code. En effet en faisant le choix, le programme zappe le premier gets ! Cependant le premier gets me laisse entrer ma valeur ! Je suppose que gets lit quelque chose et le prend pour valeur mais je ne vois pas où cela se passe !

Voici le code :

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
#include<stdio.h>
#include<stdlib.h>
 
//-----< ENTETES DE FONCTIONS >-----//
 
void AFFICHAGE_DE_LA_CMD();
void MENU();
 
//----------------------------------//
 
 
 
//-----< FONCTION PRINCIPALE >-----//
 
int main (void)
{
     AFFICHAGE_DE_LA_CMD();
 
     while ( 1 )
           {
                     MENU();
           }
}
 
//---------------------------------//
 
 
 
 
 
 
 
void AFFICHAGE_DE_LA_CMD()
{
     system("echo off");
     system("title CREATION TACHES PLANIFIEES");
     system("MODE CON lines=40 cols=83");
     system("color 19");
}
 
 
void MENU()
{
 
       int choix=0;
       int validation=0;
       char HM[5];
       char CHEMIN[999];
       char commande_AT[999];
       FILE *taches=fopen("taches.txt","w");
 
       printf(" \n +------------------------------------------------------------------------------+\n");
       printf(" |                                       MENU                                   |\n");
       printf(" +------------------------------------------------------------------------------+\n");
       printf(" |                                                                              |\n");
       printf(" |                              [1] Cr\202er une tache                             |\n"); 
       printf(" |                                                                              |\n");
       printf(" |                             [2] Supprimer les taches                         |\n");
       printf(" |                                                                              |\n");
       printf(" +------------------------------------------------------------------------------+\n\n");
       printf("                             Choix (1/2) : ");
 
       scanf("%d",&choix);
 
       if ( choix==1 )
          {
                     printf("\n +------------------------------------------------------------------------------+\n");
                     printf(" |                        REGLAGE DE L'HEURE D'EXECUTION                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |       Rentrez l'information au format suivant -> heure:min  ex 08:34         |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                      valeur : \n\n");
                     gets(HM);
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |                        CHEMIN D'EXECUTION DE LA TACHE                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |                 exemple :   C:\\DOS1\\DOS2\\nom_fichier.***                     |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                      valeur : ");
                     gets(CHEMIN);
 
                     sprintf(commande_AT," AT \"%s\" /EVERY:l,ma,me,j,v,s,d \"%s\" ", HM, CHEMIN);
                     fprintf(taches," AT %s /EVERY:l,ma,me,j,v,s,d %s ", HM, CHEMIN );
                     system(commande_AT);
                     fclose(taches);
          }
 
 
       if ( choix==2 )
          {
                     printf("\n +------------------------------------------------------------------------------+\n");
                     printf(" |                        CONFIRMATION DE LA SUPPRESSION                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                      valeur (1->oui)(2->non): ");
                     scanf("%d",&validation);
 
                                      if ( validation==1 )
                                         {
                                                          system("AT /DELETE /YES");
                                                          printf("\n +------------------------------------------------------------------------------+\n");
                                                          printf(" |                              TACHES SUPPRIMEES                               |\n");
                                                          printf(" +------------------------------------------------------------------------------+\n\n");
                                         }
 
                                    else { system("exit"); }     
          }
 
 
       if ( choix==3 )
          {
                   system("exit");
          }
 
}

Merci encore

**Thierry Chappuis** · 05/04/2008, 19h57

gets() saisit une chaine à partir du flux d'entrée standard et la place chacun de ses caractères dans un tableau de caractères dont l'adresse est passée en 1er argument. Cette fonction est un bug. Elle est officiellement dépréciée par la norme. Il est donc fortement recommandé de ne pas l'utiliser au profit de fgets() (qui est bien plus sûre).

Thierry

**darkwall_37** · 05/04/2008, 20h39

Ok merci, je ne connais pas cette fonction. Je suis allé voir votre lien mais je n'ai pas trop compris.
Qu'est ce que je suis censé mettre comme flux. Quand je mets stdin il me rétorque à l'exécution que la liste est vide.

Merci d'avence

**info_amel** · 05/04/2008, 22h36

Envoyé par darkwall_37

Bonjour,

J'ai un souci avec le premier gets du code. En effet en faisant le choix, le programme zappe le premier gets ! Cependant le premier gets me laisse entrer ma valeur ! Je suppose que gets lit quelque chose et le prend pour valeur mais je ne vois pas où cela se passe !

beh je ne suis pas si forte en c mais je vais essayé de t'aider

:

tu as écrit :
scanf("Md",&choix);
puis
gets(hm);

lors de l'execution de scanf(...) la valeur entrée est le choix MAIS dans le flux il reste le caractère "entrer" que tu as presser .
lors de l'execution de gets(...) elle prends comme valeur le caractère "entrer" qui a rester dans le flux .
ALORS la solution est aprés scanf tu met l'instruction "getchar();" puis le programme ne va pas zapper le gets

bon courage

**darkwall_37** · 05/04/2008, 22h50

Merci ! bon de toute façon leur histoire de fgets ça marche pas parce que je ne sais pas m'en servir correctement a priori donc peut importe j'ai collé un fflush(stdin) avant chaque gets et ça marche nikel !

Merci

**nicolas.sitbon** · 05/04/2008, 23h10

Envoyé par darkwall_37

Merci ! bon de toute façon leur histoire de fgets ça marche pas parce que je ne sais pas m'en servir correctement a priori donc peut importe j'ai collé un fflush(stdin) avant chaque gets et ça marche nikel !

Merci

Plutôt que d'écrire de bêtise, instruit toi :
http://emmanuel-delahaye.developpez....tes.htm#saisie

**darkwall_37** · 06/04/2008, 21h37

Des bétises certes mais ça marche, je n'arrive pas à mettre en œuvre ton fgets ! et pourtant j'ai essayé !

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
#include<stdio.h>
#include<stdlib.h>
 
//-----< ENTETES DE FONCTIONS >-----//
 
void AFFICHAGE_DE_LA_CMD();
void MENU();
 
//----------------------------------//
 
 
 
//-----< FONCTION PRINCIPALE >-----//
 
int main (void)
{
     AFFICHAGE_DE_LA_CMD();
 
     while ( 1 )
           {
                     MENU();
           }
 
     system("exit");
}
 
//---------------------------------//
 
 
 
 
 
 
 
void AFFICHAGE_DE_LA_CMD()
{
     system("@echo off");
     system("title CREATION TACHES PLANIFIEES");
     system("MODE CON lines=15 cols=83");
     system("color 19");
}
 
 
void MENU()
{
 
       int choix=0;
       int validation=0;
       char HM[5];
       char CHEMIN[999];
       char commande_AT[999];
       FILE *taches=fopen("taches.txt","w");
 
       printf(" \n +------------------------------------------------------------------------------+\n");
       printf(" |                                       MENU                                   |\n");
       printf(" +------------------------------------------------------------------------------+\n");
       printf(" |                                                                              |\n");
       printf(" |                            [1] Cr\202er une tache                               |\n"); 
       printf(" |                                                                              |\n");
       printf(" |                     [2] Regler l'heure de reboot du PC                       |\n");
       printf(" |                                                                              |\n");
       printf(" |                           [3] Supprimer les taches                           |\n");
       printf(" |                                                                              |\n");
       printf(" +------------------------------------------------------------------------------+\n\n");
       printf("                                   Choix (1/2) : ");
 
       scanf("%d",&choix);
 
       if ( choix==1 )
          {
 
                     system("@echo off");
                     system("title AJOUT DE TACHES");
                     system("MODE CON lines=8 cols=83");
                     system("color 19");
 
                     fflush(stdin);
                     printf("\n +------------------------------------------------------------------------------+\n");
                     printf(" |                        REGLAGE DE L'HEURE D'EXECUTION                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |       Rentrez l'information au format suivant -> heure:min  ex 08:34         |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                      valeur : ");
                     fflush(stdin);
                     gets(HM);
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |                        CHEMIN D'EXECUTION DE LA TACHE                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |                 exemple :   C:\DOS1\DOS2\nom_fichier.***                     |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                      valeur : ");
                     fflush(stdin);
                     gets(CHEMIN);
 
                     system("title CREATION TACHES PLANIFIEES");
                     system("MODE CON lines=15 cols=83");
 
                     sprintf(commande_AT," AT %s /EVERY:l,ma,me,j,v,s,d %s ", HM, CHEMIN);
                     fprintf(taches," AT %s /EVERY:l,ma,me,j,v,s,d %s ", HM, CHEMIN );
                     system(commande_AT);
                     fclose(taches);
          }
 
 
       if ( choix==2 )
          {
                     char HR[5];
                     char REBOOT[999];
 
                     system("title HEURE DE REBOOT");
                     system("MODE CON lines=9 cols=83");
 
                     printf("\n +------------------------------------------------------------------------------+\n");
                     printf(" |                          REGLAGE DE L'HEURE DE REBOOT                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n");
                     printf(" |       Rentrez l'information au format suivant -> heure:min  ex 08:34         |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                            valeur : ");
                     fflush(stdin);
                     gets(HR);
 
 
                     sprintf(REBOOT," AT %s /INTERACTIVE /EVERY:l,ma,me,j,v,s,d Shutdown -r", HR);
                     fprintf(taches," AT %s /INTERACTIVE /EVERY:l,ma,me,j,v,s,d Shutdown -r ", HR);
                     system(REBOOT);
                     fclose(taches);
 
                     system("title CREATION TACHES PLANIFIEES");
                     system("MODE CON lines=15 cols=83");
 
          }
 
 
       if ( choix==3 )
          {
 
                     system("title SUPPRESSION DE TACHES");
                     system("MODE CON lines=5 cols=83");
 
                     printf("\n +------------------------------------------------------------------------------+\n");
                     printf(" |                        CONFIRMATION DE LA SUPPRESSION                        |\n");
                     printf(" +------------------------------------------------------------------------------+\n\n");
                     printf("                              valeur (1/2): ");
                     scanf("%d",&validation);
 
                                      if ( validation==1 )
                                         {
                                                          system("AT /DELETE /YES");
                                                          printf("\n +------------------------------------------------------------------------------+\n");
                                                          printf(" |                              TACHES SUPPRIMEES                               |\n");
                                                          printf(" +------------------------------------------------------------------------------+\n\n");
                                         }
 
                                    else { system("exit"); }     
 
                     system("title CREATION TACHES PLANIFIEES");
                     system("MODE CON lines=15 cols=83");
          }
 
 
       if ( choix==3 )
          {
                   system("exit");
          }
 
}

Comme cela ça marche essayes si tu ne me crois pas !

**Médinoc** · 06/04/2008, 21h56

Reste appuyé sur une touche quelques minutes lors du gets(HM), et tu verras si ça marche toujours....

**darkwall_37** · 06/04/2008, 22h01

lol, faut pas abuser non plus : je cherche pas par tous les moyens à faire planter le programme ! tu n'as qu'a mettre tes fgets et si ça marche je suis preneur sinon j'ai déjà essayé et j'ai pas envie de me reprendre la tête, je m'y repencherais à l'occasion.

**Thierry Chappuis** · 06/04/2008, 23h09

Envoyé par darkwall_37

lol, faut pas abuser non plus : je cherche pas par tous les moyens à faire planter le programme !

Toi, non! Mais l'utilisateur lambda... parfois (souvent?)!

Thierry

**Emmanuel Delahaye** · 06/04/2008, 23h15

Envoyé par Thierry Chappuis

Toi, non! Mais l'utilisateur lambda... parfois (souvent?)!

En tout cas, les 12 ouistitis qui me servent à tester les entrées ça ne va pas arréter.

**Emmanuel Delahaye** · 06/04/2008, 23h16

Envoyé par Emmanuel Delahaye

En tout cas, les 12 ouistitis qui me servent à tester les entrées ça ne va pas arréter.

**Emmanuel Delahaye** · 06/04/2008, 23h31

Envoyé par darkwall_37

Des bétises certes mais ça marche, je n'arrive pas à mettre en œuvre ton fgets ! et pourtant j'ai essayé !
<...>
Comme cela ça marche essayes si tu ne me crois pas !

Peut importe que 'ça marche ou pas'...

gets() est un bug qui ne doit pas être utilisé.
fflush() n'est défini que pour les flux sortants.

Voici une version plus sûre et plus portable :

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
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
static void purge (void)
{
   int c;
   while ((c = getchar ()) != '\n' && c != EOF)
   {
   }
}
 
static void clean (char *s)
{
   char *p = strchr (s, '\n');
   if (p != NULL)
   {
      *p = 0;
   }
   else
   {
      purge ();
   }
}
 
//---------------------------------//
 
void AFFICHAGE_DE_LA_CMD (void)
{
   system ("@title CREATION TACHES PLANIFIEES");
   system ("@MODE CON lines=15 cols=83");
   system ("@color 19");
}
 
void MENU (void)
{
 
   int choix = 0;
   int validation = 0;
   char HM[5];
   char CHEMIN[999];
   char commande_AT[999];
   FILE *taches = fopen ("taches.txt", "w");
 
   printf
      (" \n +------------------------------------------------------------------------------+\n");
   printf
      (" |                                       MENU                                   |\n");
   printf
      (" +------------------------------------------------------------------------------+\n");
   printf
      (" |                                                                              |\n");
   printf
      (" |                            [1] Creer une tache                               |\n");
   printf
      (" |                                                                              |\n");
   printf
      (" |                     [2] Regler l'heure de reboot du PC                       |\n");
   printf
      (" |                                                                              |\n");
   printf
      (" |                           [3] Supprimer les taches                           |\n");
   printf
      (" |                                                                              |\n");
   printf
      (" +------------------------------------------------------------------------------+\n\n");
   printf ("                                   Choix (1/2) : ");
 
   scanf ("%d", &choix);
   purge ();
 
   if (choix == 1)
   {
      system ("@title AJOUT DE TACHES");
      system ("@MODE CON lines=8 cols=83");
      system ("@color 19");
 
      printf
         ("\n +------------------------------------------------------------------------------+\n");
      printf
         (" |                        REGLAGE DE L'HEURE D'EXECUTION                        |\n");
      printf
         (" +------------------------------------------------------------------------------+\n");
      printf
         (" |       Rentrez l'information au format suivant -> heure:min  ex 08:34         |\n");
      printf
         (" +------------------------------------------------------------------------------+\n\n");
      printf ("                      valeur : ");
 
      fgets (HM, sizeof HM, stdin);
      clean (HM);
 
      printf
         (" +------------------------------------------------------------------------------+\n");
      printf
         (" |                        CHEMIN D'EXECUTION DE LA TACHE                        |\n");
      printf
         (" +------------------------------------------------------------------------------+\n");
      printf
         (" |                 exemple :   C:\\DOS1\\DOS2\nom_fichier.***                     |\n");
      printf
         (" +------------------------------------------------------------------------------+\n\n");
      printf ("                      valeur : ");
      fgets (CHEMIN, sizeof CHEMIN, stdin);
      clean (CHEMIN);
 
      system ("title CREATION TACHES PLANIFIEES");
      system ("MODE CON lines=15 cols=83");
 
      sprintf (commande_AT, " AT %s /EVERY:l,ma,me,j,v,s,d %s ", HM, CHEMIN);
      fprintf (taches, " AT %s /EVERY:l,ma,me,j,v,s,d %s ", HM, CHEMIN);
      system (commande_AT);
      fclose (taches);
   }
 
   if (choix == 2)
   {
      char HR[5];
      char REBOOT[999];
 
      system ("@title HEURE DE REBOOT");
      system ("@MODE CON lines=9 cols=83");
 
      printf
         ("\n +------------------------------------------------------------------------------+\n");
      printf
         (" |                          REGLAGE DE L'HEURE DE REBOOT                        |\n");
      printf
         (" +------------------------------------------------------------------------------+\n");
      printf
         (" |       Rentrez l'information au format suivant -> heure:min  ex 08:34         |\n");
      printf
         (" +------------------------------------------------------------------------------+\n\n");
      printf ("                            valeur : ");
      fgets (HR, sizeof HM, stdin);
      clean (HR);
 
      sprintf (REBOOT,
               "@AT %s /INTERACTIVE /EVERY:l,ma,me,j,v,s,d Shutdown -r", HR);
      fprintf (taches,
               "@AT %s /INTERACTIVE /EVERY:l,ma,me,j,v,s,d Shutdown -r ", HR);
      system (REBOOT);
      fclose (taches);
 
      system ("@title CREATION TACHES PLANIFIEES");
      system ("@MODE CON lines=15 cols=83");
 
   }
 
   if (choix == 3)
   {
 
      system ("@title SUPPRESSION DE TACHES");
      system ("@MODE CON lines=5 cols=83");
 
      printf
         ("\n +------------------------------------------------------------------------------+\n");
      printf
         (" |                        CONFIRMATION DE LA SUPPRESSION                        |\n");
      printf
         (" +------------------------------------------------------------------------------+\n\n");
      printf ("                              valeur (1/2): ");
      scanf ("%d", &validation);
      purge ();
      if (validation == 1)
      {
         system ("@AT /DELETE /YES");
         printf
            ("\n +------------------------------------------------------------------------------+\n");
         printf
            (" |                              TACHES SUPPRIMEES                               |\n");
         printf
            (" +------------------------------------------------------------------------------+\n\n");
      }
 
      else
      {
         system ("exit");
      }
 
      system ("@title CREATION TACHES PLANIFIEES");
      system ("@MODE CON lines=15 cols=83");
   }
 
   if (choix == 3)
   {
      system ("exit");
   }
 
}
 
//-----< FONCTION PRINCIPALE >-----//
 
int main (void)
{
   AFFICHAGE_DE_LA_CMD ();
 
   while (1)
   {
      MENU ();
   }
 
   system ("exit");
}

valeur de gets

C

Discussions similaires

Partager

Partager