Problème avec un switch !

Version imprimable

Voici le bout de code qui pose problème :

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
		codeLu = getchar();
		while(codeLu != '0')
		{
			switch(codeLu)
			{
			case '1':
				{
					printf("\nRentrez le nouveau prenom : ");
					fflush(stdin);
					gets(prenomLu);
					for(k=0; k<j; k=k+1)
					{
						if((strcmp(Tab[i].nom, Tab[k].nom) == 0))
						{
							if((strcmp(Tab[k].prenom, prenomLu) == 0))
								break;
						}
					}
 
					if(k<j)
						printf("\nModification rejetee : la personne existe deja.\n");
					else
						strcpy(Tab[i].prenom, prenomLu);
					printf("\nSi vous souhaitez effectuer une autre modification, rentrez 1, 2 ou 3, sinon rentrez 0 pour quitter.\n");
					codeLu = getchar();
					if(codeLu == '0')
						printf("\nAffichage des nouvelles informations concernant la personne :\n\n\t%-25s %-25s %-15s %-10s\n\n", Tab[i].nom, Tab[i].prenom, Tab[i].tel, Tab[i].codeCat);
					break;
				}
			case '2':
				{
					printf("\nRentrez le nouveau telephone : ");
					fflush(stdin);
					gets(telLu);
					strcpy(Tab[i].tel, telLu);
					printf("\nSi vous souhaitez effectuer une autre modification, rentrez 1, 2 ou 3, sinon rentrez 0 pour quitter.\n");
					codeLu = getchar();
					if(codeLu == '0')
						printf("\nAffichage des nouvelles informations concernant la personne :\n\n\t%-25s %-25s %-15s %-10s\n\n", Tab[i].nom, Tab[i].prenom, Tab[i].tel, Tab[i].codeCat);
					break;
				}
			case '3':
				{
					printf("\nRentrez la nouvelle categorie : ");
					fflush(stdin);
					gets(codeCatLu);
					strcpy(Tab[i].codeCat, codeCatLu);
					printf("\nSi vous souhaitez effectuer une autre modification, rentrez 1, 2 ou 3, sinon rentrez 0 pour quitter.\n\n");
					codeLu = getchar();
					if(codeLu == '0')
						printf("\nAffichage des nouvelles informations concernant la personne :\n\n\t%-25s %-25s %-15s %-10s\n\n", Tab[i].nom, Tab[i].prenom, Tab[i].tel, Tab[i].codeCat);
					break;
				}
			default:
				{
					printf("\nCode errone. Veuillez saisir a nouveau le code SVP.\n\n");
					codeLu = getchar();
					if(codeLu == '0')
						printf("\nAffichage des nouvelles informations concernant la personne :\n\n\t%-25s %-25s %-15s %-10s\n\n", Tab[i].nom, Tab[i].prenom, Tab[i].tel, Tab[i].codeCat);
					break;
				}
			}
}

Et voici ce qui s'affiche si l'on rentre un code ki rentre dans le default (ici le code rentré est 9) :

9

Code errone. Veuillez saisir a nouveau le code SVP.

Code errone. Veuillez saisir a nouveau le code SVP.

Je ne vois pas pourquoi le message apparait 2 fois...

Salut,

getchar() lit le premier caractère sur stdin, mais ne supprime pas les caractères restants. Ainsi, au second appel de getchar(), c'est '\n' qui est lu avant que l'utilisateur puisse faire quoi que ce soit. Je recommande donc de penser à vider stdin après appel de getchar :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include <stdio.h> int lire_car (void) { int c = getchar(); if(c != '\n' && c != EOF) { int c; while((c = getchar() != '\n') && c != EOF) { } } return c; }

30/10/2007, 18h48
Emmanuel Delahaye
Citation:
Envoyé par Dark_Alex69

Voici le bout de code qui pose problème :
<...>
Et voici ce qui s'affiche si l'on rentre un code ki rentre dans le default (ici le code rentré est 9) :

Code:

1 2 3 4 5 6 7 9 Code errone. Veuillez saisir a nouveau le code SVP. Code errone. Veuillez saisir a nouveau le code SVP.

Je ne vois pas pourquoi le message apparait 2 fois...
Problème récurrent. Je te conseille de revoir complètement tes saisies.

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

je ne veux plus jamais voir de gets()...
30/10/2007, 19h19
Dark_Alex69

Pour le moment on n'a appris que gets() pour recuperer des string.
30/10/2007, 19h28
Dark_Alex69

Pour le message de odsen.s, je n'ai pa trop compris ton code...

Citation:

Envoyé par Dark_Alex69

Pour le moment on n'a appris que gets() pour recuperer des string.

Berk...
Où ça ?

Citation:

Envoyé par Dark_Alex69

Pour le message de odsen.s, je n'ai pa trop compris ton code...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
#include <stdio.h>
 
/* fonction retournant le premier caractere lu
sur stdin, supprime tous les autres caracteres */
int lire_car (void)
{
   /* lecture du premier caractere */
   int c = getchar();
 
   /* si il y a d'autres caracteres dans le flux*/
   if(c != '\n' && c != EOF)
   {
       int c;
       /* on appelle getchar tant qu'il reste des caracteres a lire */
       while((c = getchar() != '\n') && c != EOF)
       {
       }
   }
   return c;
}

30/10/2007, 19h42
Dark_Alex69

Merci :) je vais essayer ta solution.
30/10/2007, 19h44
odsen.s

Je te conseille aussi de visiter les liens donnés par Emmanuel pour apprendre à faire des saisies correctes.
30/10/2007, 20h06
Dark_Alex69

Oui, je suis allé voir. Je vais essayer de me familiariser avec les fonctions fgets et compagnie.
Concernant mon problème, j'ai juste rajouté un fflush(stdin) avant le printf dans le default et le problème a disparu... Merci pour vos réponses !!!
30/10/2007, 20h17
Emmanuel Delahaye

Citation:

Envoyé par Dark_Alex69

j'ai juste rajouté un fflush(stdin) avant le printf dans le default et le problème a disparu...

Ce n'est certainement pas sur mon site que tu as trouvé ce bricolage. fflush() n'est défini que pour les flux sortants. Le comportement est indéfini sur les flux entrants.

Je peux déjà t'affirmer que c'est au mieux sans effet et au pire catastrophique, sous unixoïde, même si ça parait fonctionner sous Windows. Ce n'est donc pas une solution.
30/10/2007, 20h17
diogene

Citation:

Concernant mon problème, j'ai juste rajouté un fflush(stdin) avant le printf

Le problème est que fflush doit avoir pour argument un flux de SORTIE, pas d'entrée. Le comportement avec un flux d'entrée est dépendant de l'implantation. Ca marche, je crois, comme tu l'espères avec des systèmes type MSwindows, mais pas avec tous les unixoïde. Donc ta solution n'est pas définitive et tu dois toi-même vider le buffer d'entrée.
30/10/2007, 20h37
Dark_Alex69

Oui vous avez surement raison. Cependant, pour le moment on ne fait que du MSDOS. Je changerais de façon de faire quand on fera du UNIX...:lol:
30/10/2007, 21h05
odsen.s

Citation:

Envoyé par Dark_Alex69

Oui vous avez surement raison. Cependant, pour le moment on ne fait que du MSDOS. Je changerais de façon de faire quand on fera du UNIX...:lol:

Il vaut mieux apprendre à faire des choses correctes de façon portable, pas du bricolage...
30/10/2007, 21h27
Sve@r

Citation:

Envoyé par Dark_Alex69

Oui, je suis allé voir. Je vais essayer de me familiariser avec les fonctions fgets et compagnie.

fgets(), au contraire de gets(), permet de faire une saisie limitée (si tu dis
qu'il faut saisir 10 car., tu pourras pas en entrer 12)...

Sinon le flush(stdin) est une mauvaise idée. Il dénote un problème assez profond qui n'est pas réglé (juste masqué)...
30/10/2007, 21h49
ram-0000

Ha que c'est beau toutes ces structures de contrôle imbriquées

Celui qui a inventé les fonctions était quand même pas idiot :roll:
31/10/2007, 01h31
Haze.

Hmm, c'est marrant, dans mon IUT info, on nous a filé un projet à réaliser et il ressemble furieusement à ton morceau de code source :)
Chez nous aussi les gets() et les fflush(stdin) sont enseignés ... :?
31/10/2007, 02h33
souviron34

Pourtant c'est clair :

K&R :

Citation:

int fflush(FILE *stream)
On an output stream, fflush causes any buffered but unwritten data to be written; on an input stream, the effect is undefined. It returns EOF for a write error, and zero otherwise. fflush(NULL) flushes all output streams.

Norme C99 (http://www.open-std.org/JTC1/SC22/WG...docs/n1124.pdf)

Citation:

7.19.5.2 The fflush function
Synopsis
1 #include <stdio.h>
int fflush(FILE *stream);
Description
2 If stream points to an output stream or an update stream in which the most recent operation was not input, the fflush function causes any unwritten data for that stream to be delivered to the host environment to be written to the file; otherwise, the behavior is undefined.
3 If stream is a null pointer, the fflush function performs this flushing action on all streams for which the behavior is defined above.
Returns
4 The fflush function sets the error indicator for the stream and returns EOF if a write error occurs, otherwise it returns zero.

:aie:

Faites les venir par ici, vos profs.....