comportement bizzare lors d'un test

Version imprimable

Ceci est un programme qui enregistre un nom et un mumero de telephone.

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
void creer(struct Enreg fiche){
     int rep=0; //valeur du test
 
        printf("Sous-Programme création dune fiche.\n");
        printf("Appuyez sur une touche...\n");
        getch();
 
        //saisie du nom et du numero
             printf("\nSaisir le nom: ");
             scanf("%s", &fiche.nom);
             printf("Saisir le numero: ");
             scanf("%s", &fiche.tel);
 
//on demande si on veux conserver la fiche ou pas
 
             printf("Voulez vous conserveez cette fiche(1=yes//0=no): ");
             scanf("%d", &rep);
   /*si oui , tenter d'ouvrir le fichier*/         
             if(rep==1){
             fd=fopen("repertoire.txt", "a+");
/*echec d'ouverture*/
             if(fd==NULL){
                     printf("\nError lors de l/'ouverture du fichier spécifié.!\n");
                     }
/*ouverture réussie, on enregistre les données*/
                     else{
                          fprintf(fd, "%s\n", fiche.nom);
                          fprintf(fd, "%s\n", fiche.tel);
 
                          fclose(fd);
                          printf("Sauvegarde reussie!\n");
                          }
                     }
/*au cas ou on ne veux pas conserver la fiche*/
 
    else{
         fclose(fd);
         printf("Echec sauvegarde!\n");
         } 
 
             return;
        }

mon probleme se situe à ce stade:

Code:

1
2
3
4
5
6
//on demande si on veux conserver la fiche ou pas
 
             printf("Voulez vous conserveez cette fiche(1=yes//0=no): ");
             scanf("%d", &rep);
   /*si oui , tenter d'ouvrir le fichier*/         
             if(rep==1)

Ainsi ici "rep" est declaré de type int, et je n'ai pas de probleme.

mais quand je déclare "rep" de type char, pour qu'on lui affecte la valeur 'y' ou 'n' ...bein ça ne marche pas, le programme s'arrête. :(

Citation:
Envoyé par Linu6
Code:

1 2 3 4 5 6 //on demande si on veux conserver la fiche ou pas printf("Voulez vous conserveez cette fiche(1=yes//0=no): "); scanf("%d", &rep); /*si oui , tenter d'ouvrir le fichier*/ if(rep==1)
Ainsi ici "rep" est declaré de type int, et je n'ai pas de probleme.

mais quand je déclare "rep" de type char, pour qu'on lui affecte la valeur 'y' ou 'n' ...bein ça ne marche pas, le programme s'arrête. :(
Normal. Etant donné que "%d" avec scanf() attend l'adresse d'un int, il faut suivre la consigne. Si on fait autre chose, le comportement est indéfini. Point.

06/10/2007, 19h52
Thierry Chappuis

Citation:

Envoyé par Linu6

mais quand je déclare "rep" de type char, pour qu'on lui affecte la valeur 'y' ou 'n' ...bein ça ne marche pas, le programme s'arrête. :(

Si tu veux saisir un caractère comme 'y' ou 'n' depuis le flux d'entrée standard, je te conseille de déclarer rep comme étant de type int et d'utiliser la fonction getchar() de la bibliothèque standard. Pour travailler proprement. n'oublie pas de nettoyer le tampon du flux d'entrée standard après l'usage de getchar() (histoire d'éviter les mauvaises surprises).

Pour étudier le comportement de la fonction getchar() ou fgetc(), je te conseille de lire cet article: http://emmanuel-delahaye.developpez.com/inputs.htm.

Thierry
06/10/2007, 23h42
Linu6

Merci pour vos remarques.

:wink:
07/10/2007, 20h39
Linu6
Euh..au fait le getchar() ne me reussi pas trop.
Voici mon probleme.

Toujours dans le meme code, mais dans cette partie cette fois-ci:
Code:

1 2 3 /saisie du nom et du numero printf("\nSaisir le nom: "); scanf("%s", &fiche.nom);
j'ai voulu modifier l'affectation du nom, pour que celle-ci se fasse caractère par caractère.

Mais à l'execution, le programme saute le passage de l'affectation du nom.

Voici mon code.
Code:

1 2 3 4 5 printf("\nSaisir le nom: "); while(c!='\n'){ for(i=0; toupper(c=getchar()); i++); fiche.nom[i]=c; }
où i est une variable locale
et fiche.nom est un membre d'une structure de ce type:
Code:

1 2 3 4 struct Enreg{ char nom[40]; char tel[10]; };
07/10/2007, 22h00
Emmanuel Delahaye
Citation:
Envoyé par Linu6

Euh..au fait le getchar() ne me reussi pas trop.
Voici mon probleme.

Toujours dans le meme code, mais dans cette partie cette fois-ci:

Code:

1 2 3 /saisie du nom et du numero printf("\nSaisir le nom: "); scanf("%s", &fiche.nom);

j'ai voulu modifier l'affectation du nom, pour que celle-ci se fasse caractère par caractère.

Mais à l'execution, le programme saute le passage de l'affectation du nom.

Voici mon code.

Code:

1 2 3 4 5 printf("\nSaisir le nom: "); while(c!='\n'){ for(i=0; toupper(c=getchar()); i++); fiche.nom[i]=c; }

où i est une variable locale
et fiche.nom est un membre d'une structure de ce type:

Code:

1 2 3 4 struct Enreg{ char nom[40]; char tel[10]; };
Comme Thierry se tue à te l'expliquer, quand tu utilises getchar(), il faut ensuite purger le flux. Le bon usage minimum est :
Code:

1 2 3 4 5 6 7 8 9 10 11 printf ("Continuer o/n ?\n") c = getchar(); /* int c; */ if (c != '\n') { while (getchar()) != '\n') { } } /* agir selon c */
Evidemment, on ne fait pas ça à chaque fois et on se construit rapidement un fonction de saisie de caractères :
Code:

1 2 3 4 5 int get_c(void) { /* je te laisse completer */ }
Qui s'utilise comme ceci :
Code:

1 2 3 4 5 printf ("Continuer o/n ?\n") c = get_c(); /* int c; */ /* agir selon c */
Bref, il faut passer un peu de temps à maitriser les I/O en C avant d'écrire une application, sinon, on fonce dans le mur...

Il faut faire pareil avec les saisies de lignes. Je recommande d'écrire une fonction :
Code:

1 2 3 4 5 6 7 8 9 int get_line(char *s, size_t size) { int err = 0; /* a completer */ return err; }
qui s'utilise ainsi (usage minimaliste) :
Code:

1 2 3 4 5 /* saisie du nom et du numero */ printf("Saisir le nom :\n"); get_line(fiche.nom, sizeof fiche.nom); /* etc. */
Si il faut obtenir une valeur numérique, après un get_line() dans un tableau local (char line[32], par exemple), on fait une conversion avec sscanf() ou strtol(), par exemple.

on peut aussi construire une fonction de saisie numérique :
Code:

1 2 3 4 5 int get_integer (long *p) { /* etc. */ }
Il y a de quoi s'amuser avant d'écrire une application réelle...
07/10/2007, 23h05
Linu6

Je ne comprend pas . ...ca veut pas marcher.
:help:
07/10/2007, 23h46
Emmanuel Delahaye

Citation:

Envoyé par Linu6

Je ne comprend pas . ...ca veut pas marcher.
:help:

Eh ? On est censé deviner ce qui ne va pas ?
07/10/2007, 23h46
Thierry Chappuis

Qu'est-ce que tu ne comprends pas? Peux-tu poster ton code dans son état actuel?

Thierry

Voila le code: tout marche sauf la saisie du nom par la methode getcher() :(

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
 
 
//Definition du type.
struct Enreg{
       char nom[40];
       char tel[10];
       };
 
 
//prototypes des fonctions
 
int menu(void);
void creer(struct Enreg fiche);
void recherche(struct Enreg fiche);
FILE *fd;
void saisi_data(struct Enreg *fiche);
 
 
//programme ppal
int main()
{
    /*déclaration et initialisation
    des varoables*/
 
    struct Enreg fiche={0, 0};
 
    int choix;
 
 
    /*Menu*/
    do{
        choix=menu();
        switch(choix){
                      case 1:
 
                           creer(fiche);
                           printf("\n");
 
                           continue;
 
                       case 2:
                           recherche(fiche);
 
                           printf("\n");
 
                           continue;  
 
 
                      default:
                              printf("Au revoir...\n\n");
 
                              }
                             } while(choix!=3);
 
           system("PAUSE");                   
 
}
 
/*Programme affichant le menu*/
int menu(void)
{
    int i;
    do{
        printf("\nMenu Principal\n");
               printf(" 1 - Creation\n");
               printf(" 2 - Recherche\n");
               printf(" 3 - Fin\n");
               printf("Indiquez l'op choisie(1. 2. 3.): ");
               scanf("%d", &i);
               /*Tant que la valeur ne correspond pas 
               au choix proposés au menu affiche: Error!*/
               if(i<1 || i>3) 
               printf("\nErr recommencez\n");
               }while(i<1 || i>3);
               printf("\n");
               return(i);
               }
/*programme permettant de creer une fiche de renseignement*/               
void creer(struct Enreg fiche){
     int rep=0; //valeur de test
 
 
        printf("Sous-Programme création dune fiche.\n");
        printf("Appuyez sur une touche...\n");
        getch();
 
 
             saisi_data(&fiche); /*Appel de la fonction
              de saisie des valeurs*/
 
 
 
 
             printf("Voulez vous conserveez cette fiche(1=yes//0=no): ");
             scanf("%d", &rep);
 
             if(rep==1){
 
             fd=fopen("repertoire.bin", "a+");
             if(fd==NULL){
                     printf("\nError lors de l/'ouverture du fichier spécifié.!\n");
                     }
                     else{
 
                          fseek(fd, 0, SEEK_END);
                          fwrite(&fiche, sizeof(struct Enreg), 1, fd);
 
 
 
 
                          fclose(fd);
                          printf("Sauvegarde reussie!\n");
                          }
                     }
    else{
         fclose(fd);
         printf("Echec sauvegarde!\n");
         } 
 
             return;
        }
 
 
     /*programme de recherche*/
void recherche(struct Enreg fiche){
     char mot[40]={0};
 
     int rep;
 
        printf("Sous-Programme recherche dune fiche.\n");
        printf("Appuyez sur une touche...\n");
        getch();
 
        printf("\nEntrer le nom recherché: ");
        scanf("%s", &mot);
        fd=fopen("repertoire.bin", "r");
        if(fd==NULL){
                     printf("Echec lors de l'ouverture du fichier");
                     }
                     else{
 
        fseek(fd, 0, SEEK_SET);
        do{
        fread(&fiche, sizeof(struct Enreg), 1, fd);          
 
 
 
        if(strcmp(fiche.nom, mot)==0){
 
        printf("%s ", fiche.nom);
        printf(" %s\n", fiche.tel);
 
        printf("Continuer la rechercher?(0=oui/1=non): ");
        scanf("%d", &rep);
        if(rep!=0) break;
        }
 
        if(feof(fd)){
        printf("\n\nFin du fichier\nAppuyez sur une touche...\n");
        getch();}
 
        } while(!feof(fd));  
 
        fclose(fd);
        }  
        }   
/*programme de saisie des valeur*/
void saisi_data(struct Enreg *fiche){
 
    char c;
    int i;
    int cpt = 0;
 
 
    printf("Entrez le nom: ");
    fflush(stdout);
 
 
    for (i = 0; i < sizeof fiche->nom - 1 && (c = getchar()) != '\n' && c != EOF; i++)
    {
        fiche->nom[i] = toupper(c);
        cpt++;
    }
 
    printf("Entre le Numero: ");
    scanf("%s", &fiche->tel);
 
    return;
 
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
/* -tc- c doit être de type int */
int c;
 
printf("Entrez le nom: ");
fflush(stdout);
 
for (i = 0; (c = getchar()) != '\n' && c != EOF && i < sizeof fiche - 1; i++)
{
    fiche->nom[i] = toupper(c);
}
/* -tc- Il faut ajouter manuellement le caractere nul */
fiche->nom[i] = 0;
 
/* -tc- bien entendu, ne pas oublier de nettoyer le tampon du flux d'entree
    standard si necessaire */
if (c != '\n' && c != EOF)
{
    while ((c = getchar()) != '\n' && c != EOF)
    {
    }
}

Thierry

Citation:

Envoyé par Linu6

Voila le code:

Ca aiderait si le code était moins buggé :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:25: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:29: warning: missing braces around initializer
main.c:29: warning: (near initialization for `fiche.nom')
main.c:29: warning: missing initializer
main.c:29: warning: (near initialization for `fiche.tel')
main.c:61: warning: control reaches end of non-void function
main.c: In function `creer':
main.c:89: warning: implicit declaration of function `getch'
main.c: In function `recherche':
main.c:139: warning: char format, different type arg (arg 2)
main.c: In function `saisi_data':
main.c:183: warning: comparison between signed and unsigned
main.c:190: warning: char format, different type arg (arg 2)
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 10 warnings

Je conseille de mieux régler le compilateur :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

afin qu'il te dise mieux où sont les problèmes (ça de résout pas tout, mais ça fait le ménage...)

Ceci est déjà mieux :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
 
//Definition du type.
struct Enreg
{
   char nom[40];
   char tel[10];
};
 
/*Programme affichant le menu*/
static int menu (void)
{
   int i;
   do
   {
      printf ("\nMenu Principal\n");
      printf (" 1 - Creation\n");
      printf (" 2 - Recherche\n");
      printf (" 3 - Fin\n");
      printf ("Indiquez l'op choisie(1. 2. 3.): ");
      scanf ("%d", &i);
      /*Tant que la valeur ne correspond pas
         au choix proposés au menu affiche: Error! */
      if (i < 1 || i > 3)
         printf ("\nErr recommencez\n");
   }
   while (i < 1 || i > 3);
   printf ("\n");
   return (i);
}
 
/*programme de saisie des valeur*/
static void saisi_data (struct Enreg *fiche)
{
   int c;
   size_t i;
   int cpt = 0;
 
   printf ("Entrez le nom: ");
   fflush (stdout);
 
   for (i = 0;
        i < sizeof fiche->nom - 1 && (c = getchar ()) != '\n' && c != EOF;
        i++)
   {
      fiche->nom[i] = toupper (c);
      cpt++;
   }
 
   printf ("Entre le Numero: ");
   scanf ("%s", fiche->tel);
 
   return;
 
}
 
static void pause (void)
{
   printf ("Appuyez sur une touche...\n");
   int c = getchar ();
 
   while (c != '\n')
   {
      c = getchar ();
   }
}
 
/*programme permettant de creer une fiche de renseignement*/
static void creer (struct Enreg fiche)
{
   int rep = 0;                 //valeur de test
 
   printf ("Sous-Programme création d'une fiche.\n");
   pause ();
 
   saisi_data (&fiche);         /*Appel de la fonction
                                   de saisie des valeurs */
 
   printf ("Voulez vous conserveez cette fiche(1=yes//0=no): ");
   scanf ("%d", &rep);
 
   if (rep == 1)
   {
 
      FILE *fd = fopen ("repertoire.bin", "a+");
      if (fd == NULL)
      {
         printf ("\nError lors de l/'ouverture du fichier spécifié.!\n");
      }
      else
      {
 
         fseek (fd, 0, SEEK_END);
         fwrite (&fiche, sizeof (struct Enreg), 1, fd);
 
         fclose (fd);
         printf ("Sauvegarde reussie!\n");
      }
   }
   else
   {
      printf ("Echec sauvegarde!\n");
   }
 
   return;
}
 
     /*programme de recherche */
static void recherche (struct Enreg fiche)
{
   char mot[40] = { 0 };
 
   printf ("Sous-Programme recherche dune fiche.\n");
   pause ();
 
   printf ("\nEntrer le nom recherché: ");
   scanf ("%s", mot);
   {
      FILE *fd = fopen ("repertoire.bin", "r");
      if (fd == NULL)
      {
         printf ("Echec lors de l'ouverture du fichier");
      }
      else
      {
         int rep = 0;
         int n;
         do
         {
            n = fread (&fiche, sizeof (struct Enreg), 1, fd);
 
            if (strcmp (fiche.nom, mot) == 0)
            {
               printf ("%s ", fiche.nom);
               printf (" %s\n", fiche.tel);
               printf ("Continuer la rechercher?(0=oui/1=non): ");
               scanf ("%d", &rep);
            }
         }
         while (n == 1 && !rep);
 
         fclose (fd);
      }
   }
}
 
int main (void)
{
   /*déclaration et initialisation
      des varoables */
 
   struct Enreg fiche = { "", "" };
 
   int choix;
 
   /*Menu */
   do
   {
      choix = menu ();
      switch (choix)
      {
      case 1:
 
         creer (fiche);
         printf ("\n");
 
         continue;
 
      case 2:
         recherche (fiche);
 
         printf ("\n");
 
         continue;
 
      default:
         printf ("Au revoir...\n\n");
 
      }
   }
   while (choix != 3);
 
   return 0;
}

Mais il faudrait traiter les entrées correctement, soit en apprenant à utiliser correctement scanf(), soit en utilisant une autre méthode basée sur fgets().

Pose des questions si tu ne comprends pas.

11/10/2007, 19h20
Linu6

:arf:
Non ca ne marche toujours pas,.. :ouin:
le programme se compile correctement, mais lorsque le programme est lancé, l'étape de la saisie du nom est sauté, et on me demande dirrectement la confirmation pour sauvegarder la fiche.
:triste:
11/10/2007, 19h35
Emmanuel Delahaye

Citation:

Envoyé par Linu6

:arf:
Non ca ne marche toujours pas,.. :ouin:
le programme se compile correctement, mais lorsque le programme est lancé, l'étape de la saisie du nom est sauté, et on me demande dirrectement la confirmation pour sauvegarder la fiche.
:triste:

Oui, c'est normal, les saisies sont incorrectes.

des pistes :

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez.com/notes.htm#fgetc
http://emmanuel-delahaye.developpez.com/inputs.htm

etc.

Re

J'ai ( à peu près) réussi à avoir ce que je voulais, mais comme toujours il faut qu'il y ait un mais...

Dans ce code la saisie se fait caractère par caractère, et la conversion en Majuscule se fait comme il le faut. Mais le probleme c'est que je n'arrive pas à mettre fin au boucle.

Alors (pour le moment) je suis obligé de demander à l'utilisateur de saisir un '0' pour terminer la saisie, car la methode

Code:

c = getchar ()) != '\n'

ne marche pas.

voici le code:
Code:

1 2 3 4 5 6 printf("\nSaisir le nom (termine par '0'): "); for(i=0; (fiche.nom[i]=fgetc(stdin)) && fiche.nom[i]!='0'; i++){ fiche.nom[i]=toupper(fiche.nom[i]); }

Citation:
Envoyé par Linu6
Re

J'ai ( à peu près) réussi à avoir ce que je voulais, mais comme toujours il faut qu'il y ait un mais...

Dans ce code la saisie se fait caractère par caractère, et la conversion en Majuscule se fait comme il le faut. Mais le probleme c'est que je n'arrive pas à mettre fin au boucle.

Alors (pour le moment) je suis obligé de demander à l'utilisateur de saisir un '0' pour terminer la saisie, car la methode

Code:

c = getchar ()) != '\n'

ne marche pas.

voici le code:
Code:

1 2 3 4 5 6 printf("\nSaisir le nom (termine par '0'): "); for(i=0; (fiche.nom[i]=fgetc(stdin)) && fiche.nom[i]!='0'; i++){ fiche.nom[i]=toupper(fiche.nom[i]); }
Non. La marque de fin de saisie est '\n' et pas autre chose.

Le principe est expliqué dans les liens que je t'ai passé. Fait de ton mieux et poste le code qui ne fonctionne pas. Tu as essayé avec fgets() ?