do while + test de caractère

Version imprimable

23/11/2006, 16h22
v4np13

do while + test de caractère
Bonjour,
j'ai un petit problème avec mon programme, il m'affiche deux fois la même ligne "sans raison", le copie colle une partie du code:
Code:

1 2 3 4 5 6 7 8 9 10 11 char lettre; while (....){ ... do { lettre='\0'; printf("Entrez une lettre : "); scanf("%c",&lettre); } while ((lettre<97)||(lettre>122)); .... }
Les valeurs 97 et 122 se référent aux lettres a et z en ASCII.

La première fois, il me l'affiche une fois "Entrez une lettre : " et les fois suivantes, il l'affiche 2 fois "Entrez une lettre : Entrez une lettre : ".

Je ne comprends pas pourquoi il fait cela, pourriez-vous m'expliquer? Comment pourrais-je faire pour qu'il l'affiche une seule fois à chaque fois.

Merci d'avance.
23/11/2006, 16h45
Franck.H

Il faut vider le buffer du clavier, il te reste un caractère parasite, soit le retour avec la touche Entrée ... voir dans la FAQ: Comment vider le buffer clavier ?

Tu mets ca juste paès ton scanf et tout devrais rentrer dans l'ordre ;)
23/11/2006, 16h53
v4np13

C'est quand même bizarre ça, mais ça fonctionne :)

Merci
23/11/2006, 17h08
Franck.H

Citation:

Envoyé par v4np13

C'est quand même bizarre ça, mais ça fonctionne :)

Merci

Il n'y a rien de bizarre ... quand tu tapes sur Entrée, ca envoie un caractère dans le tampon, chose tout à fait normale mais vu que tu ne lis qu'un seul caractère dans ta fonction hé bin il reste le second qui t'as servi pour valider ton choix ce qui fait que lorsque le programme de repose la question hé bien il lit la suite du tampon c'est tout ;)
23/11/2006, 17h38
Emmanuel Delahaye
Citation:
Envoyé par v4np13

Code:

1 2 3 4 5 6 7 8 9 10 11 char lettre; while (....){ ... do { lettre='\0'; printf("Entrez une lettre : "); scanf("%c",&lettre); } while ((lettre<97)||(lettre>122)); .... }
Ton code est horrible pour plusieurs raisons :
- Indentation innommable.
- Sortie incomplète sur stdout (pas de '\n') Il manque un fflush (stdout);
- Usage non maitrisé de scanf() entrainant une instabilité du code.
- Code inutilement compliqué. (lettre = getchar() aurait fait la même chose, mais avec le même défaut).
- Test des caractères non portable. Il faut utiliser les fonctions de <ctype.h>.
- Pas de traitement en cas d'erreur.
23/11/2006, 19h31
v4np13
Merci pout tes explication Franck.H, on ne pouvait faire plus simple :)
Citation:
Envoyé par Emmanuel Delahaye

Ton code est horrible pour plusieurs raisons :

Indentation innommable.
Sortie incomplète sur stdout (pas de '\n') Il manque un fflush (stdout);
Usage non maitrisé de scanf() entrainant une instabilité du code.
Code inutilement compliqué. (lettre = getchar() aurait fait la même chose, mais avec le même défaut).
Test des caractères non portable. Il faut utiliser les fonctions de <ctype.h>.
Pas de traitement en cas d'erreur.
Molo molo :D Je débute, laisse moi le temps de m'y retrouver ;) Je n'ai pas 5-20 ans ou plus de C dans la peau comme toi, seulement 1 mois.

L'indentation, c'est normal, j'ai coupé le code qui me paraissait indispensable pour illustrer mon problème, pas besoin de vous encombrez avec les 100 autres lignes qui fonctionnent. Mais j'indente dans mes sources, pas de soucis de ce côté là.

Le deuxième point, j'ai pas trop compris. J'ai pas mis le \n parce que je ne voulais pas aller à la ligne. fflush je ne connais pas mais je regarderai dans les manuels.

Getchar, je ne connaissais pas non plus. Je suis vraiment dans les débuts, le reste va venir.

Je vais regarder ce que offre la libraire ctype.h.

Le traitement est dans les lignes de code non-publiées ;)

Merci à vous deux. :)