Discussion :

[Freedom57]

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

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.

[Pouet_forever]

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).

[Freedom57]

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

[Cladouros]

Salut,

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

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

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

[Sve@r]

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

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

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.

[Freedom57]

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.

[diogene]

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 :

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

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

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 :

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

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