Discussion :

[v4np13]

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 :

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

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.

[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

[v4np13]

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

Merci

[Franck.H]

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

[Emmanuel Delahaye]

Code :

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

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.

[v4np13]

Merci pout tes explication Franck.H, on ne pouvait faire plus simple

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