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

[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?

[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

[Christophe]

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]

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

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