Probleme de char dans une boucle

Bonjour , voila mon 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

#include <stdio.h> #include <stdlib.h> #define Max 10   typedef struct s_point s_point; struct s_point { char c; int x,y; }; void affiche_point (s_point pt) { printf ("Point %c de coordonnées %d %d\n",pt.c,pt.x,pt.y); }   int main () { int i,n; s_point point[10]; printf("Combien de point voulez vous saisir?(<10)\n"); scanf ("%d",& n); for (i=0;i<n;i++) { printf("Entrez le nom du point\n"); scanf ("%c",& point[i].c); printf("Entrez coordonnées x\n"); scanf ("%d",& point[i].x); printf("Entrez coordonnées y\n"); scanf ("%d",& point[i].y); } for (i=0;i<n;i++) affiche_point (point[i]); return 0; }

---

Seulement à l'exécution , Je ne peux pas incrémenter les coordonnées de mes points.Je ne comprends pas pourquoi. Voila un exemple:

Citation:

---

Combien de point voulez vous saisir?(<10)
2
Entrez le nom du point
A
Entrez coordonnées x
Entrez coordonnées y
Entrez le nom du point
B
Entrez coordonnées x
Entrez coordonnées y
Point
de coordonnées -1075282924 134513148
Point A de coordonnées -1216849324 0

---

Merci de m'aiguiller.

Pense à vider le buffer de stdin après chaque scanf. Je te renvoi à la FAQ pour plus d'infos (http://c.developpez.com/faq/?page=cl...buffer_clavier).

Bonjour , merci pour cette réponse rapide.J'ai bien effectué les changements que tu m'as conseillés cependant , ça ne marche pas mieux , differement , mais pas mieux.Donc bon , je comprends toujours pas pourquoi ça marche ne pas...

Salut,

En vidant le buffer après chaque scanf comme dans le code ci-dessous, cela fonctionne très bien ;)

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

#include <stdio.h> #include <stdlib.h> #define Max 10   typedef struct s_point s_point;   struct s_point { char c; int x,y; };   void affiche_point (s_point pt) { printf ("Point %c de coordonnées %d %d\n",pt.c,pt.x,pt.y); }   int main () { int i,n; s_point point[10]; printf("Combien de point voulez vous saisir?(<10)\n"); scanf ("%d",& n); while(getchar() != '\n'); // Vidage du buffer   for (i=0;i<n;i++) { printf("Entrez le nom du point\n"); scanf ("%c", &point[i].c); while(getchar() != '\n'); // Vidage du buffer printf("Entrez coordonnées x\n"); scanf ("%d", &point[i].x); while(getchar() != '\n'); // Vidage du buffer printf("Entrez coordonnées y\n"); scanf ("%d", &point[i].y); while(getchar() != '\n'); // Vidage du buffer }   for (i=0;i<n;i++) affiche_point (point[i]); return 0; }

---

Résultat :

Citation:

---

Combien de point voulez vous saisir?(<10)
2
Entrez le nom du point
A
Entrez coordonnées x
2
Entrez coordonnées y
4
Entrez le nom du point
B
Entrez coordonnées x
5
Entrez coordonnées y
6
Point A de coordonnées 2 4
Point B de coordonnées 5 6

---

Les entrées à partir du clavier ne sont pas simples.
Pour mieux comprendre, il faut suivre le contenu du buffer clavier. Ci-dessous figure l'état du programme à différentes étapes en montrant - le scanf() à exécuter, -le contenu de la console (affichage des messages (simplifiés) en italiques et entrées de l'utilisateur), -l'état du buffer clavier après l'entrée au clavier et avant la conversion par scanf(), -le résultat de la conversion et -l'état du buffer après celle-ci.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

Code affichage/entrée buffer Après conversion clavier résultat buffer clavier Combien de points .... scanf("%d",& n) 2<entrée> '2','\n' n<-2 '\n' Entrez .... point scanf("%c",& point[0].c); '\n' point[0].c<-'\n' vide Entrez .... x scanf("%d",& point[0].x); A<entrée> 'A','\n' ECHEC 'A','\n' Entrez .... y scanf("%d",& point[0].y); 'A','\n' ECHEC 'A','\n' Entrez .... point scanf("%c",& point[1].c); 'A','\n' point[1].c<- 'A' '\n' Entrez .... x scanf("%d",& point[1].x); 1<entrée> '\n','1','\n' point[1].x <- 1 '\n' Entrez .... y scanf("%d",& point[1].y); 2<entrée> '\n','2','\n' point[1].y <- 2 '\n' Point de coordonnées ... (avec 2 valeurs bizarres) Point A de coordonnées 1 2

---

Le problème se situe dans le scanf("%c",...) : il faut que le buffer soit vide avant de lire ce caractère. Or cette lecture est toujours précédée par un scanf("%d",...) qui laisse dans le buffer ce qui a été tapé après le nombre. Et il y a alors toujours au moins le caractère de fin de ligne.
Il est donc nécessaire d'avoir un buffer vide avant le scanf("%c",...).

Dans le cas des scanf("%d",...), le problème ne se pose pas de la même façon car la conversion ignore les caractères blancs qui précèdent le nombre et le '\n' fait partie de ces caractères blancs. Encore faut-il que ce soit bien un nombre, et uniquement un nombre, qui soit entré à la console. Si on veut éviter les erreurs dues à des entrées erronées des nombres ou des nombres suivis par autre chose que des blancs, il faut également vider le buffer après leur lecture pour éliminer les caractères intempestifs et tester la valeur de retour de scanf() pour savoir combien de conversion ont réussi.

Pour s'assurer d'avoir un buffer vidé après chaque lecture, on peut utiliser la méthode de la FAQ, ou compléter le format du scanf() pour le faire :

Code:

---

1 2	scanf ("%d%*[^\n]%*c",....); scanf ("%c%*[^\n]%*c",....);

---

Ceci est toutefois insuffisant pour assurer un bon fonctionnement en cas d'entrée erronée (non numérique) pour les scanf("%d",...). Pour se garantir, on peut utiliser pour chaque entrée en "%d" un système du genre

Code:

---

1 2 3 4 5 6

do { printf("Combien de point voulez vous saisir?(<10)\n"); // message de saisie res = scanf ("%d",& n); // lire un entier scanf ("%*[^\n]%*c"); // vider le buffer clavier (ou utiliser fonction de la FAQ) }while(res !=1); // tant qu'un entier n'a pas été lu

---

code qui peut avantageusement faire l'objet d'une fonction : int LireEntier(char* message);

Citation:

---

Envoyé par Cladouros

Salut,

En vidant le buffer après chaque scanf comme dans le code ci-dessous, cela fonctionne très bien ;)

---

Salut

Le vrai problème provient de scanf lui-même. Il faut bien comprendre que scanf pose 2 soucis
1) il attend une entrée formatée. Or, ce que tape l'utilisateur peut-être n'importe quoi
2) dès qu'il ne comprend plus ce qu'il lit, alors il s'arrête de lire et laisse ce qu'il n'a pas lu dans le buffer stdin qui est alors traité lors du scanf() suivant

Diogène propose une solution pour régler le souci. Comme il n'y a jamais qu'une seule solution à un problème, je propose la mienne
1) commencer par lire tout le clavier dans un gros buffer texte en utilisant fgets(). Ainsi le clavier reste clean
2) traiter le buffer texte via sscanf().
3) si le nombre d'entrées récupéré par sscanf ne correspond pas au nombre attendu, alors recommencer en 1

Ce qui peut se traduire par

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

char tampon[80 + 1]; // +1 pour le '\0' int nbr;   while (1) { printf("Entrez un nombre entier\n"); fgets(tampon, 80 + 1, stdin); // +1 car fgets enlève 1   if (sscanf(tampon, "%d", &nbr) == 1) break;   printf("Erreur de saisie - Recommencez\n"); }

---

Et cette méthode s'adapte aussi bien aux doubles.

Ok merci pout toutes ces réponses , pas simple tout ça...Enfin bon je crois avoir compris l'essentiel.Il ne me reste plus qua à mettre en pratique...
Merci.