test de la validité d'un caractere

Bonjour, le probleme suivant n'en ai pas un dans l'absolu puisque j'ai des alternatives, cependant j'aimerais arriver à faire ce que j'avais initialement prévu :
Code:

1 2 3 4 5 6 7 8 do //test si l'on a bien saisit un 'o' ou un 'n' { printf("Voulez-vous arreter la saisie ? (saisir 'o' ou 'n') ");//affichage question fflush(stdin); scanf("%c",&rep); //saisie reponse //printf("drapeau");//inutile au prog final } while(rep!='o'&&rep!='n');
PB : LA QUESTION ("Voulez-vous arreter la saisie ?") EST POSEE 2 FOIS, POURQUOI ?
idees de reponses : le fflush ? on ne teste pas un caractere ainsi ?

Il faut savoir que lorsque l'on remplace les caracteres "rep" par une chaine de caractere "rep[0]", la question n'est posée qu'une fois comme voulu.

LE PROG DANS SON INTEGRALITE, ça sert à calculer des moyennes de notes (exo cours):

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
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
#include <stdio.h> 
#include <stdlib.h> 
//constantes: 
#define NB_NOTES_MAX 100 
#define NB_ELEVES_MAX 100 
int main(int argc, char *argv[]) 
{ 
  //variables :
  int n; //designe le n-ième élève
  int m; //curseur pour designer l'eleve n-1 
  int i; //curseur pour designer la note i de l'élève n-1 
  float notes[NB_ELEVES_MAX][NB_NOTES_MAX];//matrice regroupant les notes de chaque élève
  float cumul; //somme des notes (pour un élève)
  int nb_notes; 
  int nb_eleves; 
  char rep;  
 
  //demande, saisie, affichage du nombre d'élèves : 
  printf("nombre d eleves :"); 
  scanf("%u",&nb_eleves);   
  printf("vous avez %u eleves\n\n",nb_eleves);   
 
  //saisie des notes des n élèves
  for(m=0;m<nb_eleves;m++) 
  { 
  n=m+1;   
  cumul=0; //somme des notes raz 
        //saisie des notes de l'élève n 
        for (i=1;i<=NB_NOTES_MAX;i++) 
        {
                 //pose la question après la saisie d'une note 
                 if(i!=1) //pose la question après la saisie d'une note
                 {
                          printf("Voulez-vous arreter la saisie ? (saisir 'o' ou 'n') ");//affichage question
                          do  //test si l'on a bien saisit un 'o' ou un 'n'
                          {
                          fflush(stdin);
                          scanf("%c",&rep);                             //saisie reponse
                          //printf("drapeau");  //essayez avec ce drapeau vous serez peut-etre surpris,  
                                                          //en cherchant à resoudre ce problème  
                                                          //j'ai compris la complexité de l'utilisation des caracteres
                                                          //j'avais d'autres alternatives comme utiliser une chaine de caractere au lieu d'un caractere, ça fonctionne !
                          }
                          while(rep!='o'&&rep!='n');
                 }
                 //fin if
 
                //l'utilisateur a saisit au moins une note et veux arreter  
                if(i!=1&&rep!='n'){nb_notes=i-1;i=NB_NOTES_MAX;}
 
                //sinon saisie de la note i de l'élève n
                else  
                {                                                                          
 
                printf("saisir note %u de l eleve %u: ",i,n);
 
                scanf("%f",&notes[m][i]);
 
                cumul=cumul+notes[m][i]; //calcul de la nouvelle somme des i notes de l'élève n
 
                }
 
       }   //fin du cycle de saisie des i notes de l'élève n 
 
       //calcul de la moyenne des i notes de l'élève n 
       notes[m][0]=cumul/nb_notes; 
       printf("\n"); //saut de ligne après saisie des notes d'un élève 
 
  } //fin du cycle de saisie des notes des n élèves   
 
  // affichage des moyennes 
  printf("Les moyennes sont :\n"); 
  for (m=0;m<nb_eleves;m++) 
  { 
      n=m+1; 
      printf("eleve %u     m = %.2f\n",n,notes[m][0]); 
  }
   system("PAUSE"); 
   return 0; 
}

Je ne vois rien de choquant dans la boucle... (à part que fflush(stdin) ne marche que sous Windows)...

Re: test de la validité d'un caractere

Citation:
Envoyé par pimousse92
Code:

1 2 3 4 5 6 7 8 9 do //test si l'on a bien saisit un 'o' ou un 'n' { printf("Voulez-vous arreter la saisie ? (saisir 'o' ou 'n') ");//affichage question fflush(stdin); scanf("%c",&rep); //saisie reponse //printf("drapeau");//inutile au prog final } while(rep!='o'&&rep!='n');
PB : LA QUESTION ("Voulez-vous arreter la saisie ?") EST POSEE 2 FOIS, POURQUOI ?
idees de reponses : le fflush ? on ne teste pas un caractere ainsi ?

Il faut savoir que lorsque l'on remplace les caracteres "rep" par une chaine de caractere "rep[0]", la question n'est posée qu'une fois comme voulu.
scanf() n'est pas la bonne fonction pour faire des saisies.

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

fflush() n'est défini que pour les flux sortants.

Citation:
Envoyé par pimousse92
Code:

1 2 3 4 5 6 #define NB_NOTES_MAX 100 #define NB_ELEVES_MAX 100 int main(int argc, char *argv[]) { //variables : float notes[NB_ELEVES_MAX][NB_NOTES_MAX];//matrice regroupant les notes de chaque élève
10 000 float en mémoire auto, t'y vas fort...

En embarqué, ca te pète à la tronche, point.

On est bien d'accord, l'algorithme n'est pas choquant... en revanche son execution l'est plus. L'avez vous testé ?
J'essayerai avec autre chose que scanf pour la saisie.

et sans le fflush, ça donne quoi ? (je ne suis pas chez moi, je n'ai pas accès à un compilateur).

sans le fflush ça donne la meme chose.

Je précise que j'ai mes tp sous windows avec devcpp et que je travail chez moi avec le gcc de linux (ubuntu)

Citation:
Envoyé par Emmanuel Delahaye
Citation:
Envoyé par pimousse92
Code:

1 2 3 4 5 6 #define NB_NOTES_MAX 100 #define NB_ELEVES_MAX 100 int main(int argc, char *argv[]) { //variables : float notes[NB_ELEVES_MAX][NB_NOTES_MAX];//matrice regroupant les notes de chaque élève
10 000 float en mémoire auto, t'y vas fort...

En embarqué, ca te pète à la tronche, point.
Je n'ai pas été sensibilisé au problème.

Citation:

Envoyé par pimousse92

sans le fflush ça donne la meme chose.

Je précise que j'ai mes tp sous windows avec devcpp et que je travail chez moi avec le gcc de linux (ubuntu)

Très bien. Comme ça , la portabilité n'est pas une option.

Encore mieux, PC/Windows d'un coté et Mac/OsX de l'autre, avec des échanges de données par fichiers. Ca forme la jeunesse. Les centres de formations devraient être équipés de matériels hétérogènes en réseau...

je proposerais un simple :

Code:

c=toupper(getch());

Scanf est "deprecated" !!!

Citation:

Envoyé par Emmanuel Delahaye

Encore mieux, PC/Windows d'un coté et Mac/OsX de l'autre, avec des échanges de données par fichiers.

Le bonheur, surtout lorsque l'on oublie que les retours chariot ne sont pas les mêmes; après quelques jours passé à tourner en rond on oublie plus.

Citation:

Envoyé par issou

Scanf est "deprecated" !!!

8O scanf obsolète, il va être l'heure d'allez me coucher!
scanf est juste difficile à maitriser et par conséquence déconseillé aux débutants.

Citation:

Envoyé par issou

je proposerais un simple :

Code:

c=toupper(getch());

Scanf est "deprecated" !!!

Gné ? D'usage difficile, peu recommandée, mais pas deprecated (contrairement à ato*())

Par contre, getch() n'est pas standard, donc non portable et non recommandé.

Les solutions sont connues :

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

Je viens d'imprimer les "notes" d'Emmanuel, je n'ai pas encore bien saisi l'équivalent à utiliser de scanf mais ça va venir...

Citation:

Envoyé par pimousse92

Je viens d'imprimer les "notes" d'Emmanuel, je n'ai pas encore bien saisi l'équivalent à utiliser de scanf mais ça va venir...

Service minimum : fgets() + sscanf()...

J'ai trouvé une solution à mon problème :
bien que scanf ne soit pas souhaitable pour une saisie au clavier, mon algo est censé marcher.
mis à part que mon fflush(stdin); se comporte mal.
j'utilise donc pour vider le flux une methode décrite dans la faq :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 char saisir_un_caractere(void) { char car; int x; printf("%s saisi une lettre : ", joueur_devine); while ((x = getchar()) != '\n' && x != EOF); //cf le commentaire ci-dessous //fflush(stdin); //ne fonctionne pas sur mon ordinateur avec le compilateur gcc de linux scanf("%c",&car); return car; }
Citation:

La méthode la plus sûre pour vider le buffer clavier consiste à consommer tout les caractères présents dans ce buffer jusqu'à ce qu'il soit vide :

#include <stdio.h> int c: while ((c = getchar()) != '\n' && c != EOF); // mange le tampon

N.B. : certains compilateurs offre une extension proposant une méthode 'simple' pour vider ce buffer clavier en utilisant la fonction fflush.

fflush(stdin);

Toutefois la norme n'indiquant le fonctionnement de fflush que sur des flux sortant, cette méthode n'est pas standard et peut conduire à des résultats indéfinis, il convient donc de l'éviter.

le probleme venait donc du fflush sous linux

Citation:
Envoyé par pimousse92
Code:

1 2 //fflush(stdin); //ne fonctionne pas sur mon ordinateur avec le compilateur gcc de linux
Citation:

le probleme venait donc du fflush sous linux
Bien evidemment.
fflush n'est defini que pour les flux sortants, et stdin n'est pas sortant mais entrant.

comment expliques tu que fflush(stdin) fonctionne sous devcpp ? ce n'est pas si evident quand on a pas l'info, la preuve cela n'a sauté aux yeux de personne...

Citation:

Envoyé par pimousse92

ce n'est pas si evident quand on a pas l'info

Voir ci après...

Citation:

Envoyé par pimousse92

le probleme venait donc du fflush sous linux

Non. Le problème vient du fait que vous avez supposé que fflush() faisait quelque-chose sans lire le man. Avant d'accuser fflush(), lisez correctement :

Citation:

Envoyé par man fflush

FFLUSH(3) Manuel du programmeur Linux FFLUSH(3)

NOM
fflush - Vider les buffers d'un flux.

SYNOPSIS
#include <stdio.h>

int fflush (FILE *flux);

DESCRIPTION
La fonction fflush force l'écriture de toutes les données se trouvant dans les buffers de l'espace utilisateur, et met
à jour le flux à travers la fonction sous-jacente d'écriture. Le statut d'ouverture du flux n'est pas affecté.

Si l'argument flux est NULL, fflush vide tous les flux en sortie.

VALEUR RENVOYÉE
Si elle réussit intégralement, cette fonctions renvoie 0. Sinon, elle renvoie EOF, et la variable errno contient le
code d'erreur.

ERREURS
EBADF flux n'est pas ouvert, ou du moins pas en écriture.

Est-ce que "stdin" est un flux en sortie ouvert en écriture ? Jusqu'à preuve du contraire, non.

doucement, je débute.
j'admet mon erreur néanmoins j'ai toujours du mal à comprendre car j'ai bien vu que fflush pouvait fonctionner d'après la faq dans certaines conditions et je me rapelle bien l'avoir utiliser sous devcpp avant la saisie d'un caractere avec un stdin.
je ne comprend pa bien man.
comment d'après man utiliser fflush dans mon algo ?

Ben il fonctionne sous DevCpp car DevCpp c'est sous Windows et fflush(stdin) fonctionne sous Windows ???!!!

Le man est pourtant bien explicite (voir les phrases mises en gras par David Schris)

Citation:

Envoyé par pimousse92

doucement, je débute.

Rien de méchant dans mon commentaire, juste un rappel.
Moi non plus je ne lisais pas (assez) la doc. à mes débuts...et puis j'en ai eu marre de ne pas comprendre pourqoi ceci ou pourquoi cela...

Citation:

Envoyé par pimousse92

j'admet mon erreur néanmoins j'ai toujours du mal à comprendre car j'ai bien vu que fflush pouvait fonctionner d'après la faq dans certaines conditions

Pour ce qui est de la FAQ, je ne sais pas.

Citation:

Envoyé par pimousse92

et je me rapelle bien l'avoir utiliser sous devcpp avant la saisie d'un caractere avec un stdin.

Sauf que Dev-C++ c'est sous Windows et qu'un terminal sous Linux n'est peut-être (<-et c'est peu dire) pas géré de la même façon que sous Windows.
Donc, une fonction de plus haut niveau comme fflush() n'est probablement pas implémentée de la même manière.

Citation:

Envoyé par pimousse92

je ne comprend pa bien man.
comment d'après man utiliser fflush dans mon algo ?

D'après le man tu ne peux pas puisqu'il dit que c'est pour les flux ouverts en écriture.

aprés ces précisions, est-ce que je peux "accuser" mon utilisation de fflush sous linux ? :wink:

De toutes façons, tu peux accuser qui tu veux : ça n'a jamais fait mieux fonctionner un programme :wink:

Citation:

Envoyé par pimousse92

aprés ces précisions, est-ce que je peux "accuser" mon utilisation de fflush sous linux ? :wink:

Oui en effet c'est a cause de ca que ca ne fonctionnais pas.
Je reprends ton post initial.
Ton message ete affiché 2 fois car pour le scanf(%c", &rep); lorsque tu appui sur entrée pour confirmer la saisie le \n et mis dans le tampon et comme ton fflush ne fonctionnait pas lors du 2eme scanf rep prenait la valeur dans le tampon (\n).
En gros tu avais un scanf sur 2

fflush(stdin) est garanti marcher sous Windows, mais ce n'est pas défini par le standard et par conséquent, il y a des OS où ce n'est pas implémenté (par exemple, pratiquement (ou absolument) tous les unixoïdes...)

Oui mais non

Vu qu on peut vider le buffer d'entree avec une simple boucle, il est plus propre de na pas utiliser fflush(stdin) du tout. Meme si ca marche.
(En plus comme ca c est portable :wink: )

Citation:

Envoyé par pimousse92

comment expliques tu que fflush(stdin) fonctionne sous devcpp ? ce n'est pas si evident quand on a pas l'info, la preuve cela n'a sauté aux yeux de personne...

C'est une extension MS-DOS Windows. Comme c'est pas portable, vu du C, ça n'existe pas, et il donc recommandé de ne pas pas l'utiliser, surtout que des solutions alternatives existent, à commencer par fgets() qui résout définitivement les problèmes de caractères non lus.

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

Citation:

Envoyé par Emmanuel Delahaye

Citation:

Envoyé par pimousse92

comment expliques tu que fflush(stdin) fonctionne sous devcpp ? ce n'est pas si evident quand on a pas l'info, la preuve cela n'a sauté aux yeux de personne...

C'est une extension MS-DOS Windows. Comme c'est pas portable, vu du C, ça n'existe pas, et il donc recommandé de ne pas pas l'utiliser, surtout que des solutions alternatives existent, à commencer par fgets() qui résout définitivement les problèmes de caractères non lus.

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

et pour filtrer des lettres est-ce qu'il suffit de remplacer

Code:

ret = sscanf (saisie, "%[0-9-]s", temp);

par

Code:

ret = sscanf (saisie, "%[a-z-]s", temp);

question préliminaire : j'utilises getchar pour faire une pause, est-ce correct ?
la question sur les flux :
j'utilises

Code:

while ((x = getchar()) != '\n' && x != EOF);

plutot que

Code:

fflush(stdin);

pour vider le flux avant un getchar.
(sans ça le prog ne fais pas de pause lorsque l'on utise getchar pour une deuxieme fois)
jusque là tout va bien sauf pour le premier getchar où je dois taper 2 fois sur entrée plutot qu'une... comment se fait-il ?

lien vers mon prog (voir le sous programme une_pause();
http://pastebin.com/429647

oui, c'est correct d'utiliser getchar()

fflush(stdin) n'est pas standard, ça ne marche pratiquement que sous windows

Citation:

Envoyé par pimousse92

(...)
j'utilises

Code:

while ((x = getchar()) != '\n' && x != EOF);

plutot que

Code:

fflush(stdin);

pour vider le flux avant un getchar.
(sans ça le prog ne fais pas de pause lorsque l'on utise getchar pour une deuxieme fois)
jusque là tout va bien sauf pour le premier getchar où je dois taper 2 fois sur entrée plutot qu'une... comment se fait-il ?

lien vers mon prog (voir le sous programme une_pause();
http://pastebin.com/429647

pour ne pas perdre le fil...

Citation:

Envoyé par pimousse92

et pour filtrer des lettres est-ce qu'il suffit de remplacer

Code:

ret = sscanf (saisie, "%[0-9-]s", temp);

par

Code:

ret = sscanf (saisie, "%[a-z-]s", temp);

C'est possible, mais je ne connais pas assez bien scanf() pour répondre avec certitude.

Citation:

Envoyé par pimousse92

question préliminaire : j'utilises getchar pour faire une pause, est-ce correct ?

Ca dépend de ce qu'il y a eu avant. Avec fgets(), c'est facilement prévisible et mesurable, avec scanf(), je ne sais pas faire. Je préconise d'utiliser exclusivement fgets() pour les saisies et donc d'avoir un conportement parfaitement et simplement prévisible.

Citation:

la question sur les flux :
j'utilises

Code:

while ((x = getchar()) != '\n' && x != EOF);

Ca, ça sert à nettoyer le buffer stdin si on est sûr qu'il y a encore des restes... On s'en sert par exemple après un fgets() si on a pas trouvé le '\n'.

Citation:

plutot que

Code:

fflush(stdin);

Comme on se tue à te l'expliquer, ceci n'est pas défini par le langage C. Qu'il existe une extension non portable n'en fait évidemment pas une solution universelle. Formellement déconseillé.

Citation:

pour vider le flux avant un getchar.
(sans ça le prog ne fais pas de pause lorsque l'on utise getchar pour une deuxieme fois)
jusque là tout va bien sauf pour le premier getchar où je dois taper 2 fois sur entrée plutot qu'une... comment se fait-il ?

lien vers mon prog (voir le sous programme une_pause();
http://pastebin.com/429647

Je préconise un usage maitrisé et exclusif de fgets(). Le reste, c'est trop fort pour moi et ça me prend la tête. Pas le temps pour ces c*nneries...

Dans mon interet, je vais revenir à des choses plus simples (je penses aux saisies) afin de mieux comprendre le C... J'ai bien compris mes maladresses même si je les ai remis sur le tapis, c'était en fait pour mieux exposer ma démarche initiale...

Est-ce que vous avez un lien qui m'explique comment faire une pause jusqu'à l'appui d'une touche du clavier ?

Citation:

Envoyé par pimousse92

Est-ce que vous avez un lien qui m'explique comment faire une pause jusqu'à l'appui d'une touche du clavier ?

fgets(). On en sort par <enter>.