valeur de gets

Version imprimable

valeur de gets

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:

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

05/04/2008, 19h57
Thierry Chappuis

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
05/04/2008, 20h39
darkwall_37

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
05/04/2008, 22h36
info_amel

Citation:

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

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
05/04/2008, 22h50
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
05/04/2008, 23h10
nicolas.sitbon

Citation:

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

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

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

06/04/2008, 21h56
Médinoc

Reste appuyé sur une touche quelques minutes lors du gets(HM), et tu verras si ça marche toujours....
06/04/2008, 22h01
darkwall_37

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.
06/04/2008, 23h09
Thierry Chappuis

Citation:

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
06/04/2008, 23h15
Emmanuel Delahaye

Citation:

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.
06/04/2008, 23h16
Emmanuel Delahaye

Citation:

Envoyé par Emmanuel Delahaye

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

Citation:

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:

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