Discussion :

[zephyre]

Bonjour,

J'ai lu sur le site developpez.com, ici = https://c.developpez.com/faq/?page=G...buffer-clavier, comment vider le buffer clavier.

Je pense ne rien avoir oublié et pourtant cela ne fonctionne toujours pas.
J'ai des problèmes lors de ma saisie multiples lorsque je lance le programme.
Je ne sais pas si vraiment le problème vient de là mais généralement avec la fonction scanf() c'est ça.

Ci-dessous le code source (environs 45 lignes) :

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
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
 
// BIBLIOTHEQUES :
// ======================================================================================
#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include <string.h>
// ======================================================================================
 
// FONCTION VIDAGE DU BUFFER CLAVIER :
// ======================================================================================
void clean_stdin(void) 
{ 
    int c; 
 
    do
    { 
        c = getchar(); 
    } while (c != '\n' && c != EOF);
}
// ======================================================================================
 
void main(void)
{
	// DECLARATION DES VARIABLES :
	int note = 0;
	float moyenne;
	bool saisieNote = true;
	int somme_note = 0;
	int nombre_note = 0;
 
	// ALGO :
	while(saisieNote)
	{
		while(note < -1 || note > 20)
		{
			printf("Erreur !\n");
			printf("Veuillez recommencer la saisie !\n");
		}
 
		printf("Entrez - 1 pour arretez la saisie.\n");
		printf("Entrez une note : \n");
		scanf("%d\n", &note);
 
		nombre_note++;
		somme_note = somme_note + note;
 
		if(note == -1)
			saisieNote = false;
 
		clean_stdin();
	}
 
	moyenne =  (float) ( ((float)somme_note) / ((float)(nombre_note)) );
 
	printf("Moyenne de la classe : %f\n", moyenne);
}

Es ce que quelqu'un à une idée ?
Merci pour votre aide.

[chrtophe]

Pour commencer, tu dois analyser le retour de scanf.

Je préfère l'utilisation de fonction fgets.

[Sve@r]

Bonjour

J'ai lu sur le site developpez.com, ici = https://c.developpez.com/faq/?page=G...buffer-clavier, comment vider le buffer clavier.

C'est une mauvaise solution. Un buffer clavier ne se vide pas, il se gère. Et s'il contient des infos utiles? Si le programme est mis en sortie de pipe?
chrtophe a donné la bonne solution. D'abord fgets() pour récupérer la ligne saisie, puis sscanf() pour extraire la note de la ligne. Et en vérifiant son retour pour savoir si elle a pu ou pas extraire ladite note.

Quant à ton code, supprime le "\n" au scanf() et tu pourras saisir des notes.

Je ne sais pas si vraiment le problème vient de là mais généralement avec la fonction scanf() c'est ça.

Normal, scanf() est fait pour faire se la saisie "formatée" et pas de la saisie clavier. Si tu l'utilises pour un travail pour lequel elle n'a pas été prévue, normal que ça ne fonctionne pas correctement.

Autres détail: il n'y a rien qui te choque dans la boucle while (note < -1 || note > 20)? Une boucle où rien ne change?? Et ton calcul? Tu additionnes la note y compris si elle vaut -1...
Et t'as pas besoin de tout caster en float. Pour qu'une opération soit en float il suffit d'un seul float dans l'un des opérandes. Attention toutefois à l'ordre des opérations car dans un calcul style 10/3+1.0 le premier calcul 10/3 ne se fait pas en float. Donc un float suffit mais à condition qu'on le place de façon judicieuse.
Et main() n'est pas de type void.

[zephyre]

Bonjour,
C'est noté, ou peut t'on trouver le code source de la fonction fgets() et de la fonction sscanf().
En cherchant sur le net je n'ai trouvé que les prototypes de ces 2 fonctions ?
Il faut déjà que je comprenne comment fonctionne ces 2 fonctions.
Exemple ici :
https://devdocs.io/c/io/fgets

Autres détail: il n'y a rien qui te choque dans la boucle while (note < -1 || note > 20)? Une boucle où rien ne change?? Et ton calcul? Tu additionnes la note y compris si elle vaut -1...
Et t'as pas besoin de tout caster en float. Pour qu'une opération soit en float il suffit d'un seul float dans l'un des opérandes. Attention toutefois à l'ordre des opérations car dans un calcul style 10/3+1.0 le premier calcul 10/3 ne se fait pas en float. Donc un float suffit mais à condition qu'on le place de façon judicieuse.
Et main() n'est pas de type void.

Je vais corriger tout ça.
T'inquiète pas Sve@r, j'ai vraiment écrit mon code en quelques minutes.
Je boucle à l'infini, y a pas de condition de sortie dans la boucle while.
Et j'additionne le -1 dans le calcule de la moyenne.
Laisse moi juste du temps.

[Sve@r]

C'est noté, ou peut t'on trouver le code source de la fonction fgets() et de la fonction sscanf().

Euh??? A priori je dirais "dans les sources de la libC". Mais je ne pense pas que ça t'apporte grand chose (en plus les trucs ils doivent être hardcodés de ouf que c'en sera incompréhensible).

En cherchant sur le net je n'ai trouvé que les prototypes de ces 2 fonctions ?
Il faut déjà que je comprenne comment fonctionne ces 2 fonctions.

Tout ce qu'il y a à savoir c'est que fgets() fait saisir une chaine et que sscanf() peut extraire des éléments de cette chaine (et renvoie le nb d'éléments extraits ce qui permet de vérifier que ça a fonctionné)

Code c :

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

1
2
3
4
5
6
7
8
9
10
int saisieInt() {
	char str[1024 + 1];
	int n;
	while (1) {
		fputs("Entrez un nombre :", stdout); fflush(stdout);
		fgets(str, 1024 + 1, stdin);
		if (sscanf(str, "%d", &n) == 1) return n;
		fputs("Vous n'avez pas saisi un nombre correct !!!\n", stdout);
	}
}

Pas 100% robuste (si on a un abruti qui rentre plus de 1024 caractères ça va merder) mais ça peut résoudre 99% des cas. On peut faire 100% robuste mais on entre là dans un domaine de ratio "probabilité de risque/coût de développement pour s'en protéger".

Laisse moi juste du temps.

Ok, prends ton temps, on sera encore là quand tu reviendras

[zephyre]

Je connais maintenant les éléments de base en langage C.
Je pense que c'est une bonne idée de lire le code source d'une fonction pour comprendre ce qu'elle fait.

Cela permet de se familiariser avec la langage en question et en plus c'est excellent pour développer la logique algorithmique.
Je vais donc essayer même si c'est hardcoder, et si j'y arrive pas je laisse tomber.

Elle est sûrement installé sur le système en dur sinon mon programme serait incapable de tourner.
La question c'est ou ?

Comme ça je serai en mesure une fois que j'aurais acquis un certains niveau en langage C, de créer mes propres fonctions à partir de 0.
Je sais qu'il faut faire absolument du code source qui puisse être réutilisable et portable d’où la recommandation d'utiliser les bibliothèques standards.
Cependant dans un premier temps je ne pense pas que ce soit une bonne idée, car on ne réfléchis pas à l'algo qu'il y a derrière.
On se contente "bêtement" d'appliquer la fonction.

Et de plus on doit être capable de s'en sortir dans n'importe quelles situations, donc autant commencer par recoder (dans une certaine mesure) toutes les fonctions des bibliothèques standard.

Je pense que c'est un excellent exercice en plus.

Autrement dit, il est impossible d'innover si on ne maîtrise pas l'existant.

[Sve@r]

Je pense que c'est une bonne idée de lire le code source d'une fonction pour comprendre ce qu'elle fait.

Dans l'absolu oui. Dans la pratique...
Là le souci c'est que tu t'attaques à des fonctions très bas niveau (elles vont lire le buffer). Mais bon fais tes propres expériences, les sources sont disponibles.

Elle est sûrement installé sur le système en dur sinon mon programme serait incapable de tourner.

Héhé... tu connais le C mais pas encore trop la chaine de compilation.
La chaine de compilation, c'est ".c => .o => .so (ou .a) => exécutable". Mais l'exécutable n'a pas besoin des ".c" et ".o" pour tourner (il a quand-même besoin des ".so" mais ce sont eux-aussi des compilés).
Mais t'es pas obligé de me croire. Donc tu crées un ".c" quelconque (qui dit "Hello"), tu le compiles, puis tu l'effaces => l'exécutable continuera de fonctionner.

La question c'est ou ?

De partout sur le net => https://android.googlesource.com/pla.../stdio/fgets.c

Comme ça je serai en mesure une fois que j'aurais acquis un certains niveau en langage C, de créer mes propres fonctions à partir de 0.
Je sais qu'il faut faire absolument du code source qui puisse être réutilisable et portable d’où la recommandation d'utiliser les bibliothèques standards.
Cependant dans un premier temps je ne pense pas que ce soit une bonne idée, car on ne réfléchis pas à l'algo qu'il y a derrière.
On se contente "bêtement" d'appliquer la fonction.

C'est un peu pareil avec ceux qui construisent des avions. Ils ne s'embêtent pas à construire des roues, ils se contentent de "bêtement" acheter des roues standard car ce n'est pas ça qui fait la force d'un avion.
Tu auras déjà pas mal de travail à créer ton algo en utilisant ce qui est disponible sur étagère, autant éviter de se mettre un handicap inutile. Je t'assure ça ne te fera pas progresser.

Et de plus on doit être capable de s'en sortir dans n'importe quelles situations, donc autant commencer par recoder (dans une certaine mesure) toutes les fonctions des bibliothèques standard.

Oui, et aussi être capable de faire du feu avec des silex, sait-on jamais si les briquets disparaissent...

Je pense que c'est un excellent exercice en plus.

Oui, excellent pour ceux qui ont un niveau plus élevé que "connaitre les bases". Pour ceux qui veulent écrire des drivers par exemple. Et pour ceux qui ont le temps

Autrement dit, il est impossible d'innover si on ne maîtrise pas l'existant.

Il faut aussi savoir faire la part des choses et s'autoriser un "niveau plancher" au dessous duquel il est inutile d'aller voir.

[chrtophe]

Pas 100% robuste (si on a un abruti qui rentre plus de 1024 caractères ça va merder) mais ça peut résoudre 99% des cas.

En quoi ça va merder ? Si le buffer est dépassé avec fgets la lecture s'arrête non ?

[koala01]

Salut

En quoi ça va merder ? Si le buffer est dépassé avec fgets la lecture s'arrête non ?

Oui, a priori, tu n'auras pas de problème avec le contenu de str...

Par contre, le buffer du clavier n'aura pas été vidé, car il y restera tous les caractères supplémentaires introduits par cet abruti

C'est à dire que tu risques de te retrouver avec une sortie complètement à l'est à l'extraction suivante

[Sve@r]

En quoi ça va merder ? Si le buffer est dépassé avec fgets la lecture s'arrête non ?

C'est à dire que tu risques de te retrouver avec une sortie complètement à l'est à l'extraction suivante

Voilà. C'est à ça que je pensais