Discussion :

[philippppe]

Bonjour!

J'essaye de vérifier la saisie utilisateur en ne prenant que des entiers. A votre avis, que se passe t-il en matière de mémoire et de rapidité?
Pour le compiler j'ai utiliser l'option -Wall et j'ai un warning à cause de l'expression régulier:

Code :

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

scanf(" %[01]",&valid);

Je n'ai pas non plus pu ecrire:

Code :

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

scanf(" %3[0-9]",&nb);

Comment ça se fait?

Et voici 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
while(valid!=49){
		while(verifNb!=1){
			puts("\nEcrivez un entier\n");
			verifNb = scanf(" %3d",&nb);
			viderBuffer();
			}
		verifNb=0;
		printf("Avez-vous bien ecrit: %d\n",nb);
		while(valid!=49 && verifValid !=1){
			puts("Taper 0 pour non ou 1 pour valider\n");
			verifValid = scanf(" %[01]",&valid);
			viderBuffer();
			}
		verifValid=0;
	}
	printf("%d est valide\n",nb);

Merci par avance pour vos conseils!

[foetus]

Le format de saisie (documentation cplusplus.com de la fonction printf en anglais)

Maintenant, regarde bien ton code

[Sve@r]

Bonjour

Merci par avance pour vos conseils!

Mon premier conseil est qu'il n'est jamais nécessaire de vider le buffer (ta fonction videBuffer() que tu appelles toutes les 3 mn). Déjà parce que celui qui gère bien ses IO ne se retrouve jamais avec un buffer contenant des trucs qu'il n'est pas censé contenir. Et surtout parce qu'un buffer ce n'est pas forcément le clavier. Si par exemple tu es dans le monde Unix, un programme peut recevoir ses données depuis un autre programme via un pipe et dans ce cas le buffer c'est la sortie de l'autre programme => si tu le vides inconsidérément tu perds les données de l'autre programme. Accessoirement j'espère que cette fonction videBuffer() ne travaille pas à coup de fflush(stdin)...

Sinon moi quand je veux "sécuriser" mes saisies je passe par fgets() + sscanf(). Le fgets() c'est pour récupérer le contenu de mon clavier sans me préoccuper de sa nature et le sscanf() c'est pour extraire le nombre se trouvant dans le contenu extrait.
Exemple

Code c :

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
int saisieInt(char *prompt) {
	char tampon[1024 + 1];
	int nb;
 
	while (1) {
		fputs(prompt, stdout);
		fflush(stdout);
		fgets(tampon,1024 + 1, stdin);
		if (sscanf(tampon, "%d", &nb) > 0) break;
		printf("Saisie incorrecte, recommencez !!!\n");
	}
	return nb;
}
 
int main() {
	int nb=saisieInt("Votre nombre ?");
	printf("nb=%d\n", nb);
}

Il restera un petit bémol à cette façon de faire: si l'entrée est plus grande que 1024 il restera des trucs sans stdin. Dans la majorité des cas cette probabilité peut être négligée (ratio "danger/coût de codage") d'autant plus que la valeur 1024 peut être mise en macro ce qui permet de recalibrer facilement le code si nécessaire. Et si vraiment c'est problématique alors on peut remplacer fgets() par getline() qui, elle, récupère toute une ligne quelle que soit sa taille (mais fatalement elle est plus lourde).

[philippppe]

Le format de saisie (documentation cplusplus.com de la fonction printf en anglais)

Maintenant, regarde bien ton code

Je suis désolé, j'avais déjà vu ce genre de texte et je ne le comprends pas.
Pourquoi scanf(" %[01]",&valid); genere un warning?

Bonjour [...]
.

Dans mon cas il s'agit de récupérer des entiers saisis au clavier, de taille max imposés et de rejeter tout le reste. Je ne comprends pas pourquoi tu me conseilles autre chose que scanf.
Pour ce qui est de vider le buffer toutes les trois minutes, je croyais bien faire en renettoyant après chaque utilisation de scanf. ( et non je n'ai pas fflush(stdin)...)

quand est-il des trois while, est-ce tres gourmand en ressources?

[Bousk]

Et c'est censé vouloir dire quoi %[01] comme pattern ?
Parce que vu que c'est absolument pas permis comme syntaxe, faudrait peut-être commencer par savoir ce que tu espères faire.

Récupérer seulement 0 ou 1 ? %1d et tu vérifies que c'est correct.
Mais dans un cas aussi trivial, autant récupérer la string entière, et donc éviter tous les maux de tête de buffer pas entièrement vidé et autre, et vérifier '0' ou '1'.

[Sve@r]

Pourquoi scanf(" %[01]",&valid); genere un warning?

Peut-être parce que le caractère "%" implique un format indiquant à la fonction le type qu'elle devra récupérer (%d=int, %f=float, etc...) et que toi tu n'en mets pas...

Dans mon cas il s'agit de récupérer des entiers saisis au clavier, de taille max imposés et de rejeter tout le reste. Je ne comprends pas pourquoi tu me conseilles autre chose que scanf.

Parce que scanf (scan formaté) implique un truc "formaté" et que ce que tape l'utilisateur est tout sauf formaté. Rien que le <return> qu'il doit taper pour valider son int fout déjà la dawa dans le code car il n'est pas traité par "%d" et reste donc à faire chier la saisie suivante (et bonjour la migraine si la saisie de ce fameux int a lieu dans une alternative => un coup elle y est un coup elle n'y est pas !!!). D'où mon fgets() initial pour vider tout stdin tout en le récupérant pour le traiter plus tard.

Pour ce qui est de vider le buffer toutes les trois minutes, je croyais bien faire en renettoyant après chaque utilisation de scanf.

"je croyais", "renettoyer" tout est là. Comme je l'ai dit, celui qui gère ses IO sait en permanence ce que contient son buffer et n'a donc pas besoin de "renettoyer" quoi que ce soit. Si tu écris int a=0, tu sais ce que contient "a". Tu ne vas pas réécrire a=0 toutes les 15 lignes en croyant bien faire...

qu'en est-il des trois while, est-ce tres gourmand en ressources?

Pas spécialement (il y a pas de calcul compliqué, pas de tableaux à déplacer). Mais bon ce n'est pas non plus le summum de la programmation. Déjà tu as deux variables différentes (verifNb et verifValid) pour faire la même chose à deux moments différents (une variable suffit donc). Plus tu testes deux fois valid != 49 (pourquoi 49 d'ailleurs?) et tu affectes ta variable verifNb sans la traiter. Ce n'est pas grand chose mais c'est toujours plus gourmand que si tu ne le faisais pas.

[foetus]

Je suis désolé, j'avais déjà vu ce genre de texte et je ne le comprends pas.
Pourquoi scanf(" %[01]",&valid); genere un warning?

Arrête de faire le toto le format est %[flags][width][.precision][length]specifier

• Il est où le specifier (spécificateur en français apparemment) ?
• Les crochets ne sont pas autorisés : c'est pour dire que le champs est facultatif "The format specifier can also contain sub-specifiers: flags, width, .precision and modifiers (in that order), which are optional"

[philippppe]

merci pour vos agreables reponses d'êtres supérieurs... toute mes excuses d'avoir posté une question de debutants sur un forum d'expert.
Je quitte ce site plein de compréhension et de bienveillance. bravo les gars

[foetus]

merci pour ton agréable question "Pourquoi scanf(" %[01]",&valid); genere un warning?" posée 2 fois... toute mes excuses d'avoir répondu à une question de débutants alors qu'il n'y a qu'à lire la documentation [en anglais certes].
Je te remercie pour ta gentillesse "ce site plein de compréhension et de bienveillance". bravo O.P. (<- original poster)


Édit : Voici le warning

Code :

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

1
2
3
4
5
6
7
main.c:15:15: warning: format ‘%[01’ expects argument of type ‘char *’, but argument 2 has type ‘int *’ [-Wformat=]
   15 |     scanf("%[01]",&valid);
      |            ~~~^   ~~~~~~
      |               |   |
      |               |   int *
      |               char *
      |            %l[

J'en conclus que tu ne sais pas lire l'anglais. Ton format ne passe pas et toi, au lieu de regarder la documentation, tu viens poster

[chrtophe]

j'ai un warning à cause de l'expression régulier

Je ne savais pas que scanf gérait les expressions régulières (je suis pas programmeur), mais encore faut il les présenter correctement (pas de % devant).
C'est bien indiqué dans la doc.

Il restera un petit bémol à cette façon de faire: si l'entrée est plus grande que 1024 il restera des trucs sans stdin.

Sauf que dans le cas de figure ici présent, l'entrée est sensée être un entier. Si tu dois traiter un entier de 1024 caractères ...... Je pense que tu as été plus que large.

[Sve@r]

Sauf que dans le cas de figure ici présent, l'entrée est sensée être un entier. Si tu dois traiter un entier de 1024 caractères ...... Je pense que tu as été plus que large.

Non désolé que tu n'aies pas pas bien compris l'idée que je voulais exposer. Il ne faut pas voir "taille allouée vs taille utilisée" mais "taille allouée vs taille saisie". Si l'utilisateur à qui on demande "Votre nombre ?" a dans l'idée de taper plus de 1024 octets, ce qu'il a tapé et qui dépasse 1024 reste alors dans stdin. Et ça peut être alors un souci...

[Invité]

Bonjour,

Quitte à diriger vers la documentation, autant diriger vers la bonne fonction : scanf. Ce qui permettra de s'apercevoir que ce format "%[01]" est parfaitement valide.
(Et oui, il faut le % devant, sinon la fonction attendra littéralement « [01] » comme entrée, pas juste « 0 » ou « 1 », et n'aura pas de paramètre d'entrée correspondant.)

Le soucis, c'est que ce format n'attend pas l'adresse d'un entier comme paramètre correspondant mais celle d'une chaîne de caractères, comme le rappelle très bien le message d'erreur donné par foetus :

main.c:15:15: warning: format ‘%[01’ expects argument of type ‘char *’, but argument 2 has type ‘int *’ [-Wformat=]

Toutefois ici tu souhaites juste saisir « 0 » ou « 1 », pas des suites du genre « 1010101 », ce que permet "%[01]" et qui engendrera entre autre des risques de dépassement de buffer (tout comme "%s").
On limite donc à un seul caractère : "%1[01]", mais on n'oublie pas de prévoir le '\0' marquant la fin de la chaîne :

Code :

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

char valid[2];

Tu pourras dès lors vérifier le retour de la fonction pour t'assurer que l'utilisateur a bien saisi un « 0 » ou un « 1 » :

Code :

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

if (scanf(" %1[01]", valid) == 1) // 1 = nombre de paramètres lus avec succès, ici un seul : valid

et vérifier le caractère saisi avec if (valid[0] == '0' /* ou '1' */) (ou encore if (strcmp(valid, "0" /* ou "1" */) == 0)).

Note : tu as bien fait d'insérer un espace avant le % de ce format, car contrairement aux autres formats comme "%d", celui-ci n'ignore pas les éventuels caractères blancs situés avant. Entre autres le '\n' venant de la touche Entrée d'une saisie précédente. Il traiterait ainsi le '\n' plutôt que le '0' ou '1' qui suit, ce qui ferait échouer la lecture.

Pour approfondir le fonctionnement de cette fonction, je recommande la lecture de ce tutoriel Scanf démythifiée.

Bien sûr, sans doute plus simple, tu as aussi la possibilité d'utiliser un entier avec le format "%d" et vérifier derrière s'il est égal à 0 ou 1.

[chrtophe]

Merci Winjerome pour cette explication.

Pour moi la doc officielle n' est pas très explicite sur cet aspect. Dans une regex au sens large, on peut indiquer le nombre de caractère concerné, j'aurais pas eu pour logique de placer le % en amont.

donc ne pas tenir compte de mon 1er post.

[kaitlyn]

Salut,

Elle est là ton erreur et certainement celle de l'auteur de la discussion, c'est de croire que scanf prend en charge les expressions régulières. Il n'en est rien. Tout ce que (f)scanf accepte et comprend c'est une chaîne de contrôle, c'est un ensemble de directives où le symbole modulo % annonce une spécification de conversion. Dans les faits cela se traduit par le pilotage séquentiel d'un rudimentaire parseur, un peu comme ce qui se fait pour parser la ligne de commande si c'est plus parlant. Donc c'est vraiment quelque chose de très sommaire au regard des processeurs, compilateurs et moteurs impliqués dans les expressions régulières.

[foetus]

Quitte à diriger vers la documentation, autant diriger vers la bonne fonction

Effectivement, la documentation cplusplus.com de scanf en anglais

Et tu as raison je me suis trompé, croyant que le format printf est le même que celui de scanf. Les différences sont

• un format %[ * ][width][length]specifier plus simple sans flags ni .precision - [sûrement] plus adapté à la saisie qu'à l'affichage
• un specifier (spécificateur) ayant en + 2 ajouts "expression régulière" [characters] (scanset) et [^characters] (negated scanset). Et 2-3 trucs en moins (A, G, X)

Pour 1 exemple, il faut rechercher sur les internets scanf scanset.
Mais effectivement ce spécificateur est assez confusant parce qu'il ne spécifie pas explicitement le type, mais "Any number of the characters" et "Any number of characters none of them" indiquent bien 1 chaîne de caractères en entrée.
Ce qui semble logique (l'entrée étant chaîne de caractères si je ne me trompe pas, ainsi il n'y a aucune conversion que des tests) et ce que le warning indique également.

[chrtophe]

Elle est là ton erreur et certainement celle de l'auteur de la discussion, c'est de croire que scanf prend en charge les expressions régulières.

En tout cas la syntaxe est similaire, il doit probablement y avoir une raison.

[kaitlyn]

En tout cas la syntaxe est similaire, il doit probablement y avoir une raison.

Oui les 26 lettres de l'alphabet.

Non, plus sérieusement. Tout ça est issu du monde scientifique. Pour construire une expression compacte et concise on fait systématiquement appel à l'usage de symboles et de lettres, c'est culturel. Que dans le cas présent tu vois des similitudes entre ces deux grammaires c'est tout à fait normal, mais ça ne reste que des façades.